Erste Versuche mit Container und Iterator

SeppJ

cpp_beginner_offl schrieb:

Auf der Suche nach "Style" bin ich irgendwann bei "passt schon" hägengeblieben.
Ich wäre für jede Anregung eines Profis dankbar!!

Hauptaufreger ist wohl diese Stelle:

if(check == YES && (it+1 != myData.end()) ){
        std::cout<<"\n\tInserted!!\n";     
        it=myData.insert(it+1,spec ) ;
        spec+=100.0;
    }  else{
             std::cout<<"\nInsert not possible or requested\n";
       }

Das ist schlecht, da die Logik des if-else-Konstrukts nicht gut sichtbar ist. Besser:

if(check == YES && (it+1 != myData.end())){
    std::cout<<"\n\tInserted!!\n";     
    it=myData.insert(it+1,spec ) ;
    spec+=100.0;
  } 
  else{
    std::cout<<"\nInsert not possible or requested\n";
  }

Ich persönlich hasse die '{' am Zeilenende und würde es so machen:

if(check == YES && (it+1 != myData.end()))
    {
      std::cout<<"\n\tInserted!!\n";     
      it=myData.insert(it+1,spec ) ;
      spec+=100.0;
    } 
  else
    {
      std::cout<<"\nInsert not possible or requested\n";
    }

Dann gehen wir nochmal den Code selbst durch und setzen ein paar Leerzeichen und entfernen ein paar an anderer Stelle:

if(check == YES && (it + 1 != myData.end()))
    {
      std::cout << "\n\tInserted!!\n";     
      it = myData.insert(it + 1, spec);
      spec += 100.0;
    } 
  else
    {
      std::cout << "\nInsert not possible or requested\n";
    }

(Wo wir schon dabei sind: Multiple exclamation marks are a sure sign of a diseased mind)

Allgemein:
http://en.wikipedia.org/wiki/Indent_style
Wichtig ist jedenfalls, dass die logische Blockstruktur des Codes gut sichtbar ist. Ebenso macht man das bis hinunter auf die Ebene der Ausdrücke. Zwischen Ausdrücken und den Operatoren, die sie verbinden, kommt ein Leerzeichen, um die logische Trennung hervorzuheben:
http://en.wikipedia.org/wiki/Programming_style#Spaces

Hallo SeppJ, vielen Dank für deine Code Aufsführung , ich werde mir das zu Herzen nehmen, das macht schon einiges Aus. (Bis auf die geschweiften Klammern, da find ich den KR Style "besser".

(Wo wir schon dabei sind: Multiple exclamation marks are a sure sign of a diseased mind)

I guess, that´s right, why else should i start trying to program c++ :-)))

