С Новым годом! Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.88/8: Рейтинг темы: голосов - 8, средняя оценка - 4.88
 Аватар для 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
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
05.10.2016, 16:56
Ответы с готовыми решениями:

Собираюсь сделать сайт на ООП с MVC
Собираюсь сделать сайт на ООП с MVC(тестовый, для закрепления зананий), подскажите, где, что и как распределять? какая должна быть...

MVC и ООП
Пытаюсь разобраться с патетрном MVC в стили ООП, и не фига не пойму, если в процедурном стили мне более мение было понятно, то в ООП вообще...

ООП с MVC
Подскажите кто знает. Есть логика в моделе вот код public function get_data() { $result =...

9
Эксперт PHP
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
Эксперт PHP
5755 / 4134 / 1508
Регистрация: 06.01.2011
Сообщений: 11,276
05.10.2016, 18:59
Лучший ответ Сообщение было отмечено FreeZon как решение

Решение

1) Лучше использовать единую точку входа в приложение. То есть, все запросы адресовать на index.php, кроме адресов на CSS-стили, изображение прочие статические вещи.
2) Классы хранить лучше в файлах, имеющих название, как и сам класс. То есть имя "functions.php" не подходит.
3) То, что в файле config.php -- заменить на класс Config, который будет работать с файлами настроек. Файлы настроек делайте в любом подходящем виде. Самый простой -- массив:
PHP
1
2
3
4
5
<?php
 
return [
    'настройка' => 'значение'
];
В общем -- идёте в правильном направлении (я про цель разобраться с MVC), но пока нужно дорабатывать.
1
 Аватар для FreeZon
8 / 10 / 8
Регистрация: 30.12.2013
Сообщений: 577
05.10.2016, 19:08  [ТС]
Para bellum, Спасибо за советы.
0
14 / 50 / 3
Регистрация: 26.12.2009
Сообщений: 406
05.10.2016, 22:53
Цитата Сообщение от Para bellum Посмотреть сообщение
Самое первое и, можно сказать, главное -- это не используется автозагрузка, а всё подключается вручную.
Всему свое время.
Вот когда надоест вручную подключать или станет неудобно, тогда и следует прибегать к автозагрузке.

Добавлено через 44 секунды
Цитата Сообщение от Para bellum Посмотреть сообщение
Лучше использовать единую точку входа в приложение.
Может ему еще фреймворк написать?

Добавлено через 1 минуту
Цитата Сообщение от FreeZon Посмотреть сообщение
Ну че ни кто не хочет посмотреть, или тут все такие начинающие как я, и нету "отцов" по php
Чего Вам так невтерпеж?
Прошел же только час.
0
 Аватар для FreeZon
8 / 10 / 8
Регистрация: 30.12.2013
Сообщений: 577
05.10.2016, 22:56  [ТС]
Madrid, Прогресс не стоит на месте я уже в индексе сделал автолоад. Просто про автолоад рассказывали уже в след. видео уроке. До которого я еще не дошел, но уже посмотрел и разобрался.
Миниатюры
Проверьте сайт на ООП и MVC  
0
14 / 50 / 3
Регистрация: 26.12.2009
Сообщений: 406
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. Продолжая совет предыдущего оратора об автозагрузке.
Когда-то потом почитайте о пространствах имен.
1
 Аватар для FreeZon
8 / 10 / 8
Регистрация: 30.12.2013
Сообщений: 577
05.10.2016, 23:20  [ТС]
Madrid, Спасибо, все почитаю, а про sql иньекции я знаю, только пока не знаю как на основе mvc и ооп сделать валидатор. Буду пока так. Дальше посмотрим что по урокам.

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

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

2. У меня скрины дома не показываются
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
05.10.2016, 23:52
Помогаю со студенческими работами здесь

Литература ООП и MVC
Здравствуйте, поставил себе цель за лето выучить php, mvc, oop , если есть возможность ссылки на литературу для электронной книги (...

Грамотная реализация MVC ООП
Доброе утро всем, уже вторые сутки лазию в просторах интернета и не могу найти нормальную реализацию MVC ООП, без ООП нашел, но все жи...

Разграничение прав в ООП и MVC
Подскажите, как сделать разграничение прав админ/модератор/пользователь в приложении, использующем ООП и MVC Например, в ФП...

-ООП+ -MVC Выборка данных из БД
Всем привет! Столкнулся с проблемой логики))))Точнее с логикой проблемы у меня видимо)) Из названия темы видно, что я пытаюсь изучить... ...

Можно ли писать ооп с mvc?(php)
В общем это весь вопрос. Можно ли писать ооп с mvc?(php)


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
10
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru