Comments 36
Как редко на хабре можно увидеть вменяемый php код, который не заслуживает ни строчки критики (ну разве только добавить типизацию). Прямо только за это можно говорить что вы — молодец.
UPD. Ой, открыл репозиторий, забираю свои слова обратно. Особая дичь в файлике gerda.php. Но всё равно это лучше, чем у большинства.
Я не заморачивался с ООП, поэтому преднамеренно использовал слово „скрипт“ с самого начала.
Типизацию я сам люблю, у нас в компании почти запрещено использовать двойные равно.
Да дело не в двойных равно. 99% всех сюда писавших не заморчаиваются даже по использованию стандартов. Я увидел код и обрадовался, мол вот, код отформатирован по PSR, можно даже читать.
Ан нет, открыл репу на гитхабе с глобальными константами, с выравниванием табами, без поддержки композера и докой на русском языке (т.е. явно видно что написан по-быстрому за пару часиков). Но это всё равно, повторюсь, лучше, нежели у тех, кто делает публикации, приводя в качестве примера глобалсы и стиль php начала двухтысячных.
Если подытоживать — ваш код вполне читаемый, но одноразовый и не расширяемый. Туда не всунуть ни симфонёвую консоль, ни поставить локально в систему, ну т.е. никуда. Но в качестве примера вполне годится.
ваш код вполне читаемый, но одноразовый и не расширяемый. Туда не всунуть ни симфонёвую консоль, ни поставить локально в систему, ну т.е. никуда.
Вы уверены, что все это нужно для наколеночного скрипта? Тут ведь главная идея — не код на PHP показать.
Ну, по пунктам:
Этот код и так был написан за пару часиков.
По поводу PSR. Я использую PSR-2T — PSR-2 со смарт-табуляцией.
Для меня это дело принципа. Моё мнение по этому вопросу вот такое и пока что не менялось:
https://habrahabr.ru/post/308974/#comment_9783180
Глобальная константа там всего одна — это версия скрипта, ну вы серьёзно?!
Первая дока на русском языке преднамеренно, потому что предназначена для русскоязычной аудитории (поэтому и пост на Хабре, а не на чём-то англоязычном, хотя я и говорю по-английски). Если будет время (у меня ещё есть работа) — будет дока и на других языках.
- По поводу композер-пакета полностью согласен — можно сделать.
Когда работаешь в разных средах, на разных компах и не своих в том числе + консоли, то «прыгание табов» просто раздражает, где то это 2 пробела, где то 4, где то 8, встречал и 3 пробела.
Ну а пробел это пробел, лишь бы шрифт был моноширинный (хотя должно быть пофиг, они же ведущие)
https://github.com/ben174/git-draw
Это удобнее. Простите :)
Супер. Я подготовил шаблон для тех, кто работает на Бали:
$commits = [
' ' // Sun
' ### # # ## ### ## # # # # ## ## ## ## ### # # ' // Mon
' # # # # # # # # # # # # # # # # # # # # # ## ' // Tue
' # ## ### ## # # ### #### # # ### ### ### # # # # # ' // Wed
' # # # # # # # # # # # # # # # # # # # # # ## # ' // Thu
' ### # # #### ### ## # # # ## ## ## #### # # # # ' // Fri
' ' // Sat
];
А GitHub не забанит?
Я не удержался и повторил на свой лад:
https://github.com/weekend-software/gerpy
Простите, а за что он должен забанить? Не думаете же вы, что каждый проект на GitHub'е должен начинаться именно на нём. Может хранилище существовало уже год, а автор только сегодня решил его импортировать? Git — это же распределённая система контроля версий, в ней нет главного хранилища, они все равноправны.
> поведение Гитхаба при построении графика активности, учитывающее
> предоставленные клиентом локальные даты коммитов.
Преднамеренное, поскольку дата коммита подписывается, собственно, при коммите. Если бы сервис исправлял даты коммитов — вряд ли он набрал такую популярность, поскольку его задача — доступ к исходникам, а не контроль за тем, когда разработчик будет делать commit/push.
Рисуем коммитами на Гитхабе