void vect_dyn2(void){ // du brauchst hier kein void in die Klammern schreiben

Müssen wahrscheinlich nicht, aber falsch ist es auch nicht.

volkard

cpp_beginner_offl schrieb:

void vect_dyn2(void){ // du brauchst hier kein void in die Klammern schreiben

Müssen wahrscheinlich nicht, aber falsch ist es auch nicht.

Es ist sehr ungewöhnlich. Die anderen stolpern beim Lesen drüber. Die denken sich, daß Du echt die falschen Bücher liest. Und ich lese mir dann schon die Sachfrage gar nicht durch, da ich regelmäßig die Erfahrung mache, daß Proggern, die so weit auf Abwegen sind, nicht sehr zu helfen ist.

DocShoe

volkard schrieb:

Es ist sehr ungewöhnlich. Die anderen stolpern beim Lesen drüber. Die denken sich, daß Du echt die falschen Bücher liest. Und ich lese mir dann schon die Sachfrage gar nicht durch, da ich regelmäßig die Erfahrung mache, daß Proggern, die so weit auf Abwegen sind, nicht sehr zu helfen ist.

Geht mir bei vielen anderen Dingen ähnlich.

Ich muss dazu sagen, das ich hier im Forum, Abteilung 'C' zurecht gewiesen wurde (hört sich jetzt härter an als es war )dass ich funktionen mit

int mach_was();

festgelegt habe, es müsse int mach_was(void) heißen.
Also ich hab mir das dann so angewohnt, funktionen ohne Parameter mit foo(void) zu machen. Sollte ich mir das wieder abgewöhnen? Prinzipiell ist es ja egal, oder?

Columbo

Wie meinst du das genau?

Ein Element wird immer vor dem Iterator eingefügt der der insert -Methode übergeben wurde. Das geht einher mit rechtsoffenen Intervallen.

Das mit dem Iterator stimmt so?

Ja, aber ich würde es so

for (auto it=myData.begin(); it != myData.end(); ++it)

schreiben. Das mit dem Präfix-Inkrement ist Konsistenz - und auto spart seit C++11 viel Schreibarbeit.

Vielleicht würde ich myData.end() zwischenspeichern falls das eine performance-kritische Schleife ist. Range-based for Schleifen machen das automatisch.

for (auto d : myData)

Offenbar brauchst du aber den Iterator in der Schleife, daher ist das hier nicht einsetzbar.

Nathan

cpp_beginner_offl schrieb:

Ich muss dazu sagen, das ich hier im Forum, Abteilung 'C' zurecht gewiesen wurde (hört sich jetzt härter an als es war )dass ich funktionen mit
int mach_was();
festgelegt habe, es müsse int mach_was(void) heißen.

Richtig, das ist auch gut so.
Denn in C gibt es ein Feature, dass so eine Dekleration für sämtliche Argumente passt, d.h. ebenfalls für int mach_was(int) oder int mach_was(float, float) ( http://ideone.com/WReK4S ).
In C++ gibt es das nicht, weswegen das void überflüssig ist. Einen wirklichen Grund das wegzulassen, außer, dass es alle machen, hab ich aber auch nicht.

volkard

cpp_beginner_offl schrieb:

Ich muss dazu sagen, das ich hier im Forum, Abteilung 'C' zurecht gewiesen wurde

C hat doch gar nichts mit C++ zu tun. Lerne nicht beide Sprachen gleichzeitig.

C hat doch gar nichts mit C++ zu tun. Lerne nicht beide Sprachen gleichzeitig.

Mach ich nicht gleichzeitig. C hab ich für mich "soweit abgeschloßen" (ich bin natürlich Welten von Euch Profis entfernt, für mich ist es aber auch nur ein Hobby, mein Geld verdien ich zwar auch mit Programmieren, aber komplett andere Schiene.), das es für meine Bedürfniße reicht, jetzt will ich hald in C++ "reinschnuppern" bzw. mir da einen Grundstock erarbeiten.

Denn in C gibt es ein Feature, dass so eine Dekleration für sämtliche Argumente passt, d.h. ebenfalls für int mach_was(int) oder int mach_was(float, float) ( http://ideone.com/WReK4S ).
In C++ gibt es das nicht, weswegen das void überflüssig ist. Einen wirklichen Grund das wegzulassen, außer, dass es alle machen, hab ich aber auch nicht.

Jetzt wo du es sagst, stimmt, da war mal was.

Noch was zum Stil:

Variante 1 :

if ( x == y ){
    bla bla;
}

oder
Variante 2 :

if (x == y){
    bla bla;
}

abgesehen von den Klammern, wills jetzt nicht übertreiben mit den Leerzeichen?

volkard

cpp_beginner_offl schrieb:

Noch was zum Stil:

Variante 1 :
if ( x == y ){
    bla bla;
}
oder
Variante 2 :
if (x == y){
    bla bla;
}
abgesehen von den Klammern, wills jetzt nicht übertreiben mit den Leerzeichen?

Ich empfehle, fast alle nutzlosen Leerzeichen wegzumachen. Es ist UNMÖGLICH, einen einigermaßen konsistenten Stil mit Leerzeichen hinzukriegen.

for (auto it=myData.begin(); it != myData.end(); ++it)

Da sieht mans Elende doch schon. != kriegt Leerzeichen aber = nicht. Die Leerzeichenschmücker (leider noch in der deutlichen Überzahl) bemerken diese Unschönheiten scheinbar gar nicht, aber mir kommts Kotzen. () kriegen Leerzeichen drum bei Schleifen aber nicht bei Funktionsaufrufen? Wo ist da der Sinn? Das alles ist eine Sammlung von aberwitzig vielen persönlichen Sonderregelungen.

Also ganz einfach

for(auto it=myData.begin();it!=myData.end();++it)

und dafür die Schriftart um 2-5 Punkte größer einstellen.

SeppJ

Von den beiden: Variante 2.

Wobei mich an stört, wenn die öffnenden und schließenden Klammern nicht in einer Spalte stehen.

Nathan

volkard schrieb:

cpp_beginner_offl schrieb:
Noch was zum Stil:

Variante 1 :
if ( x == y ){
    bla bla;
}
oder
Variante 2 :
if (x == y){
    bla bla;
}
abgesehen von den Klammern, wills jetzt nicht übertreiben mit den Leerzeichen?
Ich empfehle, fast alle nutzlosen Leerzeichen wegzumachen. Es ist UNMÖGLICH, einen einigermaßen konsistenten Stil mit Leerzeichen hinzukriegen.
for (auto it=myData.begin(); it != myData.end(); ++it)
Da sieht mans Elende doch schon. != kriegt Leerzeichen aber = nicht. Die Leerzeichenschmücker (leider noch in der deutlichen Überzahl) bemerken diese Unschönheiten scheinbar gar nicht, aber mir kommts Kotzen.

Ehm, wenn dann ist man auch konsequent:

for (auto it = myData.begin(); it != myData.end(); ++it)

Leerzeichen kommen nach Trennern wie , ; (wie im Deutschen) und vor und hinter binären Operatoren.
Nach und vor Klammern kommen keine Leerzeichen, außer bei if, while, for,... da kommt ein Leerzeichen nach dem Keyword.
Das ist dann auch konsequent.

volkard

SeppJ schrieb:

Von den beiden: Variante 2.

Wobei mich an stört, wenn die öffnenden und schließenden Klammern nicht in einer Spalte stehen.

Mich stört, wenn die schließende Klammer nicht unter dem ersten Zeichen von if/while/for… steht, also Dein Vorschlag

if(...)
  {
    tuwas1();
    tuwas2();
  }
else
  {
    anderes();
  }

von der ersten Seite geht gar nicht.

Sehr spät bin ich umgestiegen von

if(...)
{
    tuwas1();
    tuwas2();
}
else
{
    anderes();
}

nach

if(...){
  tuwas1();
  tuwas2();
}
else
  anderes();

, weil mir die Klammern eher die Übersicht nehmen, als was zu bringen. Ich wünschte mir, allein die Einrückung würde steuern.

if(...)
  tuwas1();
  tuwas2();
else
  anderes();

~(Dann geht natürlich wieder einiges an der Schriftgröße!)~

if ( !(x == y) )  //v1

   vs: 

if (!(x == y))    //v2

Macht wohl jeder wie er will . Für mich definitv v2 ! ( Auch wenn bei umfangreicheren Bedingungen mal 4 )))) zusammenkommen )

