Попинайте мой первый проект на yii2

Собираем здесь завершённые проекты, использующие Yii. Один проект — одна тема.
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

Доброго дня всем.
Сделал новостной сайт на basic-шаблоне. Хотелось бы узнать компетентное мнение. Если есть замечания (а они наверняка есть) с удовольствием ознакомлюсь и приму к сведению)
http://www.zvu-74.ru/
Аватара пользователя
za4me
Сообщения: 152
Зарегистрирован: 2016.06.26, 15:29
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение za4me »

А что пинать то? Дизайн с версткой?
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Re: Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

za4me писал(а): 2017.04.21, 12:55 А что пинать то? Дизайн с версткой?
Если вы в этом компетентны, почему бы и нет
Аватара пользователя
ElisDN
Сообщения: 5841
Зарегистрирован: 2012.10.07, 10:24
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение ElisDN »

Vadim7423 писал(а): 2017.04.21, 18:40 почему бы и нет
Тогда уж спросите на форуме дизайнеров, а не программистов.
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Re: Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

ElisDN писал(а): 2017.04.21, 18:51 Тогда уж спросите на форуме дизайнеров, а не программистов.
Я всего лишь ответил человеку на его вопрос.
Уж извините, код выложить не могу и доступ в админку тоже дать не могу. Я лишь предлагаю оценить работу с точки зрения программиста, если вы программист, однако если вы компетентны в области дизайна и верстки то ради бога, я и эти замечания готов выслушать. Если нет замечаний то и писать в этой теме не обязательно.
Аватара пользователя
za4me
Сообщения: 152
Зарегистрирован: 2016.06.26, 15:29
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение za4me »

Vadim7423 писал(а): 2017.04.21, 19:58 Я лишь предлагаю оценить работу с точки зрения программиста
Vadim7423 писал(а): 2017.04.21, 19:58 код выложить не могу
Абсурдно звучит.
Если вы хотите замечаний по дизайну и верстке, упоминание Дмитрия выше думаю подходит вам больше всего.
Аватара пользователя
za4me
Сообщения: 152
Зарегистрирован: 2016.06.26, 15:29
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение za4me »

Vadim7423 писал(а): 2017.04.21, 19:58 Я лишь предлагаю оценить работу с точки зрения программиста
Vadim7423 писал(а): 2017.04.21, 19:58 код выложить не могу
Абсурдно звучит.
Если вы хотите замечаний по дизайну и верстке, совет Дмитрия написанный выше думаю подходит вам больше всего.
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Re: Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

za4me писал(а): 2017.04.22, 01:16 Абсурдно звучит.
Если вы хотите замечаний по дизайну и верстке, совет Дмитрия написанный выше думаю подходит вам больше всего.
Абсурдно - это с вашей субъективной точки зрения. Если вы элементарно не можете оценить тот же функционал и удобство реализованных возможностей на сайте, то можно просто промолчать, я уже писал об этом. Зачем писать ради того чтобы по умничать? Какие то вещи типа поиска или авторизации или добавления того же комментария можно и так посмотреть, там не надо быть гением, чтобы определить каким образом это реализовано на уровне кода.
Аватара пользователя
ElisDN
Сообщения: 5841
Зарегистрирован: 2012.10.07, 10:24
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение ElisDN »

Vadim7423 писал(а): 2017.04.22, 07:56 ...там не надо быть гением, чтобы определить каким образом это реализовано на уровне кода.
Не очень удобно раскладываете классы по папкам. А вон-тот код запросов я бы их контроллера вынес в ActiveQuery.
Аватара пользователя
maleks
Сообщения: 1985
Зарегистрирован: 2012.12.26, 12:56

Re: Попинайте мой первый проект на yii2

Сообщение maleks »

Vadim7423 писал(а): 2017.04.22, 07:56 Какие то вещи типа добавления того же комментария можно и так посмотреть,
Попробовал комент добавить, но утомился капчу набирать, это какое то издевательство.

А по сайту, ничего такой портальчик, и архив и теги есть, ничего сложного правда не видно чтобы о чем то судить.
Yii2 universal module sceleton - for basic and advanced templates
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Re: Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

