Нужен Code Review?

Question

[pers1307]

Всем привет!

Ребята, нужен Code Review. Я начинашка в веб программировании, написал простой блог из 3х страниц. Нужна обратная связь по качеству кода, проэкта и всего всего. Чтобы я мог поднять свой навык в следующем проекте.

Ссылку на блог прилагаю:
https://cloud.mail.ru/public/699V/5r1szrcLs

Comments

[sergey]

советую обратить внимание на github, а то хз что выложили и просите

[pers1307]

SM: Спасибо, обращу.

Answer 1

[index0h]

1. Каталог .idea должен быть проигнорирован
2. MVC - читаем и используем.
3. PSR-1, PSR-2, PSR-4 - читаем и используем.
4. По хорошему, require в вашем коде должен быть 1 раз при подключении автолодера.
5. Вперемешку html+css+js - это гуано))
6. В вашем коде, увы нечего спасать.

Comments

[pers1307]

Спасибо огромное! Учту в следующих проектах. MVC уже начал расшаривать.

[pers1307]

index0h: Рома, привет! Я посмотрел твой канал на ютубе, мне очень понравилось. Прошу тебя: можешь взять надо мной шество и стать моим наставником на пути становления PHP - программистом. Буду бесконечно благодарен. Можешь скинуть свои контакты?

[index0h]

Могу, но не на постоянной основе. Skype: index0h
1-е задание:

Отформатировать код, согласно PSR-1, PSR-2, PSR-4, PSR-5 (он пока-что не принят, посему про стандарт читаем тут: https://github.com/phpDocumentor/fig-standards/blo...
И выложить проект на github.com

Про PSR читаем тут: https://github.com/php-fig/fig-standards/tree/mast...
PSR-4 реализуем на базе composer, про него читеаем тут: https://getcomposer.org/doc/

Рекомендуемые настройки PhpStorm:

Settings->Editor->inspections->PHP->PHPDoc:
'Missing return tag': Severity: Error
'Missing @trows tag(s)': Severity: Error
[x] Analyze immediete calls
[ ] Ignore Runtime exceptions
[ ] Ignore Logic exceptions

'Missing PHPDoc comment': Severity: Error
[ ] Constant
[x] Function [x] Skip if empty
[ ] Class
[x] Method [x] Skip if empty
[x] Field
[ ] Class constant

'PHPDoc comment matches function/method signature': Severity: Error

Требует установки PHP_CodeSniffer- гуглим, устанавливаем.

Settings->Editor->inspections->PHP:
PHP Code Sniffer validation: Severity: Error
'Coding standard': PSR-2

[pers1307]

index0h: Спасибо! Написал в скайп.

Answer 2

[Котик Антон]

Архив?.. На мэйле?.. Скажите, что вы пошутили.

Всегда забавляли подобные копирайты говнокодеров:

<div id="footer">
    <p>&copy; Дизайн - собственность Перескокова Юрия</p>
    <p>&copy; CMS - собственность Перескокова Юрия</p>
</div>

Вот правда, зачем это?

Великолепно, просто нет слов:

class Get_Set_DB_Class
    {
        public function ConnectForDB()
        {
            $mysqli = new mysqli("mysql.0hosting.me","u509998950_pers","13071992","u509998950_blog");
            return $mysqli;
        }
        public function Close($mysqli)
        {
            $mysqli->close();
        }

    }

     $Connect = new Get_Set_DB_Class();

Хотите обратную связь - получайте. Посмотрев ваш код, хочется порекомендовать вам "обратить внимание на великолепные возможности, открывающиеся в области сантехники".

Comments

[Алексей Скобкин]

Владислав Турчинский: И вопросов лишних не задавать :)

[Indus Indusovich]

27cm Правильно ли я понимаю данный товарищь попросту незачем создал класс?Можно ведь было обойтись простой функцией,верно?Я просто тоже пишу что то подобное.

[pers1307]

Indus Indusovich: Да, данный класс лишний. Вообще писал проект на скорую руку, поэтому есть косяки, которые даже мне очевидны.

[pers1307]

27cm: Я конечно учту ваши пожелания по коду. Но, искренне советую перестать быть настолько едким человеком. Вам же легче жить будет. И не стоит мнить себя Богом, это ещё некому на пользу не шло.
А от цели я своей не откажусь, это принципиально.

Answer 3

[sanex3339]

MVC

Comments

[pers1307]

Спасибо! Уже начал учить.

Answer 4

[Андрей Саныч]

и это.. храните пароли в зашифрованном виде

Comments

[pers1307]

Спасибо! Обязательно озадачусь этой проблемой.