Какие советы вы можете дать к имеющемуся коду?

Question

[Карл Майнхофф]

Не так давно я кидал сюда свой репозиторий "велосипеда", который я пишу. Я получил очень много конструктивной критики, которая мне хорошо помогла. Теперь я делаю это еще раз, чтобы понять, насколько улучшился мой код.
Вот ссылка на репозиторий: bitbucket.org/kfkdevelcenter/intranet-core
core/http писал не я, и его пока не трогаю. Больше всего интересует мнение о библиотеке для баз данных.

Comments

[Александр Аксентьев]

You do not have access to this repository.

[GeekFromUa]

Насколько я знаю в битбаккет чтоб шарить надо платный аакаунд а в гидхабе наоборот, или я ошибаюсь?

[Avrong]

GeekFromUa: На Bitbucket бесплатны как открытые, так и закрытые репозитории.

[GeekFromUa]

Алексей Трифонов: Окей, спасибо, буду знать

[Карл Майнхофф]

Александр Аксентьев, GeekFromUa : пофиксил, теперь

Answer 1

[dmitriy]

сразу замечание .idea нужно игнорить, и вообще прогоните через https://scrutinizer-ci.com он вам оценку вашего кода и произведет.

Comments

[Карл Майнхофф]

Спасибо, попробую

[Карл Майнхофф]

Попробовать не получилось. Этот сервис поддерживает только гит, а я использую hg

Answer 2

[Александр Аксентьев]

Кроме очевидного "зачем".

Зачем там извращаться над автолоадером?
www.php-fig.org/psr/psr-4

В целом код не очень
www.php-fig.org/psr/psr-1
www.php-fig.org/psr/psr-2
if/else без фигурных скобок вообще вырывают мозг, особенно когда рядом есть скобки для другого уровня вложения.

Кроме SQLite ничего не будет поддерживать при желании - запросы строятся без учета того что за база подключена.

switch(){}
https://bitbucket.org/kfkdevelcenter/intranet-core...,256,275,289

В целом без примера кода не очень понятно что на выходе получается.

Comments

[Карл Майнхофф]

Насчет аутолоадера - спасибо за ссылку, почему то не мог найти PSR на эту тему. Query builder тестировал с mysql, вродь работало. Ежели руки дойдут, потестирую на всех заявленных. Switch я не использую вовсе, возможно это и неправильно. Скобки вокруг одинарных ифов поставлю, возможно оно получше станет. А насчет "код не очень" - можно ли поподробнее? Вроде я сделал тем стандартам, которые вы указали. Спасибо за помощь.

[Александр Аксентьев]

Карл Кремень: "не очень"я как раз и писал по поводу оформления скобок местами, еще отступы какие-то очень большие.
И еще функциональные части немного грязновато выглядят местами, код большими кучами концентрированными написан как бы :)
Читабельность плохая.

Свитчей очень не хватает, целые стены IFов, тоже выглядит ужасно и не читабельно.

[Карл Майнхофф]

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