[kodstruktur] vad är snyggast?

if(villkor) {
 
}
else {

}

enda rätta.

Röstar också på Jonasys

lola, 50 / 50

hahaha

if(1)
{
}
else
{
}

- naturligtvis!

Under dom givna alternativen så vägrar jag delta i röstningen, båda förslagen är förkastliga enligt undertecknad. Röstar på Jonasys.

Kör på "hans" sätt, dvs:

void foobar( int foo )
{
  if( foo )
  {
     char *s = "foo";
  }
  else
  {
     char *t = "bar";
  }
}

Detta eftersom man ser direkt vilka klamrar som hör ihop, oavsett om det är if/for/while/do eller en funktion.

Skriver man på "ditt" sätt måste man ju scanna med blicken tvärs över koden vilket jag tycker är extremt jobbigt, känns inte som man får samma överblick. Man måste ju alltid kolla slutet på raden efter ett { för att vara säker på att det inte bara är en enkel if/for/while/do. Thomas har ett bra exempel där.

Jag håller med jonasy

Det har blivit en hel del Symbian på senaste tiden och då är det följande som gäller:

if ( foo )
    {

    // koda här

    }
else
    {

    // koda mer här

    }

... även om jag personligen inte gillar det. =/

char *foo;

if (bar)
{
    /* Gör något coolt med foo */
}
else
{
    /* Gör något _ännu_ coolare med foo */
}

Så där gör jag. Alla andra sätt är naturligtvis fel, men jag förlåter er.

Kan vi inte ha en diskussion om vilken editor som är bäst också?

char *foo;

if (bar)
{
    /* Gör något coolt med foo */
}
else
{
    /* Gör något _ännu_ coolare med foo */
}

Det är ju lätt, Visual studio.

OnT: Jag använder trådskaparens metod.

Med brace matching undviker man att första måsvingen tar upp en hel egen rad bara för att öka läsbarheten.

Ruby \o/

Det viktiga när man kör Ruby (och använder {} istället för do-end) är att du får syntax-error om du försöker använda BSD-style

Jag brukar bara använda braces i Ruby för one-liners, så behöver inte bry mig.

andledningen till att jonasys är den bästa är att för varje closing-bracket kan du ju spåra vilken sats den tillhör genom att bara gå rakt uppåt.

Sen blir det ju inte så jävla många extrarader.

BSD/Allman, dvs hans sätt.

bool CBoard::ZapChains()
{
	ClearMarks();
	bool Stable = true;
	for(int32 y = 0; y < EHeight; y++)
		for(int32 x = 0; x < EWidth; x++)
		{
			int32 Count = FindChain(x, y);
			if(Count >= 4)
			{
				ClearMarks();
				m_aZaps[m_NumZaps].m_x = 0;
				m_aZaps[m_NumZaps].m_y = 0;
				m_aZaps[m_NumZaps].m_Count = ZapChain(x, y);
				m_aZaps[m_NumZaps].m_x /= m_aZaps[m_NumZaps].m_Count;
				m_aZaps[m_NumZaps].m_y /= m_aZaps[m_NumZaps].m_Count;
				m_NumZaps++;
				Stable = false;
			}
		}
	return Stable;
}

(snip ifrån teepop)

Beror helt enkelt på HUR JAG vill att den ska vara...
Vill jag att koden ska vara lättläslig för andra personer, då kör jag hans.
Annars kör jag lite blandat hej vilt
Har provat att koda ett program på EN rad. Trodde jag skulle dö efter några funktioner

Ex.

#include <iostream> using namespace std; int main() { int i; for(i = 0; i < 10; i++) { cout << "Hello World" << endl; }; return 0; }

Tror det avslutar diskussionen. Om matricks säger att BSD/Allman är rätt så är det så.

Om er kod skiljer sig, please adjust at your earliest convenience.

Diskussionen är inte slut än, tycker att Jonasys sätt är bäst, antagligen eftersom det var så jag fick lära mig första gången jag började leka med programmerande (C).

if(villkor) {
 
}
else {

}

den gillar jag

btw röstade på 2:an

Hans självklart. Får alltid ryck och skriver om andras kod om de använt din stil. Tycker det ser slarvligt ut.

Men, char* hejsan, antyder enligt mig iallafall, att vi har en pekare av typen char vilket känns rätt så väsentligt.

Tynnhammar, medans char *hejsan antyder att hejsan är en pekare, vilket känns rätt så väsentligt.
Det beror helt på hur man tittar på det.

Är det datatypen som är viktig eller är det pekaren i sig som är viktig?

Personligen så gör jag char *foo eftersom jag tänker: "foo är en pekare".
Hade jag tänkt "En char-pekare som heter foo" så hade jag gjort char* foo.

Jag röstar på din vän, även jag hellre hade sett en kombination:

if(villkor){

}
else{

}

Självklart så är det så här som det ska se ut

void CFoo::Func(Foo *foo) {
    while(1 < 2) {
        if(foo == foo) {
            foo->Print();
        }
    }
}

Så jag röstar på dig

"min" , gillar inte när det är för luftigt.

Jag kör på någon modifierad version av K&R...

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[]) {
    int i;  

    if (argc != 2) { 
        fprintf(stderr, "You forgot the number!\n");
        return 1;
    }

    i = atoi(argv[1]);

    if (i == 0)
        printf("Zero\n");
    else if (i == 1)
        printf("One\n");
    else    
        printf("%d\n", i);

    for (i = 0; i < 10; ++i)
        if (i % 2 == 0)
            printf("%d\n", i);

    /* En if-sats (med bracets):
    if (i == 0) {
          printf("Mjew!\n");
    }
    else if (i == 1) {
           printf("Bidew!\n");
    } else {
           printf("Annat\n");
    }
    */

    return 0;
}

Undviker {} så långt det går.

EDIT: Byggde på lite.
EDIT2: Och om vi ska börja diskutera editor: Vim är det enda jag rör numera. Jag vägrar använda någonting annat som kräver att jag sträcker mig efter piltangenterna.

När jag kodar PHP ser det ut så här:

if($tjosvejs){
   echo "hej hej";
}else{
   echo "foo";
}

Medan det ser ut så här i c++

if(tjohoo == 1)
{
     printf("hej hej");
}else
{
     printf("foo");
}

Men man kanske skulle gå över till att flytta ner else-satsen...

Alt. 1.

skriver så här:

if(whatever == TRUE) {
   do_this();
} else {
   do_that();
}

Kör som morec, dvs Min + mellanslag. Gillar inte när det är en svårläst grötig soppa, utan vill ha koden luftig och tydlig.

Jag kör oftast på "hans" sätt eftersom jag tycker att det blir mer lättläst, det blir så väldigt kompakt och ihoptryckt annars. Men jag kan tänka mig att det blir mer lättläst/överskådligt med "ditt" sätt om man har en skärm med låg upplösning eftersom man inte får plats med så mycket text då.