Код должен быть: читабельным, понимабельным, тестируемым, отлаживаемым и модфицируемым. Экономия строк нарушает все эти параметры

Но опять же не надо плодить лишнего кода, если он не добавляет читабельности там итп )

да но понять 1 иф в 10 строк проще, чем размазанное по 10 функциям логика с кучей варов

Не надо

так размазанное никогда не кратное

оно всегда с коэфициентом

10 строк разворачиваются в 5 экранов

иначе это вообще не повод для дискуссии

"размазанный" код лучше по всем параметрам

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

Все попилено на простую логику, всё покрыто тестами, поломки сразу очевидны

Хреновый код трудно тестами покрывать. "один иф" из примера выше сделает только боль, страдание и желание сменить персонал ?

так нет смысла мазать 10 условий в ифе в 3 метода покрытыми тестами

Ребята, срочный вопрос. Покупаю макбук. Есть два варианта: Прошка 12 года двухъядерный i7 2.9 16гб оперативка 750гб + 250 SSD Про 13 с ретина Двухъядерный i5 2.4 4 гб оперативка 128 SSD Что лучше взять? Разработка под PHP Играет ли

Я супер вовремя, но и то могу сказать, что 15 прошка середины 15 года очень крута :)

хотя 10 условий в реальном мире конечно не бывваает

обычно 2-3 и они точно так же мажутся в 10 раз

хотя 10 условий в реальном мире конечно не бывваает

Ха ха

Всплакнул :(

return ( [ 'result' => false ] + ( !env('APP_DEBUG') ? [] : [ 'error' => [ 'class' => get_class($e), 'message' => $e->getMessage(), 'trace' => array_merge( [$e->getFile() . ':' . $e->getLine()], array_filter(explode('#', $e->getTraceAsString())) ), ] ]) + (array) $this->response_add );

вот взял первопопаввшееся из своего проекта

это не совсем то, что я имею ввиду

Ну вот это вот нечитабельный пиздец как по мне оО

Ну вот это вот нечитабельный пиздец как по мне оО

Почему? Нормально вроде

Почему? Нормально вроде

это на грани, имхо

просто рендко попадаются действительно спорные моменты

Тернарки и прочее уместно в реткрне только в очень простых ситуациях

надо искать

Тернарки и прочее уместно в реткрне только в очень простых ситуациях

нет

ты просто не умеешь

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

Писать простые конструкции, даже если это увеличивает портянку - круто

А еще могли отладить и расширить

И тд

чёт ни 1 примера в проекте не вижу

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

это пример кода который я пишу в одного

плохого кода, на грани

Ну в одного это другой вопрос)

Ну вот это вот нечитабельный пиздец как по мне оО

А можешь рефакторинг сделать? Просто интересно

А еще могли отладить и расширить

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

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

собираешь в кучу

тратишь время

вырезхать из ифа и размазатькуда быстрее

жто моя практика

А можешь рефакторинг сделать? Просто интересно

Конечно можно, только не круто делать рефакторинг куска вырванного из контекста. Уверен там стоит переписать пол функции/метода/класса

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

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

у меня некоторого рода фиксация на оптимизацию

Конечно можно, только не круто делать рефакторинг куска вырванного из контекста. Уверен там стоит переписать пол функции/метода/класса

ровно наоборот

рефакторинг долже работать БЕЗ контекста

Я вот ща рефакторю у себя либу, кода в 2 раза больше стало, зато чрезе 2 месяца разработки будет в 3-4 раза меньше чем могло бы и я не охренею это читать )

иначе это называется реинженеринг

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

кек но нет

Я вот ща рефакторю у себя либу, кода в 2 раза больше стало, зато чрезе 2 месяца разработки будет в 3-4 раза меньше чем могло бы и я не охренею это читать )

размазать такую функцию по 3 это не оптимизация читабельности

кек но нет

Ну всё впереди ещё, значит ?

Ну всё впереди ещё, значит ?

дододо

Ну всё впереди ещё, значит ?

попробуй оптимизируй

Что именно?

вынести условие APP_DEBUG ниразу не оптимизация читабельности

но ты попробуй

Что именно?

то же самое

return ( [ 'result' => false ] + ( !env('APP_DEBUG') ? [] : [ 'error' => [ 'class' => get_class($e), 'message' => $e->getMessage(), 'trace' => array_merge( [$e->getFile() . ':' . $e->getLine()], array_filter(explode('#', $e->getTraceAsString())) ), ] ]) + (array) $this->response_add );

^

рефакторинг долже работать БЕЗ контекста

ну такое... Просто ну как по мне это пиздец полувселенского масштаба... Я не могу повериь что нужно именно в таком виде отправлять пакет и обязательно его собирать в последний момент, а не подготовив по пути к финишу...

простейший тупейший пример с тупо-отладкой аякса

Не не, я пас. Не хочу лезть в это

то чувство когда я бы размазал это за 10 сек

но это не конец карьеры как в медицине когда убил кого-то

это не конец карьеры, это ачивка

ERROR: S client not available

ну такое... Просто ну как по мне это пиздец полувселенского масштаба... Я не могу повериь что нужно именно в таком виде отправлять пакет и обязательно его собирать в последний момент, а не подготовив по пути к финишу...

ничё не понял. причём не понял контекст

это не конец карьеры, это ачивка

ёкогда у тебя 1 жертва на десяток может не уволят

когда 1 из 1- запросто

у нас любят устраивать охоту на стрелочников

Не не, я пас. Не хочу лезть в это

вообще пугает такой подход

Конечно зависит от проекта. Если ты его пишешь, чтобы разово задачу выполнить, то всем пофиг и не важно что и как, но я бы не хотел с таким кодом работать

там даже сложностей нет

Конечно зависит от проекта. Если ты его пишешь, чтобы разово задачу выполнить, то всем пофиг и не важно что и как, но я бы не хотел с таким кодом работать

это класс который готовит ответ на отдачу по аякс-запросу

это читается в функции

размазать па пару-тройку действий ваобще не проблема

равно как и прочитать беглым взглядом

слишком просто чтобы являтся камнем преткновения

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

Просто покрой это тестами и сам увидишь в чем суть атомарного кода

Просто покрой это тестами и сам увидишь в чем суть атомарного кода

так я покрываю. но только то что нужно

функцию в целом а не иф

Кстати, глянь whoops

надо каждую переменную, каждую итерацию отслеживать

каждую единичку и нолик

Прикольная штука для вывода ошибок

https://github.com/filp/whoops

Ларавельщики использовали это, ещё до того как это стало мэйнстримом

так что не удивил

И не пытался. В коде выше костыли по выводу ошибок были

Костыли наше всё.

Так пусть хотя бы одинаковыми будут ?

https://github.com/filp/whoops

злая штука

я пробовал

Ох, раз такой холивар про экономию, то мне интересно отношение к приватному конструктору, фабричному методу без аргументов и куче сеттеров :)

сеттеры збс

Я для себя недавно понял, что это оверхед на начальном этапе большой, но очень удобно

а фабрика "внешняя" спорная штука