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

Upozornění: Tohle vlákno je hodně staré.

před 7 lety

David Grudl
Nette Core | 7148
+
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.

před 7 lety

Majkl578
Moderator | 1376
+
0
-

A co jen jeden?

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

před 7 lety

Patrik Votoček
Člen | 2248
+
0
-

jeden

před 7 lety

sifik
Člen | 27
+
0
-

Jsem také pouze pro jeden

před 7 lety

VasekPurchart
Člen | 20
+
0
-

+1 k 1

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

před 7 lety

Filip Procházka
Moderator | 4693
+
0
-

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

před 7 lety

David Grudl
Nette Core | 7148
+
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.

před 7 lety

MartinitCZ
Člen | 587
+
0
-

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

před 7 lety

David Grudl
Nette Core | 7148
+
0
-

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

před 7 lety

Šaman
Člen | 2438
+
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.

před 7 lety

redhead
Člen | 1314
+
0
-

Dva.

před 7 lety

Grelek
Člen | 234
+
0
-

Jeden.

před 7 lety

David Matějka
Moderator | 6202
+
0
-

2

před 7 lety

David Grudl
Nette Core | 7148
+
0
-

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

před 7 lety

Majkl578
Moderator | 1376
+
0
-

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

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

před 7 lety

Šaman
Člen | 2438
+
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

před 7 lety

paranoiq
Člen | 389
+
0
-

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

před 7 lety

ic
Člen | 437
+
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

před 7 lety

nAS
Člen | 279
+
0
-

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

před 7 lety

David Grudl
Nette Core | 7148
+
0
-

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

před 7 lety

nanuqcz
Člen | 824
+
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!

před 7 lety

rixi
Člen | 109
+
0
-

3 a nanuqczovi +1 :)

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

před 7 lety

jiri.pudil
Člen | 928
+
0
-

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

před 7 lety

MartinitCZ
Člen | 587
+
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!

před 7 lety

uestla
Backer | 753
+
0
-

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

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