Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

Бомбалейла


ArtemPitov
 Share

Recommended Posts

Периодически или даже часто  вижу вот такие приколы с табуляцией 

 

Спойлер

0000000.png

 

и это не только в js так же в php и html 

 

Вот скажите господа, вот как так ? 

 

Спойлер

для php оставлю тут psr-1 https://www.php-fig.org/psr/psr-1/

 

 

Link to comment
Share on other sites

Для отступов, по-моему, psr-2 нужно приводить, в первом ничего нет.

Но для чистоты нужно сказать, что psr-2 нам говорит 
 

Quote

Code MUST use 4 spaces for indenting, not tabs.

Не использовать табуляцию, а использовать 4 пробела.


Тогда как сам опенкарт немного другие требования предъявляется к стилю:
 

Quote

 

Indentation

PHP files must be indented using the TAB character. 4 space tabs are not supported.

HTML in template files (.tpl) must be indented using 2 spaces, not 4 spaces or TABS. JavaScript must be indented using the TAB character.

 

В PHP файлах отступы делать табами, 4 пробела стандартом не поддерживаются.

HTML шаблоны (.tpl)  должны отступаться 2умя пробелами, не 4мя или табами. JS должен использовать таб для отступа.

 

 

 

Edited by Gorman
Link to comment
Share on other sites

Ой только давай не будем разводить холивар на тему Tab или 4 пробела.. Это путь в никуда.
А вообще в каждом нормальном редакторе, IDE есть автоформатирование кода, которое его в любом случае php tpl js или html сделает красивым и понятным.
ну есть уникумы которые пишут в notepad++ или куда хуже... так эти динозавры в принципе не должны кодить... есть sublime, phpstorm, codelobster и куда других современных редакторов
P.S мне лично табами удобнее

Link to comment
Share on other sites

20 минут назад, Gorman сказал:

Для отступов, по-моему, psr-2 нужно приводить, в первом ничего нет.

я привел начало всего, дальше как говорится больше 

Link to comment
Share on other sites

14 минут назад, Waha сказал:

Ой только давай не будем разводить холивар на тему Tab или 4 пробела.. Это путь в никуда.

в IDE это на табы настраивается и все прекрасно работает 

 

Link to comment
Share on other sites

2 часа назад, ArtemPitov сказал:

Периодически или даже часто  вижу вот такие приколы с табуляцией 

 

  Показать контент

0000000.png

 

и это не только в js так же в php и html 

 

Вот скажите господа, вот как так ? 

 

  Показать контент

для php оставлю тут psr-1 https://www.php-fig.org/psr/psr-1/

 

 

А ниче что ты js-скрипт привел в пример?, к которому psr по моему не имеет никакого отношения.

Link to comment
Share on other sites


3 минуты назад, Yoda сказал:

А ниче что ты js-скрипт привел в пример?, к которому psr по моему не имеет никакого отношения.

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

Link to comment
Share on other sites

22 hours ago, ArtemPitov said:

 

Вот скажите господа, вот как так ? 

 

  Reveal hidden contents

для php оставлю тут psr-1 https://www.php-fig.org/psr/psr-1/

 

 

 

при написании кода под опенкарт ориентироваться нужно больше на стандарты кодирования опенкарт, да есть такие: https://github.com/opencart/opencart/wiki/Coding-standards
потому что по логике весь код и движка и модулей, должен ему соответствовать
а там есть: "JavaScript must be indented using the TAB character."
И вопрос это не философии, код должен быть однообразным, а не как кому более удобно смотреть на своем личном мониторе.. потому что тогда будет бардак, что собственно и есть..
Но для того, чтобы это работало нужно принудительно проверять все модули на соответствие стандартов и если что-то не так - возвращать на доработку, как есть в других движках. Пока этого не будет будет полный бардак, так как каждый будет писать код так, как ему больше нравится, а всем остальным придется с этим всем работать.. 

  • +1 2
Link to comment
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
 Share

  • 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.