@evolutioncms - страница 2650

Alexander

чета я в кота превращаюсь

Alexey

не, это понятно, ерничить то зачем?) Понимаю что ... надоело, тем не менее)

Alexander

не, это понятно, ерничить то зачем?) Понимаю что ... надоело, тем не менее)

а ты зачем вот так? ты же понимаешь что сначала загрузится сообщение что твоя корзина пустая и в заивисимости о тскорости интеренета клента он может ахуеть) и только потом получить страницу что все хорошо.

Pathologic

ничего себе

Alexey

есть такая штука issue - попробуй)

Alexey

более того, можно отправлять без использования js и там делать редирект

Aliaksandr

Я писал мегауниверсальный класс с интерфейсом на 3 метода: loadData, prepareData и saveData, а остальное херачил по ситуации 😂😂😂

Pathologic

Я писал мегауниверсальный класс с интерфейсом на 3 метода: loadData, prepareData и saveData, а остальное херачил по ситуации 😂😂😂

небось и копипастом еще херачил

Aliaksandr

небось и копипастом еще херачил

Не, там просто фиды в разных форматах и разной структуры были, так что копипаста мало было ))

Alexander

более того, можно отправлять без использования js и там делать редирект

так это твой код, ты зачем то так сделал) вот я смотрю на него и думаю, а почему так?))

Pathologic

Не, там просто фиды в разных форматах и разной структуры были, так что копипаста мало было ))

у меня все эти универсальные классы часто заканчиваются копипастом, потому что через какое-то время лень разбираться, что там как расширять ((:

Pathologic

и импорты почти каждый раз с какими-то приколами

Alexey

так это твой код, ты зачем то так сделал) вот я смотрю на него и думаю, а почему так?))

потому что это мой код и мне показалось так логичным.

Aliaksandr

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

Aliaksandr

а дальше уже как пойдет )))

Pathologic

я поточный парсер использовал

Aliaksandr

ну я имею в виду "универсальность" - есть метод для конкретного фида - его и фигачим, нет - делаем общий )

Aliaksandr

так что как раз общий интерфейс на 3 метода и распадается - загрузил, обработал, сохранил

Aliaksandr

аякс у меня вообще универсальный как вселенная ))

Aliaksandr

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

Pathologic

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

Aliaksandr

наверно потому что я не разработчик на ларавеле ))

Aliaksandr

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

Aliaksandr

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

Aliaksandr

я так в начале походил немного по граблям multiTV и DLMenu, когда все поотключали и сайт попадал, потому что там не прилетало ни $array ни тем более $array[0]

Pathologic

так если все поотключали, то какая разница, падал сайт или нет ((:

Павел

мои глаза

Aliaksandr

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

Aliaksandr

или в меню все пункты дочерние прибили - и DLMenu тоже не прислал никакого нулевого элемента массива )

Alexey

мои глаза

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

Aliaksandr

мои глаза

np++ это святое в кругах EvolutionCMS ))

Aliaksandr

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

оно не сделано в IDE, оно просто открыто с первого попавшегося сайта по фтп в блокноте для примера )

Alexey

оно не сделано в IDE, оно просто открыто с первого попавшегося сайта по фтп в блокноте для примера )

не важно)

Alexey

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

Alexey

Я в разработке использую шторм с темной схемой, разрабатываю только через ООП и т.д. и т.п. Ок. Рабочие решения предоставьте. Клиенты запретили их публиковать.

Павел

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

Согласен, не хватает здравой критики) 1. Лютый перебор по переменным 2. Вам завезли неймспейсы, а до сих пор используется всякие хаки аля переменная контроллер класс? 3. Использование else в тернарных операторах - признак очень плохого тона

Alexey

Согласен, не хватает здравой критики) 1. Лютый перебор по переменным 2. Вам завезли неймспейсы, а до сих пор используется всякие хаки аля переменная контроллер класс? 3. Использование else в тернарных операторах - признак очень плохого тона

Можно долго рассуждать о том как правильно делать, а можно просто делать, потом дорабатывая решения. Именно в этом и был мой посыл

Pathologic

Согласен, не хватает здравой критики) 1. Лютый перебор по переменным 2. Вам завезли неймспейсы, а до сих пор используется всякие хаки аля переменная контроллер класс? 3. Использование else в тернарных операторах - признак очень плохого тона

