Comments 36

Как редко на хабре можно увидеть вменяемый php код, который не заслуживает ни строчки критики (ну разве только добавить типизацию). Прямо только за это можно говорить что вы — молодец.


UPD. Ой, открыл репозиторий, забираю свои слова обратно. Особая дичь в файлике gerda.php. Но всё равно это лучше, чем у большинства.

Я не заморачивался с ООП, поэтому преднамеренно использовал слово „скрипт“ с самого начала.


Типизацию я сам люблю, у нас в компании почти запрещено использовать двойные равно.

Да дело не в двойных равно. 99% всех сюда писавших не заморчаиваются даже по использованию стандартов. Я увидел код и обрадовался, мол вот, код отформатирован по PSR, можно даже читать.


Ан нет, открыл репу на гитхабе с глобальными константами, с выравниванием табами, без поддержки композера и докой на русском языке (т.е. явно видно что написан по-быстрому за пару часиков). Но это всё равно, повторюсь, лучше, нежели у тех, кто делает публикации, приводя в качестве примера глобалсы и стиль php начала двухтысячных.


Если подытоживать — ваш код вполне читаемый, но одноразовый и не расширяемый. Туда не всунуть ни симфонёвую консоль, ни поставить локально в систему, ну т.е. никуда. Но в качестве примера вполне годится.

ваш код вполне читаемый, но одноразовый и не расширяемый. Туда не всунуть ни симфонёвую консоль, ни поставить локально в систему, ну т.е. никуда.


Вы уверены, что все это нужно для наколеночного скрипта? Тут ведь главная идея — не код на PHP показать.

Разумеется, главная идея — это показать алгоритм, а не код, вы правы.

Ну, по пунктам:


  • Этот код и так был написан за пару часиков.


  • По поводу PSR. Я использую PSR-2T — PSR-2 со смарт-табуляцией.
    Для меня это дело принципа. Моё мнение по этому вопросу вот такое и пока что не менялось:
    https://habrahabr.ru/post/308974/#comment_9783180


  • Глобальная константа там всего одна — это версия скрипта, ну вы серьёзно?!


  • Первая дока на русском языке преднамеренно, потому что предназначена для русскоязычной аудитории (поэтому и пост на Хабре, а не на чём-то англоязычном, хотя я и говорю по-английски). Если будет время (у меня ещё есть работа) — будет дока и на других языках.


  • По поводу композер-пакета полностью согласен — можно сделать.
Если бы карма позволяла — люто заплюсовал бы. Отступы табами, выравнивание пробелами, остальное — для мазохистов (если в личном проекте) и садистов (если это кто-то другой ещё и поддерживать должен).

А вы тут не плюсуйте, это слишком локальная тусовка; если пишете на PHP, присоединяйтесь к стандарту, который я упомянул, давайте опишем его нормально, сделаем стиль для кодснифера и тому подобное :-)

Это как то исторически.
Когда работаешь в разных средах, на разных компах и не своих в том числе + консоли, то «прыгание табов» просто раздражает, где то это 2 пробела, где то 4, где то 8, встречал и 3 пробела.
Ну а пробел это пробел, лишь бы шрифт был моноширинный (хотя должно быть пофиг, они же ведущие)
Прыгает только когда половина начальных отступов пробелами а половина табами. Как говорится — взболтать, но не смешивать.
Потому что пока что даже MSVS2017RC не умеет правильно определять, когда уместно использовать пробел, а когда — таб. Есть всякие платные расширения, но платить за то, что возможно кто-то захочет изменить ширину таба… Проще поставить в all languages «insert 4 spaces instead of tab», поправить для css/js на «2 пробела» и быть уверенным, что на всех платформах отображаться будет одинаково. Правда, бекспейсом становится пользоваться неудобнее, но тут уж шашечки или ехать.
UFO landed and left these words here
Такая же идея в голову приходила… но мысль, что ради этого придется целый год комитить (пусть даже и не вручную) отбила желание

Супер. Я подготовил шаблон для тех, кто работает на Бали:


для тех, кто работает на Бали
$commits = [
    '                                                                        ' // Sun
    '  ###  #  #  ##    ###   ##  #  # #  #  ##   ##   ##   ##     ### #   # ' // Mon
    ' #     # #     #  #   # #  # #  # #  # #  # #  # #       #   #  # #  ## ' // Tue
    ' #     ##    ###    ##  #  #  ### #### #  # ###  ###   ###   #  # # # # ' // Wed
    ' #     # #  #  #  #   # #  #    # #  # #  # #    #  # #  #   #  # ##  # ' // Thu
    '  ###  #  #  ####  ###   ##     # #  #  ##   ##   ##   #### #   # #   # ' // Fri
    '                                                                        ' // Sat
];
Нет ничего более интересного, чем использовать что-то не по назначению.
UFO landed and left these words here
UFO landed and left these words here

Эээээ? Я не ставил, за что?
(Плюсанул, только не расстраивайтесь.)

UFO landed and left these words here
UFO landed and left these words here
Я как-то массивы данных с некоего изделия в виде неприличного слова «нарисовал». Начальник отдела не оценил шутку.

Простите, а за что он должен забанить? Не думаете же вы, что каждый проект на GitHub'е должен начинаться именно на нём. Может хранилище существовало уже год, а автор только сегодня решил его импортировать? Git — это же распределённая система контроля версий, в ней нет главного хранилища, они все равноправны.

> Используется существующее на данный момент (преднамеренное?)
> поведение Гитхаба при построении графика активности, учитывающее
> предоставленные клиентом локальные даты коммитов.

Преднамеренное, поскольку дата коммита подписывается, собственно, при коммите. Если бы сервис исправлял даты коммитов — вряд ли он набрал такую популярность, поскольку его задача — доступ к исходникам, а не контроль за тем, когда разработчик будет делать commit/push.
Only those users with full accounts are able to leave comments. Log in, please.