Creed 4 Опубликовано: 25 февраля 2017 Рассказать Опубликовано: 25 февраля 2017 Здравствуйте Возникло несколько вопросов по самому коду DLE (версия 11.2), багами их назвать, наверное, нельзя, интересна причина именно такой реализации. Итак, короткий список: Зачем в движке включено отображение ошибок (display_errors = true), если даже оф. документация PHP советует это делать лишь на dev-серверах? Зачем в том же блоке с включением вывода ошибок и другими параметрами ini_set установлены т.н. "собаки", это же, вроде как, считается дурным тоном в программировании? Зачем в том же блоке дублируется error_reporting и error_reporting (через ini_set), если это одно и то же в данном контексте? Почему бы не сделать просто удобную функцию debug режима, вкл./выкл. из админки, как это сделано во многих CMS, а по умолчанию ошибки будут скрыты и сайт будет работать так, как настроен изначально php.ini? И ещё пара моментов: в файле engine/classes/min/config.php осталась конструкция проверки наличия ошибок типа E_DEPRECATED в очень многих файлах переносы строк Linux варианте т.е. LF, в большинстве же в Windows варианте CRLF (пришлось переводить вручную в единый вариант для работы с git) П.С. если промахнулся с разделом, перенесите, пожалуйста, куда следует. 4 Цитата Ссылка на сообщение Поделиться на других сайтах
alex32 934 Опубликовано: 25 февраля 2017 Рассказать Опубликовано: 25 февраля 2017 (изменено) о блин, еще один суперпрограммист нарисовался. Вам то какое дело, зачем и почему? Напишите свою цмс, раскрутите ее также как дле, и там на своем форуме выё"""""тесь. А зачем и почему это сделанно в ДЛЕ это не ваша пичалька. Изменено 25 февраля 2017 пользователем alex32 Цитата Ссылка на сообщение Поделиться на других сайтах
SKYNET74 315 Опубликовано: 25 февраля 2017 Рассказать Опубликовано: 25 февраля 2017 2 часа назад, alex32 сказал: о блин, еще один суперпрограммист нарисовался. Вам то какое дело, зачем и почему? Напишите свою цмс, раскрутите ее также как дле, и там на своем форуме выё"""""тесь. А зачем и почему это сделанно в ДЛЕ это не ваша пичалька. Ну по переносам строк верно всё говорит, где то табы, где то пробелы, общий вид кода как после атомной бомбёжки, даёшь читабельный код? 0_о 4 часа назад, Creed сказал: Зачем в движке включено отображение ошибок (display_errors = true), если даже оф. документация PHP советует это делать лишь на dev-серверах? Зачем в том же блоке с включением вывода ошибок и другими параметрами ini_set установлены т.н. "собаки", это же, вроде как, считается дурным тоном в программировании? Зачем в том же блоке дублируется error_reporting и error_reporting (через ini_set), если это одно и то же в данном контексте? Почему бы не сделать просто удобную функцию debug режима, вкл./выкл. из админки, как это сделано во многих CMS, а по умолчанию ошибки будут скрыты и сайт будет работать так, как настроен изначально php.ini? 1. Это что бы было проще "нубам" помочь с их поделками, а то ~90% пользователей не смогут их включить, или перед этим доведут до самоубийства саппорт. 2. Это что бы работало везде и всюду, даже на хосте за пол бакса который настраивал чупакабра левой пяткой средней ноги. 3. Смотрим пункт 2. 4. Это из разряда "А почему в DLE такая проверка условий скудная?", там очень нужные вещи ущемляют, а дебаг который ~90% пользователям не нужен попадает под ту же фичирезалку. 3 Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 986 Опубликовано: 26 февраля 2017 Рассказать Опубликовано: 26 февраля 2017 21 час назад, Creed сказал: Зачем в движке включено отображение ошибок (display_errors = true), если даже оф. документация PHP советует это делать лишь на dev-серверах? Чтобы критические ошибки было видно, именно критические. Не критические DLE уберет и отключит сам. 21 час назад, Creed сказал: Зачем в том же блоке с включением вывода ошибок и другими параметрами ini_set установлены т.н. "собаки", это же, вроде как, считается дурным тоном в программировании? Нас не интересует тон, нас интересует работоспособность, на максимальном количество хостингов. Чтобы не вывались ошибки из за настроек хостинга. Смотрите выше то что я написал. Критическое выводим, не критическое нет. 21 час назад, Creed сказал: Зачем в том же блоке дублируется error_reporting и error_reporting (через ini_set), если это одно и то же в данном контексте? Ответ выше, для совместимости с хостингами и их настройками. 21 час назад, Creed сказал: Почему бы не сделать просто удобную функцию debug режима, вкл./выкл. из админки, как это сделано во многих CMS, а по умолчанию ошибки будут скрыты и сайт будет работать так, как настроен изначально php.ini? Какими debug режимами вы собираетесь увидеть например нехватку или утечку оперативной памяти сервера. Данная ошибка например вызывает принудительную остановку системы без вызова каких то debug функций и перехватчиков ошибок. Есть критические ошибки сервера, которые невозможно отследить функциями. Для этого данный тип ошибок нужно увидеть, чтобы знать какие меры принимать. Цитата Ссылка на сообщение Поделиться на других сайтах
SKYNET74 315 Опубликовано: 26 февраля 2017 Рассказать Опубликовано: 26 февраля 2017 3 часа назад, celsoft сказал: увидеть например нехватку или утечку оперативной памяти сервера Для этого нужно правильными священными методами вовремя мочить детей, что бы они не разрослись в армию спиногрызов и не сожрали всю память... Но это уже совсем другая история... Логировать нужно хотя бы косяки DLE, их у него тоже не мало, о текучести памяти, о 5xx, и краше и прочем уже не забота DLE думать... 1 Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 986 Опубликовано: 26 февраля 2017 Рассказать Опубликовано: 26 февраля 2017 4 часа назад, SKYNET74 сказал: Для этого нужно правильными священными методами вовремя мочить детей, что бы они не разрослись в армию спиногрызов и не сожрали всю память... Но это уже совсем другая история... и краше и прочем уже не забота DLE думать... Всегда поражался людям которые даже близко не имеют отношения к тех. поддержке DLE рассуждают о том что ей делать, и как вести логи. Да будет вам известно что на DLE работает несколько сот тысяч сайтов !!!. И в поддержу чуть ли не через день приходят вопросы о том что за ошибка, указав при этом ошибку о нехватке памяти. Несмотря на явное написание ошибки, все равно спрашивают что она значит. И для того чтобы нам ответить, достаточно ее увидеть в письме. А если бы еще и скрыли эту ошибку то нужно было бы заходить на сайт и пытаться ее воспроизвести, т.к. такие ошибки не возможно перехватить. Так что давайте вы не будете рассуждать о том как работать по вопросам поддержки. 4 часа назад, SKYNET74 сказал: Логировать нужно хотя бы косяки DLE, их у него тоже не мало, о текучести памяти, о 5xx, и краше и прочем уже не забота DLE думать... Какие конкретно "косяки" логировать? Наша забота думать о клиентах в любом случае, не зависимо от того где возникла проблема. И подобное сообщение, либо в связи с отсутствием у вас каких либо клиентов, либо ваш универсальный ответ для клиентов: "это не наша проблема, и не наша забота думать об этом". Цитата Ссылка на сообщение Поделиться на других сайтах
SKYNET74 315 Опубликовано: 26 февраля 2017 Рассказать Опубликовано: 26 февраля 2017 27 минут назад, celsoft сказал: Всегда поражался людям которые даже близко не имеют отношения к тех. поддержке DLE рассуждают о том что ей делать, и как вести логи. Да будет вам известно что на DLE работает несколько сот тысяч сайтов !!!. И в поддержу чуть ли не через день приходят вопросы о том что за ошибка, указав при этом ошибку о нехватке памяти. Несмотря на явное написание ошибки, все равно спрашивают что она значит. И для того чтобы нам ответить, достаточно ее увидеть в письме. А если бы еще и скрыли эту ошибку то нужно было бы заходить на сайт и пытаться ее воспроизвести, т.к. такие ошибки не возможно перехватить. Так что давайте вы не будете рассуждать о том как работать по вопросам поддержки. Я в своё время работал в поддержке, правда не DLE "слава те господи", но знаком с кухней... И я говорил про отключение вывода ошибок? Я говорил лишь про логирование, тогда можно видеть какие ошибки происходят на сайте, где и с какой переодичностью, и другую информацию в удобном виде. И там же задавать уровень ошибок например. 29 минут назад, celsoft сказал: Какие конкретно "косяки" логировать? Наша забота думать о клиентах в любом случае, не зависимо от того где возникла проблема. И подобное сообщение, либо в связи с отсутствием у вас каких либо клиентов, либо ваш универсальный ответ для клиентов: "это не наша проблема, и не наша забота думать об этом". Именно по этому DLE и остаётся по виду почти двумя ногами из начала 2000-ых, т.к. вместо разработки, вы занимаетесь разгребанием вопросов абсолютно не компетентных "клиентов", которые в студию не пошли, а решили сами забубенить себе "супер***носайт", но при этом в голове ноль знаний и не желающие их получать, ну и соответственно нанимать кого то, они же и так выложили кучу денег за DLE, т.к. с опенсорсом не смогли или не захотели подружится. Нужно давать удочку, а не рыбу. 2 Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 986 Опубликовано: 26 февраля 2017 Рассказать Опубликовано: 26 февраля 2017 19 минут назад, SKYNET74 сказал: Я в своё время работал в поддержке, правда не DLE "слава те господи", но знаком с кухней... И я говорил про отключение вывода ошибок? Я говорил лишь про логирование, тогда можно видеть какие ошибки происходят на сайте, где и с какой переодичностью, и другую информацию в удобном виде. Если вы работали в поддержке, то должны знать все ошибки не только выводятся и но записываются в логи сервера, где их можно всегда увидеть, в очень простом и удобном виде. Когда они происходили и с какой периодичностью. Делать двойную работу попросту бессмысленно. 19 минут назад, SKYNET74 сказал: Именно по этому DLE и остаётся по виду почти двумя ногами из начала 2000-ых, т.к. вместо разработки, вы занимаетесь разгребанием вопросов абсолютно не компетентных "клиентов", которые в студию не пошли, а решили сами забубенить себе "супер***носайт", но при этом в голове ноль знаний и не желающие их получать, ну и соответственно нанимать кого то, они же и так выложили кучу денег за DLE, т.к. с опенсорсом не смогли или не захотели подружится. Нужно давать удочку, а не рыбу. Я помойму задал конкретный вопрос: "Какие конкретно "косяки" логировать?" Вместо ответа как обычно флуд не по теме. 19 минут назад, SKYNET74 сказал: И я говорил про отключение вывода ошибок? В том то и проблема, вы начинаете говорить, не читая тему о которой идет речь. А теперь вернитесь в начало темы, прочитайте о чем пишет автор топика. И отвечалось изначально автору топика на вполне конкретный поставленный вопрос про отключение. Но вы никогда не читаете, лишь бы быстрее вырвать фразу из контента, и начать писать только по этой фразе. Автор топика писал про отключение, ему и отвечалось про отключение ошибок. Цитата Ссылка на сообщение Поделиться на других сайтах
SKYNET74 315 Опубликовано: 4 марта 2017 Рассказать Опубликовано: 4 марта 2017 В 26.02.2017 в 18:35, celsoft сказал: Если вы работали в поддержке, то должны знать все ошибки не только выводятся и но записываются в логи сервера, где их можно всегда увидеть, в очень простом и удобном виде. Когда они происходили и с какой периодичностью. Делать двойную работу попросту бессмысленно. Не у всех есть доступ к логам, не все умеют с ними работать, удобнее смотреть на сайте, и опять же включение по желанию... В 26.02.2017 в 18:35, celsoft сказал: Я помойму задал конкретный вопрос: "Какие конкретно "косяки" логировать?" Вместо ответа как обычно флуд не по теме. Да хотя бы записывать куда то фатальные ошибки mysql, а не просто показывать пользователю пугающую табличку с непонятными словечками... В 26.02.2017 в 18:35, celsoft сказал: В том то и проблема, вы начинаете говорить, не читая тему о которой идет речь. А теперь вернитесь в начало темы, прочитайте о чем пишет автор топика. И отвечалось изначально автору топика на вполне конкретный поставленный вопрос про отключение. Но вы никогда не читаете, лишь бы быстрее вырвать фразу из контента, и начать писать только по этой фразе. Автор топика писал про отключение, ему и отвечалось про отключение ошибок. 4 пункт можно прочитать однако, и не только про отключение, а про замену вываливания ошибок. Это бессмысленно обсуждать, нет полного пользовательского логирования, об ошибках работы скриптов подавно речи не может быть... 1 Цитата Ссылка на сообщение Поделиться на других сайтах
Рекомендованные сообщения
Присоединяйтесь к обсуждению
Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.