скорее всего это писалось в дремучие времена и просто тягается туда-сюда ((:

Aliaksandr

1. перебора не вижу

Aliaksandr

2. неймспейс есть

Alexey

В любом случае - у Саши оно сделано и работает

Aliaksandr

3. претензию к if/else не совсем понялЪ)

Павел

скорее всего это писалось в дремучие времена и просто тягается туда-сюда ((:

в этом весь modx особенно рево

Павел

2. неймспейс есть

тогда в переменной контроллер класс нет смысла от слова совсем

Aliaksandr

тогда в переменной контроллер класс нет смысла от слова совсем

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

Павел

3. претензию к if/else не совсем понялЪ)

ты проверяешь на тру, но посмотри насколько все будет проще если ты будешь проверять на false)

Aliaksandr

ты проверяешь на тру, но посмотри насколько все будет проще если ты будешь проверять на false)

давай по-другому - ты возьмешь эту же логику и перепишешь так, как он должен быть на твой взгляд - а потом все вместе посмотрим и обсудим ;-)

Aliaksandr

там же немного всего 30 строк, останется получается строк 15 идеального кода ))

Павел

если до выходных терпит - я в игре)

Pathologic

сложный иф можно разложить на два простых

Aliaksandr

да можно и вообще через тернарники как в перле в одну строку вытянуть - я просто люблю читабельность)

Павел

сложный иф можно разложить на два простых

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

Aliaksandr

и если вдруг у меня возникают сомнения, что может вернуться что-то не то, я лучше еще один дополнительный if всуну, чем пойду по пути который я описал выше "допустим что что этого никогда не случится и вернем сразу $array[0]" ))

EVO bot Лёшка

Тут будет чёрный список плохих клиентов/исполнителей

Павел

/blacklist@EVO_CMS_bot

EVO bot Лёшка

/blacklist@EVO_CMS_bot

Тут будет чёрный список плохих клиентов/исполнителей

Pathologic

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

на самом деле лишь бы понятный

Aliaksandr

если до выходных терпит - я в игре)

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

Павел

и если вдруг у меня возникают сомнения, что может вернуться что-то не то, я лучше еще один дополнительный if всуну, чем пойду по пути который я описал выше "допустим что что этого никогда не случится и вернем сразу $array[0]" ))

Так то да но ты уже обернул код в try catch Ты же можешь просто убрать два этих ифа в принципе)

Павел

try catch стоит использовать тогда, когда кол-во ифов близкое к неадекватным

Aliaksandr

Так то да но ты уже обернул код в try catch Ты же можешь просто убрать два этих ифа в принципе)

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

Aliaksandr

я параноик с точки зрения дополнительных if-ов ))

Павел

А какая версия php?

Павел

выше 7.1?

Aliaksandr

скорее всего это писалось в дремучие времена и просто тягается туда-сюда ((:

нет времени рефакторить, надо ж читать доки ларавеля ))

Aliaksandr

выше 7.1?

мне кажется сейчас ниже 7.3 уже не тянет evo, так что я минимум 7.2 прошу, а лучше сразу 7.4. Но конкретно на этом сайте хз, скорее всего 7.2

Павел

мне кажется сейчас ниже 7.3 уже не тянет evo, так что я минимум 7.2 прошу, а лучше сразу 7.4. Но конкретно на этом сайте хз, скорее всего 7.2

никто ж не мешает делать тогда так

Андрей

/blacklist@EVO_CMS_bot

EVO bot Лёшка

/blacklist@EVO_CMS_bot

Тут будет чёрный список плохих клиентов/исполнителей

Alexey

если до выходных терпит - я в игре)

жду выходных

Aliaksandr

никто ж не мешает делать тогда так

есть большой философский спор, когда надо делать catch, а когда if/else, я честно говоря особо не хочу холиварить на эту тему )

Павел

есть большой философский спор, когда надо делать catch, а когда if/else, я честно говоря особо не хочу холиварить на эту тему )

не не, никаких холиваров) мы же просто общаемся)

Aliaksandr

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

Aliaksandr

тут скорее try-catch выступает как сборщик ошибок, их точно также можно было в массив например собрать

Pathologic

и так дофига кода, чтобы выполнить метод класса (:

Aliaksandr

я же говорю - я ифо-параноик )) что-то не проверю - потом сплю плохо ))

Aliaksandr

можно было в одну строку написать $output = call_user_func( [ $controller, $method ] ); и все ))

Aliaksandr

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