«phpClub» — архив тем ("тредов"), посвящённых изучению PHP и веб-технологий.
Аноним 2018/10/22 20:25:31  №1283020 1
Как перестать прокрастинировать и сделать наконец тестхаб?
Ответы: >>1283051 >>1283051
Аноним 2018/10/22 21:09:14  №1283051 2
>>1283020
Оп, знаю что занят. Но глянь моих студентов:
https://github.com/Awesome-Kirill/fukingStudent
>>1283020
Я к выходным первую версию своего ТестХаба выложу
Ничего сложного, без задней мысли берешь и делаешь
Ответы: >>1285903
Аноним 2018/10/28 12:40:51  №1285903 3
>>1283051

В дополнение к замечаниям тут: >>1285501

В шаблонах надо убрать копипасту. Ты скопировал шапку и подвал в каждый шаблон, этого не должно быть.

Имена шаблонов лучше сделать соответствующими именам контроллеров и экшенов, а не случайными.

https://github.com/Awesome-Kirill/fukingStudent/blob/master/src/Controller/ListController.php#L36
> if (isset($_COOKIE['isLogin']) and $this->authentication->isValidCookie($this->model, $_COOKIE['isLogin'])) {

Это нарушение инкапсуляции. По идее, знать, как называется кука, должен только класс авторизации. Ты же вместо этого копипастишь название куки по всему коду.

Ты используешь let, это отекает большое число старых браузеров. Не очень понятно, в чем выгода. Ты знаешь, какие есть стандарты JS, в чем их различия и осознанно сделал выбор или же не думая скопипастил из какой-то статьи?

https://github.com/Awesome-Kirill/fukingStudent/blob/master/src/tmp/allVue.html#L6
> <meta name="viewport" content="initial-scale=1, maximum-scale=1, user-scalable=no">

Зачем ты запрещаешь масштабировать страницу? Этот код придумали люди, которым лень было делать нормальную верстку и проще показалось запретить масштабирование. Я негативно отношусь к таким людям.

> <hr class="container-fluid">
Что за ерунда? Почему hr?

https://github.com/Awesome-Kirill/fukingStudent/blob/master/src/tmp/vue-bootstrap-table.js
12 000 строк - не многовато ли для отображения простой таблицы?

Фреймворки вроде bootstrap лучше класть в отдельную папку, а не сваливать все в кучу. Сторонний код желательно явно отделять от своего.

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

При ошибке надо показывать форму с заполненными полями, а не редиректить.