hustbaer

Nathan schrieb:

Leerzeichen kommen nach Trennern wie , ; (wie im Deutschen) und vor und hinter binären Operatoren.
Nach und vor Klammern kommen keine Leerzeichen, außer bei if, while, for,... da kommt ein Leerzeichen nach dem Keyword.
Das ist dann auch konsequent.

+1

Wobei ich es lustig finde dass darüber noch diskutiert wird. Ich hab' den Eindruck dass das schon ein quasi-Standard ist. Man sieht nicht viel Code in Open-Source Projekten der was Leerzeichen angeht anders formatiert wäre.
Ist auch die Standardeinstellung von Visual Studio. Bei anderen IDEs nehme ich an dass die Standardeinstellungen auch so sind, aber ausprobiert hab' ich es nicht.

EOP

volkard schrieb:

Sehr spät bin ich umgestiegen von
if(...)
{
    tuwas1();
    tuwas2();
}
else
{
    anderes();
}

Ist imho die übersichtlichste Sache.

UltraEdit macht das sehr schön mit gepunkteten Linien von Block zu Block.

'{' am Zeilenende geht gar nicht. Völlig unübersichtlich.

Und - Klammern müssen schon sein wenn man nicht von Basic kommt. (asm ist wieder ne andere Baustelle)

Fytch

EOP schrieb:

volkard schrieb:
Sehr spät bin ich umgestiegen von
if(...)
{
    tuwas1();
    tuwas2();
}
else
{
    anderes();
}
Ist imho die übersichtlichste Sache.

So sehe ich das auch. Der einzige Nachteil dieser Variante (dass es mehr Zeilen braucht und daher u.U. gewisse Dinge nicht mehr auf den Monitor passen sodass man dauernd scrollen muss) wird durch neuere IDEs, bei denen man Codeblöcke reduzieren / einklappen kann, ausgebügelt.

Kellerautomat

bin ich der einzige, der kein leerzeichen zwischen keyword und klammer setzt?