Оформление кода → Разделитель вначале

[Banzeg]

В варианте с разделителями вначале верность синтаксиса оценивается беглым взглядом, во отличие от традиционного варианта. Кроме того, редактировать проще [...]

Исключительно субъективно.

[tenshi]

значит ты заметил опечатку?

[Banzeg]

Да, я заметил отсутствие запятой. Только даже если это действительно не намеренная опечатка, это не говорит о том, что какой-то из способов лучше. Просто для вас один способ удобнее другого, потому что вы к нему привыкли — вот и все.

Никогда не понимал, зачем некоторые пытаются навязывать свои привычки другим.

[tenshi]

сколько времени тебе потребовалось, чтобы её найти? заметил бы ты её, если бы я про неё не сказал?

[Banzeg]

К чему этот допрос? :) Само собой, я просматривал код по диагонали, но это не значит, что, будь эта ошибка в варианте «с разделителем вначале», она сразу бы бросилась в глаза.

[tenshi]

разве не бросилась?

[tenshi]

он мне тоже был когда-то непривычен, но ничего, я осознанно поменял привычки, о чём не жалею.

я ничего не навязываю, а просто желаю тебе повысить эффективность своей работы.

[Banzeg]

За заботу твою спасибо говорю тебе :)

[mrBrain]

Та нууу))
Мне все же по душе самый первый вариант.

[tenshi]

интересно, если провести опрос, то сколько выберет вариант «всё в одну строчку и ниибёт»?

[nooze]

Спасибо php, который позволяет ставить запятые даже на последнем элементе (как минимум в массивах)

[Tasman99]

Все известные мне языки (навскидку — c++, c#, js) поддерживают эту возможность

[tenshi]

а при вызове функций?

[xtender]

При вызове функций это не так важно:

• порядок аргументов обычно строг;
• менять порядок аргументов функции, как правило, не надо;
• пропустить запятую в аргументах крайне маловероятно, т.к. их просто обычно мало и они важны

[tenshi]

всё зависит от специфики. например — winapi c десятком параметров

[xtender]

Там порядок аргументов строг и как правило еще разных типов.

[tenshi]

да это не имеет значения. там как и везде есть функции с похожим набором параметров, вызов которых проще скопипастить и чуть подправить, чем писать с нуля.

[homm]

Из известных мне языков так разрешает только питон. В нем даже приветствуется такая запись. Например, последовательность из одного элемента записывается как (item,) и никак иначе. Вот если два и более элементов, последнюю запятую разрешается опускать (item, item).

[tenshi]

ну, это чтобы отличать кортэж от обычных скобок…

[xtender]

Еще perl

[xtender]

Кстати говоря, в Вашем варианте придется еще на каждую строку вбивать лишний табулятор после разделителя, вместо того чтобы пользоваться удобством автоматической разметки ИДЕ.(хотя, конечно, и этот вариант можно добавить в шаблоны)

[tenshi]

авторазметка всё-равно вечно неугадывает что я от неё хочу =_=" так что проще самому ставить табуляциюю, чем подчищать после иде.

[xtender]

При вложенностях в 3-4 таба, вставлять вручную нервирует)

[tenshi]

1. не надо делать глубоких вложенностей — они хуже читаются
2. автотабуляцию никто не отменял (минус 1-2 таба)
3. пара лишних нажатий клавиш — это фигня

[xtender]

Какой-то противоречивый посыл… Во-первых, 3-4 таба — это очень часто встречающаяся вложенность.Во-вторых, про авторазметку ИДЕ я как раз и говорил о лишнем табе после запятой, в ответ на что получил «авторазметка всё-равно вечно неугадывает что я от неё хочу» — собственно неудивительно, если хочется странных и неожиданных вещей. В-третьих, «пара лишних нажатий» на каждую строку это много.

[tenshi]

1. 3-4 — это уже много. 2-3, не более — оптимум
2. она также не угадывает и когда хочу очевидных. ну может мне так с иде повезло — идея.
3. ты на пробелах ещё не стал экономить? х) пара нажатий — это фигня. экономить надо время восприятия, а не время набора.

[xtender]

Ой ли… Как пример

class Blabla{
	public static void WOW(){
		for(int i=0;i<n;i++){
			tratata(hihihi);
			for(int j=0;j<n;j++){
				bugagaga();
			}
		}
	}
}

[tenshi]

разбить метод на два: один для внешнего цикла, другой — для внутреннего

вынести класс в отдельный файл и выкинуть один таб

[xtender]

Ну конеееечно… для каждого вложенного цикла создавать по методу…
Вообще, видимо, мало кода смотрите.

[tenshi]

да и вообще я — хрен с горы. лезу тут в это ваше программирование со своим уставом %-)

[xtender]

Просто для примера: svn.php.net/repository/php/php-src/branches/PHP_5_3/main/main.c

