Программирование, т. 7

 

Кстати, неплохая портянка https://github.com/ryanmcdermott/clean-code-javascript

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

чо?

 

Инфо, Дата и Рекорд не говорят ни о чем. их не стоит использовать для описания класса или в названии метода

блядь там дальше стаёт ещё хуже

 

> Encapsulate conditionals

> Avoid negative conditionals

 

вот это вообще пиздец

 

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

да мало того, ещё и убеждатся что их всегда надо будет без ! в иф-ах юзать

чето так ещё раз подумал и аж пригорел

 

 

вот нахуя такое вообще надо кроме как если оно дохуя раз реюзается?

НАХУЯ

 

по названию функции не особо похоже чтоб это было реюзаемым

по итогу ты делаешь хуйпоймичто чёрную коробку которая решает показывать спиннер или нет, но при этом читая код ты ж никогда не поймёшь почему

и по аргментам что ты туда передаёшь тоже нихуя не понятно

 

это норма в джаваскрипте так писать или что?

читаемость повышается.

понятно что это за проверка

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

Щас в какой-то статье про либу на swift наткнулся на вот такую ахуенность

SPRequestPermissionAssistant.modules.dialog.interactive.create(with: [.Camera, .PhotoLibrary, .Notification])

Пиздец, там чо не изобрели импорт конкретных модулей/методов из либ? Я ржу с этой хуйни уже минут 5, как вообще люди могут такой код писать повседневно и не хотеть убить себя?

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

зачем оставлять комменты? по внутренностям не всегда понятно

 

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

зачем оставлять комменты? по внутренностям не всегда понятно

 

Не понятно нахуй ты эту хуйню несешь?

И кто вместо однострочного коммента создает юзелесную функцию?

С суеза в голос тоже.

Обычная хуйня уже для него - вбросить какую-то залупу, которую он пробежался глазами с фразами "Ахуенчик, всем советую", а потом "Бля внимательно прочитал - чет хуйня" или "Ну там надо выборочно инфу впитывать". Второй писхевер вырастает.

 

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

зачем оставлять комменты? по внутренностям не всегда понятно

 

если поятно то нечего там трогать

если не понятно то // show spinner

нахуя делать функцию-однострочник на 1-2 раза?

Я вот там почитал, много раз увидел "...just do one thing"

То есть если мне надо в функции кроме чего либо вызвать другую функцию то я зависну в бесконечном цикле?

бтв отличные советы, больше кода - больше денег

 

 

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

зачем оставлять комменты? по внутренностям не всегда понятно

 

Не понятно нахуй ты эту хуйню несешь?

И кто вместо однострочного коммента создает юзелесную функцию?

 

С суеза в голос тоже.

Обычная хуйня уже для него - вбросить какую-то залупу, которую он пробежался глазами с фразами "Ахуенчик, всем советую", а потом "Бля внимательно прочитал - чет хуйня" или "Ну там надо выборочно инфу впитывать". Второй писхевер вырастает.

 

комменты - это юзлесная хуйня. особенно в таком случае.

в идеале их не должно быть вообще

ещё один

а функции в одну строку с двумя булевыми через && должны?

 

 

а по внутренностям if блока это не понятно? или камент оставить руки отсохнут?

зачем оставлять комменты? по внутренностям не всегда понятно

 

если поятно то нечего там трогать

если не понятно то // show spinner

нахуя делать функцию-однострочник на 1-2 раза?

 

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

комменты, которые объясняют что делает твой код - плохой тон. 

а функции которые создают месиво из юзлесс говна на каждый иф с 2+ условиями это хороший тон?

а функции которые создают месиво из юзлесс говна на каждый иф с 2+ условиями это хороший тон?

если ты будешь нормально проектировать классы - у тебя не будет кучи таких функций.

хотя конкретно тот пример хуевый. т.к. скорее всего в том случае будет вызов showSpinner. и кондишн с названием shouldShowSpinner - совсем не описывает условие

ясно

ты как обычно нихуя не прочитал и влез в средину обсуждения

ясно

 

ты как обычно нихуя не прочитал и влез в средину обсуждения

ну сами советы то правильные.

если у вас так херово названы переменные, по которым вы делаете условие, что это не очевидно с первого прочтения, то объявите себе переменную

var shouldShowSpinner = тот непонятное условие

и дальше уже if(shouldShowSpinner) {}

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

а типа 

boolean hui = fsm.state === fetching;

boolean kek = isEmpty(listNode);

if (hui && kek) {}

не комильфо?

а типа 

boolean hui = fsm.state === fetching;

boolean kek = isEmpty(listNode);

if (hui && kek) {}

не комильфо?

а зачем оно как переменные в функции внутри?

а если не переменные, то cache invalidation как?

а типа 

boolean hui = fsm.state === fetching;

boolean kek = isEmpty(listNode);

if (hui && kek) {}

не комильфо?

Нет.

Комильфо

вар статея = фсм.стате;

строка фетчинг = "fetching";.,

буль-буль рубиш = статея ====== фетчинг;;;

постоянная листКоторыйНамНадоИспользоватьВЭтойОперацииЧтобыПолучитьКанта = листНоде;

буль-буль кант = алиПустой(листКоторыйНамНадоИспользоватьВЭтойОперацииЧтобыПолучитьКанта)

буль-буль надоЛиПоказатьВерчение = рубиш &^^%$$^^$ кант;;;;

если (надоЛиПоказатьВерчение)))))))) {

  иди нахуй

}

 

а типа 

boolean hui = fsm.state === fetching;

boolean kek = isEmpty(listNode);

if (hui && kek) {}

не комильфо?

Нет.

Комильфо

 

вар статея = фсм.стате;

строка фетчинг = "fetching";.,

буль-буль рубиш = статея ====== фетчинг;;;

постоянная листКоторыйНамНадоИспользоватьВЭтойОперацииЧтобыПолучитьКанта = листНоде;

буль-буль кант = алиПустой(листКоторыйНамНадоИспользоватьВЭтойОперацииЧтобыПолучитьКанта)

буль-буль надоЛиПоказатьВерчение = рубиш &^^%$$^^$ кант;;;;

если (надоЛиПоказатьВерчение)))))))) {

  иди нахуй

}

 

годно 

 

а типа 

boolean hui = fsm.state === fetching;

boolean kek = isEmpty(listNode);

if (hui && kek) {}

не комильфо?

а зачем оно как переменные в функции внутри?

а если не переменные, то cache invalidation как?

 

да я чот проебал нить спора

не нравится что в функции идет if на два условия или чо блять

рубишу и писателю той статьи нравится однострочные одноразовые ифы в функции засовывать "чтоб понятней было"

дедскин говорит, что это нахер не надо, если всё очень плохо, напиши коммент

рубиш говорит, что комменты для пидаров (тут я согласен)

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

ну а что тут вы нагородили я хз