Jump to content
покупайте без комиссии
Search In
  • More options...
Find results that contain...
Find results in...
  • Sign Up

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


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

2 members have voted

You do not have permission to vote in this poll, or see the poll results. Please sign in or register to vote in this poll.

Recommended Posts

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

 

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

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

 

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

:-D

 

Link to post
Share on other sites

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

Link to post
Share on other sites

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

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

Link to post
Share on other sites

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

Link to post
Share on other sites

4 часа назад, pashast сказал:

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

ага

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

 

 

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

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

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

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

Link to post
Share on other sites
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)) {

Edited by lexxkrt
Link to post
Share on other sites

1 minute ago, chukcha said:

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

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

Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.