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

Бомбалейла


Recommended Posts

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

 

Спойлер

0000000.png

 

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

 

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

 

Спойлер

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

 

 

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

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

Link to post
Share on other sites
20 минут назад, Gorman сказал:

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

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

Link to post
Share on other sites
14 минут назад, Waha сказал:

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

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

 

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

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

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

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

Link to post
Share on other sites

— Можно списать у тебя?

— Без проблем, только измени немного.

 

BH3uLHSRhSI.jpg

Edited by ImLucky
Link to post
Share on other sites

Почему я, имея монитор, к примеру, 21:9 должен вглядываться в код при 2 спейсах, если могу использовать табы(4) и нормально читать код?

Link to post
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 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.