Сделайте, пожалуйста, мне код-ревью. Очень хочется услышать критику. https://github.com/json-api-php/json-api

я в ооп написал

сейчас перешлю сюда

сделай codeclimate или scrutinizer-ci интеграцию добавь это в travis

поставь бейджики(build status, codecoverage и тд), стабильной версии (тэгами или веткой пометь)

по самому коду, личное имхо привык к тому что у трейтов в имени на конеце префикс trait например HasMetaTrait чтобы в имени файла и трейта было

далее сделай ResourceInterface и используй его вместо IdentifiableResource

хотя тут тонкости json api которые я не знаю (не работал с этим)

ну и замечание что нет Interface как таковых в коде

напрягает отсутствие пхпдоков, особенно на пропертях

ну и snake_case для именования переменных, его разве кто-то в пхп юзает?

ну дело вкуса наверно

имхо тут 2 интерфейса надо ResourceInterface и RelationshipInterface

еще бы предложил в composer.json добавить команду test

чтобы composer install

composer test было (это опять же личная привычка

и дергать команды из travis.yml которые в composer объявленны

было бы composer test —coverage-clover build/logs/clover.xml

ну и еще там дергается phpunit который хз какой

не из /vendor/bin/phpunit

ну или это происходит не явно

Народ тут о W.P. вопрос задать можно ?) или не стоит )?

задавай, вай нот

а что такое W.P.??

WordPress?

http://www.wotkta.pwpnet.pl/index.php?l=ru&s=pro&f=szp

:)))

?

далее сделай ResourceInterface и используй его вместо IdentifiableResource

Спасибо, спасибо! Супер. Вот это полезно, да.

потом будет время еще посмотрю

я свою либу никак закончить не могу))

ну и замечание что нет Interface как таковых в коде

это ж плюс

это ж плюс

оно спрятано за абстрактный класс

если ты про постфиксы

я как разработчик при отрытие сторонней либы в первую очередь смотрю интерфейсы

если ты про постфиксы

постфиксы мое имхо это хорошо, но либа человека и ему свои подходы насаживать я не хочу

поэтому ему решать

я как разработчик при отрытие сторонней либы в первую очередь смотрю интерфейсы

почему я в виме могу найти интерфейсы, а вы в пхпсторме без постфикса не можете? -_-

вопрос не поиска

почему я в виме могу найти интерфейсы, а вы в пхпсторме без постфикса не можете? -_-

как это не можем?

всё мы можем )

найти можем когда знаем что искать

не знаю, вот человек говорит, что не может

интерфейсы ж

еще раз

у тебя директория и файлы

чтобы понять что внутри файла надо его открыть

это значит погрузится в детали

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

не понимаю зачем для этого постфиксы

не список интерфейсов

где я о списке интерфейсов говорил

я как разработчик при отрытие сторонней либы в первую очередь смотрю интерфейсы

здесь

я говорил что у человека там нет вообще интерфейсов

здесь

слово список покажи

чтобы понять что внутри файла надо его открыть

Шторм же интерфейсы зелёным подсвечивает

смотрю интерфейсЫ

не один, а много

Шторм же интерфейсы зелёным подсвечивает

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

а на гитхабе он никак не показывает в веб интерфейсе и в других местах

код же не писался для phpstorm

не один, а много

чего много ?

интерфейсов лол

не один, а много

ты можешь вести конструктивный диалог?

множественное число

с ы на конце

множественное число

не знаешь что сказать доебывайся до русского

ERROR: S client not available

ты спросил где

я тебе показл

вообще не понимаю к чему ты пристал

ясно все с тобой

еще и стрелки переводишь, мол я доебываюсь

почему я в виме могу найти интерфейсы, а вы в пхпсторме без постфикса не можете? -_-

открой проект на то время когда я делал review и найди там в ходе хоть один интерфейс

в виме или phpstorm или хоть чем

их там просто в коде нет от слова совсем

это ж плюс

если ты про постфиксы

я как разработчик при отрытие сторонней либы в первую очередь смотрю интерфейсы

почему я в виме могу найти интерфейсы, а вы в пхпсторме без постфикса не можете? -_-

Срач! Срач! Срач!

абстрактные классы не нужны, фигач интерфейсы, это ж ясно

почему я в виме могу найти интерфейсы, а вы в пхпсторме без постфикса не можете? -_-

мы можем найти, вопрос в том что при просмотре кода например в github

код никак не выделяется

интерфейсы никак не находятся

поэтому смотреть код на гитхаб не оч

либа это продукт для программистов

еще и к определению быстро не перейти

а програмировать можно на чем угодно

и к модулю

и тип переменной не глянуть

вот поэтому правила хорошего кода там наиболее и понятны

если у тебя код хорош ты его посмотришь где угодно и поймешь

а если ты при просмотре кода прыгаешь по файлам

или занимаешься поиском

возможно что то неудобно?

можно конечно написать что просматривать мой код надо в phpstorm для других редакторов нифига не понятно как он выглядит

и как им пользоваться

в виме тоже ок

и емаксе