Перевод гостевой книги с процедурного стиля на ООП

@Орбит · Регистрация: 30.04.2013

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

Всем привет! Есть у меня простенькая гостевая книга (файл index.php). На практике сказали избавиться от процедурного метода и поработать с классами, собственно что из этого получилось вы можете наблюдать в файле oop.php. Знающие люди укажите на ошибки и что исправить, т.к с этими классами всю голову уже сломал.

@tarasalk · 15.03.2017, 12:50

Короче главный ваш косяк в том, что вы просто обернули процедурный код в класс. Это не ООП.

Остальные ошибки:
1) не надо наследовать guestbook от database. Это разные сущности. А чтобы получить доступ к бд можно использовать агрегацию.

PHP
1
2
3
4
5
6
7
class GuestBook {
    protected $db;
 
    public function __construct($db) {
        $this->$db = $db;
    }
}

2) параметры подключения к БД жестко прописаны в классе database. Лучше хранить их отдельно в конфиге и передавать в класс через конструктор. Да и вообще, смысла в этом классе мало, хотя бы синглтон сделайте из него.
3) такое оформление никуда не годится, почитайте PSR.
4) это

PHP
1
if ($this->Submitbutton == true)

можно записать так

PHP
1
if ($this->Submitbutton)

5) не надо пихать js в php.
6) класс guestbook умеет слишком многое. Как минимум работа с бд отдельно, вывод html отдельно.

@andr-xw · 15.03.2017, 16:00

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

4) это

PHP
1
if ($this->Submitbutton == true)

можно записать так

PHP
1
if ($this->Submitbutton)

А я предпочитаю так

PHP
1
if (true === $this->Submitbutton)

@fanatikus · 15.03.2017, 17:17

Сообщение от andr-xw

А я предпочитаю так

а смысл? только больше букоффф

@andr-xw · 15.03.2017, 17:36

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

а смысл? только больше букоффф

Явное указание на тип переменной

Новые блоги и статьи Все статьи Все блоги /
Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .
модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .	Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