Проверьте сайт на ООП и MVC

@FreeZon · Регистрация: 30.12.2013

Студворк — интернет-сервис помощи студентам

Привет, я тут смотрел курс по php начальный. И начал второй (2 урока).

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

Я хочу чтобы кто нибудь из вас посмотрел мой сайт, и сказал все ли там верно сайт по паттерну mvc и ооп
Или не совсем какие замечания будут, пожалуйста. Одному очень сложно разбираться, но стараюсь. Вот на гит залил проект:
https://github.com/imbaShadow/1essonr.ru

Добавлено через 57 минут
Ну че ни кто не хочет посмотреть, или тут все такие начинающие как я, и нету "отцов" по php

@Para bellum · 05.10.2016, 17:41

Что конкретно нужно оценить? Самое первое и, можно сказать, главное -- это не используется автозагрузка, а всё подключается вручную.

@FreeZon · 05.10.2016, 18:30 **[ТС]**

Para bellum, Я только учусь, и хочу чтобы вы дали критику коду, правильно ли сделал хоть чуть чуть похоже это на MVC и OOП. И есть ли какие замечания к коду, может где то можно было сделать иначе, проще, правильнее.

@Para bellum · 05.10.2016, 18:59

1) Лучше использовать единую точку входа в приложение. То есть, все запросы адресовать на index.php, кроме адресов на CSS-стили, изображение прочие статические вещи.
2) Классы хранить лучше в файлах, имеющих название, как и сам класс. То есть имя "functions.php" не подходит.
3) То, что в файле config.php -- заменить на класс Config, который будет работать с файлами настроек. Файлы настроек делайте в любом подходящем виде. Самый простой -- массив:

PHP
1
2
3
4
5
<?php
 
return [
    'настройка' => 'значение'
];

В общем -- идёте в правильном направлении (я про цель разобраться с MVC), но пока нужно дорабатывать.

@FreeZon · 05.10.2016, 19:08 **[ТС]**

Para bellum, Спасибо за советы.

@Madrid · 05.10.2016, 22:53

Сообщение от Para bellum

Самое первое и, можно сказать, главное -- это не используется автозагрузка, а всё подключается вручную.

Всему свое время.

Вот когда надоест вручную подключать или станет неудобно, тогда и следует прибегать к автозагрузке.

Добавлено через 44 секунды

Сообщение от Para bellum

Лучше использовать единую точку входа в приложение.

Может ему еще фреймворк написать?

Добавлено через 1 минуту

Сообщение от FreeZon

Ну че ни кто не хочет посмотреть, или тут все такие начинающие как я, и нету "отцов" по php

Чего Вам так невтерпеж?
Прошел же только час.

@FreeZon · 05.10.2016, 22:56 **[ТС]**

Madrid, Прогресс не стоит на месте я уже в индексе сделал автолоад. Просто про автолоад рассказывали уже в след. видео уроке. До которого я еще не дошел, но уже посмотрел и разобрался.

@Madrid · 05.10.2016, 23:05

По теме:
1. Внешне вроде норм. Долго учитесь?

2. Несколько раз встречается

Code
1
$mysql = new SQL(DB_HOST,DB_NAME,DB_LOGIN,DB_PASSWORD);

Вряд ли стоит так делать.

3. Часть конфигов в массиве, часть в константах.

4.

Code
1
$sql = "INSERT INTO `news` (`title`, `descr`, `date`) VALUE ('".$title."', '".$descr."','".time()."')";

На первое время покатит, потом почитайте об SQL-инъекциях.

5.
queryArray
queryExec

Много таких query* Вы собрались делать?

6.

Code
1
<li>Описание:<?php echo $value['descr']; ?></li>

Потом почитайте об XSS-атаках.

7. Продолжая совет предыдущего оратора об автозагрузке.
Когда-то потом почитайте о пространствах имен.

@FreeZon · 05.10.2016, 23:20 **[ТС]**

Madrid, Спасибо, все почитаю, а про sql иньекции я знаю, только пока не знаю как на основе mvc и ооп сделать валидатор. Буду пока так. Дальше посмотрим что по урокам.

У меня вот такой вопрос, я если я подключаюсь к тому контроллеру который дал гет запросе и экшен, и я хочу добавить еще один файл из view в том же index.php надо писать я прикрепил скриншот, и обновил гит файлы. Посмотрите пожалуйста.

