Жуть
а в чем жуть?
Шрифт отвратительный...
Подумаешь, отправка письма😁 Я недавно работал с сайтом, где в init.php был код кастомного загрузчика csv файлов и все обработчики событий. Строк так с тысячу. И всë одной кучей. Вот там реально жуть))
там ошибка. я не заскринил но выше над этим кодом нет обёртки в функцию. этот код на каждом хите получается пытался выполниться.
ну отладкой возможно было сделано и забыто ))
Файл логов тогда проверь) Он скорей всего очень большой и полон "hhhhh"😁
ну так если человеку так удобнее? в чем проблема? по красоте делает каждый как считает нужным
Как минимум проблема в отсутствии разделения ответственности и удобства чтения кода. Всегда можно сказать "а чё такого, мне и так неплохо", но я придерживаюсь мнения, что помойка в init - это вообще не гуд. Да и уверен, что многие со мной согласятся.
Зато хоть в одном месте а не по всему проекту 😅
Ага. Одностраничный сайт)
Второй свежести 🙂
Не. Надо сделать ещё проверку на существование файла. И вызвать метод, который забиндит обработчики событий)
события в модуле
фигня - а модули зачем придумали? p.s увидел чтот композреосвкий автолоад - тогда ок
Если надо отправлять письмо на почту при создании элемента инфоблока, или ловить события загрузки из 1С, то не писать ведь модуль ради этого. Но и в init писать тоже некрасиво.
99 прцоентов проектов таких встречаю где не с 0 приходу)
а почему нет? модуль написать - ну минут 15 с парой событий
На проекте должен быть хотябы один модуль, туда и складывать свой код
Мне просто кажется это излишне. Модуль должен реализовывать какой-то связанный функционал. Можно конечно создать модуль "разное" и напихать туда всего. Но как-то не очень подход))
ну хз, тут в общем тонкая грань
На небольшом проекте - ну, в качестве аргумента только абстрактное "энергию и планету жалко". На тяжелом - рано или поздно возникнет "этот init.php стоит нам денег".
А, в смысле, что он на каждом хите всё подключает? Ну да, не подумал про это...
согласен, но ведь это не стандарт и не критика. ))) тут каждому свое )
Вообще, можно сказать, что стандарт. Хоть и нигде жёстко не заявленный, как в принципе и остальные стандарты программирования. Это просто набор общепринятых правил хорошего кода. Писать так никто не запретит, но и спасибо не скажут) При работе в команде, есть вариант, что коммит с таким кодом не примут и заставят переделывать. При устройстве на работу показывать такой код тоже не стоит. Ну и как выше сказали, помимо красоты, вся эта помойка подключается на каждом хите, поэтому, чем она больше, тем дольше загрузка сайта. В общем, тут нет ни одного аргумента "за", так что не надо так делать😁
папка vendor в корне сайта? а что будет если из браузера попробовать пооткрывать? не случится ли конфуз?
htaccess deny from all
Обсуждают сегодня