A teď něco vážného… Whitespace in source code

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
David Grudl
Nette Core | 8110
+
0
-

Nejsou tři volné řádky mezi metodami moc? Nebyly by lepší dva?

Dva řádky namísto tří by zkrátily třídu Presenter ze strašidelných 1465 řádků na snesitelných 1406.

Majkl578
Moderator | 1364
+
0
-

A co jen jeden?

Editoval Majkl578 (2. 7. 2013 16:18)

Patrik Votoček
Člen | 2221
+
0
-

jeden

sifik
Člen | 27
+
0
-

Jsem také pouze pro jeden

VasekPurchart
Člen | 20
+
0
-

+1 k 1

Editoval VasekPurchart (2. 7. 2013 16:22)

Filip Procházka
Moderator | 4668
+
0
-

Nejsou, lépe se to čte když je tam víc prázdných řádků.

David Grudl
Nette Core | 8110
+
0
-

Dva jsou příjemnější než jeden. Jeden vypadá dobře, když je nad metodou phpDoc a editor ji zobrazí světle šedou, takže se vytvoří iluze více místa.

MartinitCZ
Člen | 580
+
0
-

Jeden je málo, tři moc. Jsem pro dva řádky.

David Grudl
Nette Core | 8110
+
0
-

Vyzkouším chvíli pracovat se dvěma a uvidíme. Taky bych nerad, aby to pomátlo pull requesty, blame apod.

Šaman
Člen | 2634
+
0
-

Jeden používám na oddělení logických kusů kódu, mezi metodami se mi osvědčily dva. Spolu s PhpDoc je to oddělení imho dostačující. Tři používám jen výjimečně, pokud chci opticky seskupit několik metod, které k sobě logicky patří. Takže hlasuji pro dva.

redhead
Člen | 1313
+
0
-

Dva.

Grelek
Člen | 233
+
0
-

Jeden.

David Matějka
Moderator | 6445
+
0
-

2

David Grudl
Nette Core | 8110
+
0
-

Takže tohle je ta slibovaná revoluční změna ve frameworku https://github.com/…6fddb858cfbc!

Majkl578
Moderator | 1364
+
0
-

Prima, ušetřili jsme skoro 4kB, jak významné. ;)

Editoval Majkl578 (2. 7. 2013 19:53)

Šaman
Člen | 2634
+
0
-

Hurá.. doufám, že se to podaří odladit před vydáním verze 2.1, aby se to mohlo stát její nosnou součástí. :D

paranoiq
Člen | 392
+
0
-

to je až příliš odvážný krok. to programátorský svět nikdy nepřijme!

ic
Člen | 430
+
0
-

Nějak ty zdrojáky moc neodpovídají dnešním širokoúhlým monitorům… leda tak na výšku.
Každopádně v žádném případě nejsem fanda 2 řádků, to vypadá, jako by člověk chtěl dát jeden a omylem se uklep a dal ty entery 2. Letmým pohledem se dvojka hůře odlišuje od 1 odřádkování.

Proto myslím, že 1 nebo 3 ale určitě né 2 XD

nAS
Člen | 277
+
0
-

Podle nadpisu jsem se bál, že se tu bude hlasovat o změně TABů na mezery. :)

David Grudl
Nette Core | 8110
+
0
-

Jakože udělat po pětiletém výročí revert? ;) https://github.com/…369f47c3896a

nanuqcz
Člen | 822
+
0
-

Neee, neberte mi tuhle feature :-( Díky Nette frameworku jsem začal používat trojí odřádkování mezi metodama. Teď se budu muset vrátit ke dvojímu, což je pro mě krok zpátky!

rixi
Člen | 109
+
0
-

3 a nanuqczovi +1 :)

Editoval rixi (3. 7. 2013 10:19)

jiri.pudil
Nette Blogger | 1028
+
0
-

Začíná mi to tu připomínat http://xkcd.com/1172/

MartinitCZ
Člen | 580
+
0
-

nanuqcz napsal(a):

Neee, neberte mi tuhle feature :-( Díky Nette frameworku jsem začal používat trojí odřádkování mezi metodama. Teď se budu muset vrátit ke dvojímu, což je pro mě krok zpátky!

Tohle nechápu. Však tě nikdo nenutí to měnit i u tvých projektů, je to jen tvá vůle. Nadále můžeš používat 3 odřádkování. Nette není žádné náboženství, nedej bože sekta!

uestla
Backer | 796
+
0
-

Opticky se mi kód při 3 četl lépe.

Osobně u nich zůstanu, jen se mi bude Nette hůř číst…