@Madrid · 05.10.2016, 23:52

1. MVC тут вообще не при чем.
MVC - о разделении кода на уровни, не стоит с ним слишком заморачиваться. Просто понимать, что все лепить в кучу зачастую не стоит, так как это неудобно поддерживать.
ООП тут тоже не при чем. Просто сделайте корректную обработку данных (это могут быть подготовленные выражения или экранирование) в методах БД, которые принимают параметры.

sql-инъекции нужно не валидировать, а правильно обрабатывать (как данные, а не код).
Это должно лежать в классе по работе с БД (можно сказать уровень M).

2. У меня скрины дома не показываются

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@FreeZon 8 / 10 / 8 Регистрация: 30.12.2013 Сообщений: 577

	Проверьте сайт на ООП и MVC 05.10.2016, 16:56. Показов 1758. Ответов 9 Метки нет (Все метки) Привет, я тут смотрел курс по php начальный. И начал второй (2 урока). В итоге, там было задание сделать сайт новостной с архитектурой mvc и как минимум работа с базой была ооп. Я смотрел записи курсов, и я не могу ни как на проверку дать задания автору т.к он уже прошел и платный. Я хочу чтобы кто нибудь из вас посмотрел мой сайт, и сказал все ли там верно сайт по паттерну mvc и ооп Или не совсем какие замечания будут, пожалуйста. Одному очень сложно разбираться, но стараюсь. Вот на гит залил проект: https://github.com/imbaShadow/1essonr.ru Добавлено через 57 минут Ну че ни кто не хочет посмотреть, или тут все такие начинающие как я, и нету "отцов" по php 0

@Para bellum 5755 / 4134 / 1508 Регистрация: 06.01.2011 Сообщений: 11,276
	05.10.2016, 17:41
	Что конкретно нужно оценить? Самое первое и, можно сказать, главное -- это не используется автозагрузка, а всё подключается вручную. 1

@FreeZon 8 / 10 / 8 Регистрация: 30.12.2013 Сообщений: 577
	05.10.2016, 18:30 [ТС]
	Para bellum, Я только учусь, и хочу чтобы вы дали критику коду, правильно ли сделал хоть чуть чуть похоже это на MVC и OOП. И есть ли какие замечания к коду, может где то можно было сделать иначе, проще, правильнее. 0

@FreeZon 8 / 10 / 8 Регистрация: 30.12.2013 Сообщений: 577
	05.10.2016, 19:08 [ТС]
	Para bellum, Спасибо за советы. 0

@FreeZon 8 / 10 / 8 Регистрация: 30.12.2013 Сообщений: 577
	05.10.2016, 22:56 [ТС]
	Madrid, Прогресс не стоит на месте я уже в индексе сделал автолоад. Просто про автолоад рассказывали уже в след. видео уроке. До которого я еще не дошел, но уже посмотрел и разобрался. Миниатюры 0

@FreeZon 8 / 10 / 8 Регистрация: 30.12.2013 Сообщений: 577
	05.10.2016, 23:20 [ТС]
	Madrid, Спасибо, все почитаю, а про sql иньекции я знаю, только пока не знаю как на основе mvc и ооп сделать валидатор. Буду пока так. Дальше посмотрим что по урокам. У меня вот такой вопрос, я если я подключаюсь к тому контроллеру который дал гет запросе и экшен, и я хочу добавить еще один файл из view в том же index.php надо писать я прикрепил скриншот, и обновил гит файлы. Посмотрите пожалуйста. Миниатюры 0

@Madrid 14 / 50 / 3 Регистрация: 26.12.2009 Сообщений: 406
	05.10.2016, 23:52
	1. MVC тут вообще не при чем. MVC - о разделении кода на уровни, не стоит с ним слишком заморачиваться. Просто понимать, что все лепить в кучу зачастую не стоит, так как это неудобно поддерживать. ООП тут тоже не при чем. Просто сделайте корректную обработку данных (это могут быть подготовленные выражения или экранирование) в методах БД, которые принимают параметры. sql-инъекции нужно не валидировать, а правильно обрабатывать (как данные, а не код). Это должно лежать в классе по работе с БД (можно сказать уровень M). 2. У меня скрины дома не показываются 0

Проверьте сайт на ООП и MVC

Решение