Göran Krampe skrev 2011-11-18 11.05:
> On 11/18/2011 10:36 AM, Robin Rosenberg wrote:
>> Öppning-{ på egen rad gör att koden tar mera plats utan att göra den
>> tydligare. Viss
>> ska man ha lite tomma rader, men det är bättre att ha tomma
>> rader utan nånting alls på och att placera ut tomma rader av
>> tydlighetsskäl och inte
>> enligt mall.
>
> Håller helt med dig Robin, fast lustigt nog helt utan att ha några direkt vettiga logiska argument annat än att jag tycker det blir svårare (ja, jag skojar inte) att läsa
> med öppning på egen rad. Tycker det ser fult ut. :)
>
> Kanske är det "Smalltalkern" i mig som gör det.
Många metoder och klasser sträcker sig över mer än vad som ryms i ett
fönster och då måste man rulla fram och tillbaka och därför blir det
svårare.
Argumentet för { på egen rad är att det blir lättare att matcha klamrar
för de som använder notepad som IDE.
>> Det tekniska argumentet mot { på egen rad är att om man arbetar med
>> patchar/rebase/cherry-pick
>> så kan ändringar lätt hamna på fel ställe med eller utan konflikt.
>
> Ah, intressant detalj.
>
>> I övrigt är det en religiös fråga.
>
> Ja, så pass religiös att många personer på helt allvar säger att det *ska* vara på ett visst sätt. Hehe :)
Konstigt nog är det de minst betydelsefulla detaljerna som orsakar de största kontroverserna. Det är
dock bra om man försöker motivera varför man gör på ett visst sätt. BSD-stilen som nämndes i ett annat
mail är ett bra sådant exempel. Men om men man nu är i ett projekt var alla sitter med en IDE av något
slag med bra sökfunktioner så faller just de argumenten eller tappar i alla fall lite slagkraft.
> Själv så kör jag "when in Rome, do as the Romans".
Jo. Folk (även jag) blir upprörda över att man ändrar stil och oftast är det till störst
nackdel för den som bryter. Är det rak vänstermarginal som är standard så håller
man sig till det, eller all-caps om det är projektstandard, tills man får starta
ett eget projekt.
-- robin
_______________________________________________
http://www.foss-sthlm.se/
http://cool.haxx.se/cgi-bin/mailman/listinfo/foss-sthlm
Received on 2011-11-18