Jump to content
    

Как писать неподдерживаемый код?

пролистовал учебник... или "учебник" по жаваскрипт.... наткнулся на такую статью

 

Предлогается писать подобный индусский код

// код из jQuery
i = i ? i < 0 ? Math.max(0, len + i) : i : 0;

 

Это шутка какая-то, или реально сейчас учат так писать???

 

 

Share this post


Link to post
Share on other sites

классический vendor lock-in.

на что только не пойдёт программист, лишь бы его Центропупие не отпихнули от кормушки.

Share this post


Link to post
Share on other sites

Профи пишут ещё понятнее

 

i=i?i<0?Math.max(0,len+i):i:0;j=j?j<0?Math.max(0,len+j):j:0;...

 

и весь код в одну строку

 

Share this post


Link to post
Share on other sites

Это шутка какая-то, или реально сейчас учат так писать???

Это называется "сарказм". Статья пишет, что надо писать так, но подразумевает, что так писать не надо.

 

Кстати, в jQuery код очень аккуратный и понятный, вот, посмотрите.

Просто его упаковывают, чтобы уменьшить объём.

Share this post


Link to post
Share on other sites

Это шутка какая-то, или реально сейчас учат так писать???

 

А название статьи не смутило? Содержание строго соответствует названию, на учебник уважаемого издательства не особо похоже.

 

PS. Кто вручную разбирал знаменитый однострочник на перле, такие фокусы читает без запинки ;)

 

perl -e '$??s:;s:s;;$?::s;;=]=>%-{<-|}<&|`{;;y; -/:-@[-`{-};`-{/" -;;s;;$_;see'

Share this post


Link to post
Share on other sites

Это же "Вредные советы" для программиста. Заодно проверка чувства юмора.

Share this post


Link to post
Share on other sites

Для С++ есть Clang Format. http://format.krzaq.cc/ тут можно поиграть онлайн.

Но если переменные называть i, j, k, l, m - тут ничего не поможет. В смысле, все нормально, рабочее место за вами.

Edited by Slash

Share this post


Link to post
Share on other sites

Но если переменные называть i, j, k, l, m - тут ничего не поможет. В смысле, все нормально, рабочее место за вами.

Если переменные называть i, j, k, l, m и/или следовать советам статьи, то рано или позно сам себе ногу отстрелишь.... с последующим увольнением.

Share this post


Link to post
Share on other sites

Если переменные называть i, j, k, l, m и/или следовать советам статьи, то рано или позно сам себе ногу отстрелишь.... с последующим увольнением.

И что? Я все локальные переменные внутри функций так называю. И когда просматриваю функцию - сразу понимаю, что это локальная переменная не заглядывая в заголовок.

За >15лет работы ногу не отстрелил.

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

Share this post


Link to post
Share on other sites

И что? Я все локальные переменные внутри функций так называю. И когда просматриваю функцию - сразу понимаю, что это локальная переменная не заглядывая в заголовок.

За >15лет работы ногу не отстрелил.

Про однобуквенные переменные было сказано в контексте статьи. в статье предлагается использовать однобуквенные переменные не для циклов, и не внутри коротких функций. Назовите переменные например

class ComPort

{

int d,i,e;

std::strung j,q;

};

 

А ещё лучше этот же компорт

 

class Uu

{

int d,i,e;

std::strung j,q;

};

 

И когда просматриваю функцию - сразу понимаю, что это локальная переменная не заглядывая в заголовок.
1) Ну так-то глобальные переменные - зло. внутри функции все переменные должны быть локальными. Конечно есть люди, кто всё пишет с глобальными переменными и код годный.... но это не есть тру.

2)А как вы потом понимаете, что это за переменная? есть устоявшиеся правила что i, j, k, n - счетчики в циклах, это понятно. Но вот вам надо переменные локальные сделать, кол-во красных машин, кол-во белых машин, а также кол-во красных и белых фонарей. Вы назовёте их q,w,e,r. Да ещё и в аргументах функции они будут

bool funk(int q, int w, int e, int r); ))))) как в статье говориться - даже простейшее изменение займет годы оплачиваемого труда

Конечно найдется человек, который скажет: "Я с таким кодом за 15 лет ногу не отстрелил". Найдутся люди, которые в кобуре проносили заряженный пистолет 15 лет и ногу не отстрелили.... но это не значит что другие не отстрелят.

 

Share this post


Link to post
Share on other sites

например

class ComPort

...

Это конечно не комильфо.

 

2)А как вы потом понимаете, что это за переменная? есть устоявшиеся правила что i, j, k, n - счетчики в циклах, это понятно. Но вот вам надо переменные локальные сделать, кол-во красных машин, кол-во белых машин, а также кол-во красных и белых фонарей. Вы назовёте их q,w,e,r. Да ещё и в аргументах функции они будут

bool funk(int q, int w, int e, int r); ))))) как в статье говориться - даже простейшее изменение займет годы оплачиваемого труда

Нет, конечно так (однобуквенно или буква цифра i0, i1,...) я называю только временные переменные внутри функций, всякие счётчики циклов как вы указали и т.п. Есть у меня устоявшиеся правила: i<цифра> или j<цифра> или с<цифра> - int или unsigned, q<цифра> - 64-битное значение, a<цифра>, x<цифра> - float или double, n<цифра> - какое-нить количество, p<цифра> - указатель (общий случай), s<цифра> - указатель на char и т.п.

И если надо внутри функции чтобы были например переменные типа int и типа unsigned int, то это будет:

int i, i0, i1, ...;

uint j, j0, j1, ...;

чтобы дальше по коду сразу было ясно - где знаковая, а где беззнаковая операция.

Аргументы функций все многобуквенные.

И это не жёсткое правило, если нужно N красных машин и N синих машин, может быть будет nr и nb, а может nRed и nBlue - по обстоятельствам.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...