Wie rückt ihr hier ein?

this-*gt*that

Abend!

Mal eine kleine Frage bzgl. Code Formatierung. Folgende 2 Stile:

// 1) In den folgenden Zeilen eine Einrückung
OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton("MMButton_NewInfo", btnMaterial, 
    "BtnNewInfo.png", "BtnNeInfo_Hovered.png", "BtnNewINfo_Pressed.png", Vec2u(btnX, btnY)

// 2) Alles Bünding mit der öffnenden Klammer
OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton("MMButton_NewInfo", btnMaterial, 
                                                       "BtnNewInfo.png", "BtnNeInfo_Hovered.png", "BtnNewINfo_Pressed.png", 
                                                       Vec2u(btnX, btnY)

Was findet ihr besser bzw. was benutzt ihr in diesem Fall?

Ich persönlich bevorzuge Stil 2, da er für mich übersichtlicher aussieht. Allerdings hat man dadurch mehrere Zeilen für einen Funktionsaufruf, was mir auch nicht sooo gefällt.

brotbernd

Ich mach das erste. Das zweite ist zwar manchmal schön und übersichtlich, aber wie in deinem Beispiel oft auch weniger schön (wenn die Zeile bis zur Klammer schon lang ist).

pyhax

Ich benutz eher folgendes :

OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton(
   "MMButton_NewInfo", btnMaterial,
   "BtnNewInfo.png", "BtnNeInfo_Hovered.png", "BtnNewINfo_Pressed.png",        
   Vec2u(btnX, btnY)
);

Übrigens hast du eine Klammer vergessen

RussianTux

OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton
( 
   "MMButton_NewInfo", btnMaterial, "BtnNewInfo.png",
   "BtnNeInfo_Hovered.png", "BtnNewINfo_Pressed.png",         
   Vec2u(btnX, btnY) 
);

cooky451

eine metthode sollte maximal 3 parameter haben. dann entstehen solche fragen erst gar nicht

knivil

koi schrieb:

eine metthode sollte maximal 3 parameter haben. dann entstehen solche fragen erst gar nicht

Bei sprechenden Variablennamen und begrenzt auf 80 Zeichen pro Zeile ... dann auch!

pyhax

knivil schrieb:

koi schrieb:

eine metthode sollte maximal 3 parameter haben. dann entstehen solche fragen erst gar nicht

Bei sprechenden Variablennamen und begrenzt auf 80 Zeichen pro Zeile ... dann auch!

+ für Container bei STL Algorithmen jeweils noch .begin() und .end() + lambdas für die STL Algorithmen.

this-*gt*that

koi schrieb:

eine metthode sollte maximal 3 parameter haben. dann entstehen solche fragen erst gar nicht

Schwachsinn. Harte Regeln sind immer Bullshit.

Bin jetzt umgestiegen auf Variante 1.

hustbaer

Ich mache auch mehr oder weniger Variante 1.

OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton( // ich finds übersichtlicher wenn hier direkt keine Parameter mehr stehen
    "MMButton_NewInfo", btnMaterial, "BtnNewInfo.png", "BtnNeInfo_Hovered.png",
    "BtnNewINfo_Pressed.png", Vec2u(btnX, btnY));

Ich muss gestehen, dass ich mir weit weniger Sorgen um die Zeilenlänge mache, seit mir das letzte mal ein Monitor kaputtgegangen ist und als Ersatz nur noch Breitbildschirme zu beschaffen waren. Ansonsten mache ich so was ein bisschen nach Gefühl (kommt ja auch nicht dauernd vor) und lasse mich von Emacs leiten, was die Einrückung angeht. In diesem speziellen Fall würde ich wahrscheinlich zwischen alle oder die meisten Parameter einen Zeilenumbruch setzen, weil

OverlayPanel* pnlNewInfo = pnlMainMenu->addChildButton("MMButton_NewInfo",
                                                       btnMaterial,
                                                       "BtnNewInfo.png",          // einem so gleich auffällt,
                                                       "BtnNeInfo_Hovered.png",   // dass hier die Dateinamen
                                                       "BtnNewINfo_Pressed.png",  // merkwürdig aussehen.
                                                       Vec2u(btnX, btnY));

Auch ist es so bequemer, wenn ich mal mit Visual Studio unterwegs bin. Die Dateien sollen in Zukunft ein anderes Präfix haben? Kein Problem - mit Alt-Shift markieren und das neue in allen Zeilen gleichzeitig eingeben (mein Lieblingsfeature in MSVC 2010).