Перейти к содержанию
pashast

Нужно ли проверять свой код в strict режиме?

Нужно ли проверять свой код в strict режиме?  

2 пользователя проголосовало

You do not have permission to vote in this poll, or see the poll results. Пожалуйста, войдите или зарегистрируйтесь для возможности голосования в этом опросе.

Рекомендуемые сообщения

Собственно сабж, есть ли смысл проверять свой код в strict mode twigа, если пол-админки опенкарта в этом режиме не работает?

 

Моя лень говорит, что проверяй только в каталоге.

Внутренний перфекционист возмущается.

 

Дениель говорит идите нафиг https://github.com/opencart/opencart/issues/6175#issuecomment-345692932

:-D

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

не нужно, это отнимает ресурсы процессора, если только при отладке, но при отладке можно и вручную включить (либо расширением), потом отключить

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Я имел ввиду именно необходимость отладки в strict режиме.

Ясно что на рабочем сайте strict  включать никто не будет.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
4 часа назад, pashast сказал:

Дениель говорит идите нафиг

ага

Я не проверяю в стрикт, но стараюсь писать так чтобы не было проблем.

 

 

33 минуты назад, lexxkrt сказал:

не нужно, это отнимает ресурсы процессора,

Много?
Вот как раз стрикт, наооборот убирает, например

echo (isset($context["header"]) ? $context["header"] : null);
будет чистый echo

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
21 minutes ago, chukcha said:

echo (isset($context["header"]) ? $context["header"] : null);
будет чистый echo

 

а где то будет типа такого 

echo (isset($context["text_all"]) ? $context["text_all"] : $this->getContext($context, "text_all")); // strict = true

вместо

echo (isset($context["text_all"]) ? $context["text_all"] : null); 

 

и такого

if ((isset($context["categories"]) ? $context["categories"] : $this->getContext($context, "categories"))) { // strict = true

вместо

 if ((isset($context["categories"]) ? $context["categories"] : null)) {

Изменено пользователем lexxkrt

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Возможно, но где нагрузка?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
1 minute ago, chukcha said:

Возможно, но где нагрузка?

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

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

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!

Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.

Войти

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу

×

Важная информация

На нашем сайте используются файлы cookie и происходит обработка некоторых персональных данных пользователей, чтобы улучшить пользовательский интерфейс. Чтобы узнать для чего и какие персональные данные мы обрабатываем перейдите по ссылке. Если Вы нажмете «Я даю согласие», это означает, что Вы понимаете и принимаете все условия, указанные в этом Уведомлении о Конфиденциальности.