[tenshi]

и что этот говнокод должен мне доказать?

[xtender]

То есть пхпист заявляет, что код пхп — говнокод? :)

[DeusModus]

Всё чаще и чаще звучит эта фраза применительно к чему угодно. Модно что ли стало.

[tenshi]

а скажешь не говнокод?

[xtender]

А что ж на пхп пишешь?

[tenshi]

я уже года 2 на нём не пишу. а что?

[xtender]

Ухъ какое достижение. А учитывая, что пост «PHP → Независимо перегружаемые свойства» от 02.06, то заметно экспоненциальное увеличение ЧСВ не по дням, а по часам… Д'Артаньян, дело-то даже не в том насколько тот код «пыл блох»©, а в том какой Вы молодец! Осудить же так тяжело… Особенно если самим не под силу создать такое-же.

[tenshi]

мисье не знает чем отличается дата создания от даты публикации?

[xtender]

Увы, куда уж мне…

[DeusModus]

Волею судьбы довелось работать с одной системой. Довольно сложной. Поначалу клял разработчиков(полное отсутствие документации, невнятные названия директорий, повторения и прочее).
В итоге я всё это дело продебажил, рассмотрел под микроскопом, сгенерировал хоть какую-то иерархию классов, список методов и сложил в единую картину. Всё оказалось не так уж плохо. Прослеживался шаблон проектирования, имелась своя внутренняя логика.

Я вот к чему- не умею посмотреть на файл в 2000 с лишним строк и сказать что всё что там написано полная лажа. Для анализа кода нужно представлять что, как и почему.

PS: В конце концов рефакторинг существует, tenshi :) И рефакторят не всегда лажу. Люди растут, набираются опыта, голова генерирует новые идеи и старые кажутся не оптимальными.

[tenshi]

а ты посмотри в этот файл. живая иллюстрация как делать не надо.

а то, что ты вник в проект — честь тебе и хвала, только вот в хорошем коде тебе не пришлось бы столько ковыряться микроскопом, чтобы увидеть «паттерны».

[DeusModus]

Можешь показать хороший с твоей точки зрения код, применяющейся в живом, функциональном, развивающемся проекте, который разрабатывали разные люди в разные периоды жизни, росшие в профессиональном плане пока писали его?
Возможно, тебе просто везет с классными проектами. Тогда завидую белой завистью.

[tenshi]

разумеется нет. и что ты доказал? что говнокод — это хорошо и рефакторить его не надо?

[DeusModus]

Трудно общаться с человеком, который не читает что ты ему пишешь(первое предложение).
Мы только что поняли, что ты не можешь обьяснить и показать что для тебя стандарт нормального кода, но по каким-то фэншуйным параметрам оцениваешь чужой труд. Флаг в руки.

[tenshi]

могу, но мне лень. если тебе тот код кажется нормальным, то мне тебя жаль и метать бисер в пустоту я не собираюсь.

[tenshi]

я не пхпист, да и там всё на сях.

[xtender]

Ну да-ну да… Посты в профиле из блога PHP видимо о brainfuck'e…

[tenshi]

да ты прям детектив — уличил меня в использовании пхп 0_0 блин, надо срочно менять имя…

[tzlom]

JS в реализации ослиной сики IE не поддерживает ( как обычно не такой как все )

[tenshi]

поддерживает, но не правильно %-)

[Volshebnyi]

Разработчики IE негодуют! =) Они считают, что после последнего элемента массива в JS запятую ставить нельзя ни в коем случае.

[homm]

Ставьте запятую во всех элементах, включая последний и не надо е**ть мозг.

[tenshi]

всегда пишите без ошибок, иначе кол вам в задницу! х)

[ozon]

в JS в IE всех версий будет вываливаться ошибка

[ozware]

+ ко всему вышесказанному — большинство IDE сразу подчеркивают ошибку, если не так записать массив (я про php)

[tenshi]

у тебя иде подчёркивает лишнюю запятую в конце массива в пхп? во тупая х)

[ozware]

наоборот, если массив записан неправильно, то подчеркивает

[tenshi]

у меня она подчёркивает через несколько секунд после того как я закончу редактировать, что сильно раздражает…

[Krypt]

<imho>встертил бы в коде — сделал бы с автором нехорошую вещь</imho>
Разрушает ощущение зависимости строк. Нет отступа на следующей строке — оператор завершён и дальше не его.

Если не нравится отсутствие символа в конце — сделайте так:

var userList= [
    { name: 'Tom', Age: 5, race: 'cat' },
    { name: 'Jerry', Age: 3, race: 'mouse' },
    { name: 'Spike', Age: 11, race: 'dog' }]

[tenshi]

ничего оно не разрешает. знаки препинания с последующим табом имеют незначительный визуальный вес.