maleks писал(а): 2017.04.22, 09:04 Попробовал комент добавить, но утомился капчу набирать, это какое то издевательство.
Не понял, там же рекапча. Вы устали галочку ставить? Кстати вроде появилась новая версия рекапчи, где от пользователя вообще ничего не требуется, но не пробовал ее пока ставить где то.
ElisDN писал(а): 2017.04.22, 08:41 Не очень удобно раскладываете классы по папкам. А вон-тот код запросов я бы их контроллера вынес в ActiveQuery.
Я пока с вами препирался, понял что неудачно реализовал поиск. Это можно определить взглянув на адресную строку и увидев список параметров.
Вот примерно такого рода замечания я от вас ожидал, не более.
Вы всегда так постанову задачи воспринимаете?
Аватара пользователя
maleks
Сообщения: 1985
Зарегистрирован: 2012.12.26, 12:56

Re: Попинайте мой первый проект на yii2

Сообщение maleks »

Vadim7423 писал(а): 2017.04.22, 09:20
maleks писал(а): 2017.04.22, 09:04 Попробовал комент добавить, но утомился капчу набирать, это какое то издевательство.
Не понял, там же рекапча. Вы устали галочку ставить? Кстати вроде появилась новая версия рекапчи, где от пользователя вообще ничего не требуется, но не пробовал ее пока ставить где то.
С вашей версией рекапчи, мне не только галочку приходится ставить...
https://hostingkartinok.com/show-image. ... 25c5f2b443
Yii2 universal module sceleton - for basic and advanced templates
Vadim7423
Сообщения: 59
Зарегистрирован: 2016.07.07, 20:21

Re: Попинайте мой первый проект на yii2

Сообщение Vadim7423 »

maleks писал(а): 2017.04.22, 09:42 С вашей версией рекапчи, мне не только галочку приходится ставить...
Да, есть такое. Хотел сначала сделать скрытое поле чекбокс, которое в случае заполнения означало бы, что форму заполняет робот, но что то усомнился в надежности такого способа. Возможно вернусь к нему, а может попробую последний вариант рекапчи
Аватара пользователя
Йож
Сообщения: 574
Зарегистрирован: 2015.08.26, 03:05

Re: Попинайте мой первый проект на yii2

Сообщение Йож »

Vadim7423 писал(а): 2017.04.22, 09:20Кстати вроде появилась новая версия рекапчи, где от пользователя вообще ничего не требуется
Гениально!! А пока выбирать кучу витрин, знаков и гор. В топку эту рекапчу. С учетом, что csrf включен, я годами спама через формы не вижу.
А ну еще кастомный validator на наличие ссылки в сообщении. (зачем спамеры спамят? Верно, чтоб ссылку воткнуть в сообщение)
Аватара пользователя
Йож
Сообщения: 574
Зарегистрирован: 2015.08.26, 03:05

Re: Попинайте мой первый проект на yii2

Сообщение Йож »

А что это в поиске за цифры у параметров?
http://www.zvu-74.ru/search?sq=Миасс&ne ... =6&video=5
Почему бы не использовать =1 у разделов, в которых ищем?

Еще можно мильон дублей нафигачить для поисковиков:
http://www.zvu-74.ru/search?sq=Миасс&video=500

Нужно либо canonical использовать, либо вообще запретить в robots.txt /search
Аватара пользователя
pistol
Сообщения: 216
Зарегистрирован: 2014.07.12, 15:18
Откуда: Курган
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение pistol »

В поиске XSS.
Аватара пользователя
rodion_zlobin
Сообщения: 207
Зарегистрирован: 2017.01.11, 16:33

Re: Попинайте мой первый проект на yii2

Сообщение rodion_zlobin »

https://yadi.sk/i/Wgmxvf6r3HgaRS Исправьте меню по возможности. В целом приятные впечатления от сайта
Аватара пользователя
SiZE
Сообщения: 2813
Зарегистрирован: 2011.09.21, 12:39
Откуда: Perm
Контактная информация:

Re: Попинайте мой первый проект на yii2

Сообщение SiZE »

Почта на мэйле... расходимся. :lol:
Dima1987
Сообщения: 20
Зарегистрирован: 2017.06.26, 13:49

Re: Попинайте мой первый проект на yii2

Сообщение Dima1987 »

http://www.zvu-74.ru/web/admin/

не очень красиво смотрится
Ответить