а твой способ провоцирует ошибки вида «не закрытый массив»

[Krypt]

При быстром просмотре кода взгляд упирается в крайний левый символ, проблема в этом.
Вообще, эта ошибка времени компиляции (по крайней мере в тех языках, что я знаю), и бороться с ней таким образом, это бороться с ветряными мельницами.

P.S.: У objective-C программистов второй пример способен вызвать нервный тик ;)

[tenshi]

афайк, глаз быстро переучивает не упираться в регулярные структуры

угу, только пока там оно ещё докомпилируется до этого места… в общем случае — чем раньше обнаруживается ошибка — тем лучше

разумеется это применимо не везде

[Krypt]

Если компилятор поддерживает прекомпилированные заголовки, то они сократят время повторной компиляци.
Для человека, не привыкшего к такой манере записи он может стать проблемой (а ещё если он включил C'шный код в проект obj-C… :) )

Вообще, как мне кажется, стоит использовать тот стиль, что навязывают IDE и среда разработки, если в ней есть авторасстановка. Если это «коммерческий» код, то вероятность того, что с ним будет работать другой программист стремится к ста процентам. Да и в случае тулзы, написанной для себя, вероятность не нулевая.

[tenshi]

иде должна помогать, а не диктовать. и уж точно код не должен писаться в рассчёте на то, что с ним будут работать только в иде или даже в конкретной иде =_="

[Krypt]

> иде должна помогать, а не диктовать.
К сожалению, реальность такова. А может и к счастью.
Visual Studio не даст вам использовать «Египетские» скобки, Xcode (среда разработки под Mac OS/iPhone os) ровняет перенесённые строки по первому двоеточию.
Для некоторых языков наиболее распространённая IDE одна, либо вообще единственная, так что для них такой расчёт смысл имеет.

[tenshi]

гомэн, случайно на минус попал — иконки слишком мелкие =_=

[iShaman]

Наркоманский метод

[tenshi]

а ты ширнись хоть раз — потом за уши не оттянешь ;-)

[DeusModus]

Лично для меня написание кода программы похоже на изложение мыслей. Замкнутое на количестве и контексте слов. И как-то сложилось, что при изложении мыслей на бумаге или при мечати запятую привычно ставить после слова.

[tenshi]

а ещё привычно ставить в конце точку, а не симиколон или фигурную скобку?

[DeusModus]

Если так далеко заходить, то мы придем к языку 1С бухгалтерии. Нет, пожалуй, никогда кроме pascal в университете.

[tenshi]

в таком случае аппеляция к естественным языкам тут не к месту

[DeusModus]

Мне судить что к месту у меня в голове.

[tenshi]

ты заблуждаешься

[DeusModus]

Никогда не любил секты именно за подобные высказывания :)

[tenshi]

переходи на сторону зла — у нас есть печеньки ;-)

[GooRoo]

Я использую подобное форматирование только когда пишу на Haskell… В основном чисто для того, чтобы сделать код ещё более «шокирующим» для неокрепших умов :)

[tenshi]

можешь пример показать?

[GooRoo]

Ну, не знаю даж. Редко на Haskell пишу. Например вот:

data Book = Book {
      id      ::  BookID
    , title   ::  String
    , authors :: [Author]
    } deriving (Show)

[tenshi]

фигурная скобка не там %-)

[GooRoo]

Это мой код, значит фигурная скобка там, где ей полагается быть :P
Я имел в виду запятые.

[tenshi]

как-то ты не последовательно…

[GooRoo]

Окей, так лучше?

data Book = Book { id      ::  BookID
                 , title   ::  String
                 , authors :: [Author]
                 } deriving (Show)

[tenshi]

да, гораздо %-)

[JayDi]

Вы извращенец.

[telegamochka]

Я много лет пользуюсь именно таким способом написания. Удобно комментировать строки, да и наглядно.

[tenshi]

удобно комментировать? почему? о0

[telegamochka]

char[] tokens = {
    "START"
  , "LOAD_LIST"
  , "CLEAR_LIST"
  , "LOAD_FILE"
  , "LOAD_LIST"
  , "NEXT_TRACK"
  , "PREV_TRACK"
//  , "RECORD"
};

[telegamochka]

char[] tokens = {
  "START",
  "LOAD_LIST",
  "CLEAR_LIST",
  "LOAD_FILE",
  "LOAD_LIST",
  "NEXT_TRACK",
  "PREV_TRACK",
//  "RECORD"
};

В С выдаст ошибку из-за запятой в конце списка.

[tenshi]

ну, а в варианте с разделителем вначале то же самое будет, если закомментить строчку в начале…

другое дело, что иногда можно делать так:

var x=
( false
|| blablabla
|| lbalbalba
)

то есть первую строчку комментить никому в голову не придёт