PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Вопрос по качеству кода

Форумы портала PHP.SU » PHP » Программирование на PHP » Вопросы новичков (Модераторы: OrmaJever, Саныч, Строитель)

Страниц (1): [1]

Описание: Я говно кодер?

Поиск в теме | Версия для печати

No0k	Отправлено: 19 Мая, 2014 - 16:51:42
Новичок Покинул форум Сообщений всего: 44 Дата рег-ции: Янв. 2014 Помог: 0 раз(а)	Форумчене, помогите пожалуйста определить. Говнокод или нет? Код создания категорий (в case break, потому что будет много разделов): Спойлер (Отобразить) PHP: скопировать код в буфер обмена <?PHP include '../../engine/includes/start.php'; $set['title'] = 'Админ панель'; include H.'/engine/includes/head.php'; if (!$admin && !$user_id) { echo Core::msg_show('У вас недостаточно прав для просмотра этой страницы!<br />'); include H . 'engine/includes/foot.php'; } switch ($act) { case 'add_section': if (isset($_POST['submit'])) { if (empty($_POST['position'])) $error .= 'Введите пожалуйста номер позиции!'; if (mb_strlen($_POST['position']) > 3) $error .= 'Слишком много каталогов!'; if (empty($_POST['name'])) $error .= 'Не введено название!<br />'; if (mb_strlen($_POST['name']) > 100) $error .= 'Слишком длинное название!<br />'; if (empty($_POST['description'])) $error .= 'Не введено описание раздела!<br />'; if (mb_strlen($_POST['description']) > 256) $error .= 'Слишком длинное название!<br />'; if (!isset($error)) { $sql->query("INSERT INTO `community_section` SET `position` = '".intval($_POST['position'])."', `name` = '".my_esc(trim($_POST['name']))."', `description` = '".my_esc(trim($_POST['description']))."' "); ?> Категория успешно создана. » <a href="/community/index.php">Ко всем категориям</a> <? } if (isset($error)) echo Core::msg_show($error); } ?> <form method="post" action="?act=add_section"> Позиция: <input type="text" name="position"/> Название раздела: <input type="text" name="name"/> Описание: <textarea name="description"></textarea> <input type="submit" name="submit" value="Создать"/> </form> » <a href="?">Отмена</a><br/> <? break; } include incDir . 'foot.php'; ?> Вот код самого вывода: Спойлер (Отобразить) PHP: скопировать код в буфер обмена <?PHP include '../engine/includes/start.php'; $set['title'] = 'Список разделов'; include incDir . 'head.php'; if($admin) { ?> <a class="link" href="admin/admin.php?act=add_section">Создать раздел</a> <? } echo "<div class='menu'>"; $res = $sql->query("SELECT * FROM `community_section` ORDER BY `position`")->result; while($post = $sql->fetch($res)) { echo '<a href="razd.php?id_razd='.$post['id'].'">'.$post['name'].'</a> <br />'; echo text::output ($post['description']).'<br />'; if($admin) { echo '[<a href="index.php?razd=edit&id_razd='.$post['id'].'">редактировать</a> \| <a href="index.php?razd=delete&id_razd='.$post['id'].'">Удалить</a>] <br />'; } } echo '</div>'; include incDir . 'foot.php'; ?> Можете подсказать ошибки? Если говно код, то аргументируйте пожалуйста. Заранее благодарю. И еще вопрос, я выводил там див так: PHP: скопировать код в буфер обмена <? echo '<div class="menu">'; // код echo '</div>'; ?> или лучше закрыть PHP синтаксис, написать на HTML и потом обратно открыть? (Отредактировано автором: 19 Мая, 2014 - 16:55:48)

Мелкий	Отправлено: 19 Мая, 2014 - 17:20:44
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	No0k пишет: в case break, потому что будет много разделов Говнокод. (спойлер открыть ещё не успел) Читайте "Совершенный код" С. Макконнелла, "Рефакторинг. Улучшение существующего кода" М. Файлера switch - это всегда кандидат на рефакторинг, тем более в языке, умеющем ООП - switch заменяется полиморфизмом. No0k пишет: или лучше закрыть PHP синтаксис, написать на HTML и потом обратно открыть? Лично мне не нравится эта гора echo. А зачем открывать обратно? ----- PostgreSQL DBA

Bio man	Отправлено: 19 Мая, 2014 - 17:30:44
Постоянный участник Покинул форум Сообщений всего: 2751 Дата рег-ции: Июль 2010 Откуда: Даугавпилс, Латвия Помог: 52 раз(а)	Конечно говнокод. Нужно как то разделять представление от логики, например, с помощью MVC паттерна. А еще лучше сразу учить ООП (например, можно взять несложный фреймворк и изучать его, yii, например). Маконелл будет сложноват для чайника, я бы посоветовал начать с М. Зандстры. Удачи!

Мелкий	Отправлено: 19 Мая, 2014 - 18:59:09
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	Bio man пишет: Маконелл будет сложноват для чайника А мне кажется в самый раз - когда говнокод ещё не вошёл в привычку (привычку ломать трудно), но написано достаточное количество кода, чтобы задуматься: говнокод ли это. Через год-два - перечитать ещё раз. Вот Фаулера возможно и рановато. ----- PostgreSQL DBA

EuGen	Отправлено: 19 Мая, 2014 - 19:11:22
Профессионал Покинул форум Сообщений всего: 9095 Дата рег-ции: Июнь 2007 Откуда: Berlin Помог: 707 раз(а)	Процедурный код. Спагетти. Плохой код. Потому что он его невозможно изолировать. А, значит, невозможно тестировать. Это - одна из причин, по которой одной из хороших практик считается объектный подход. Для объектного же подхода рекомендую ознакомиться с тем, что такое: * Dependency injection * Law of demeter * S.O.L.I.D. Для начала должно хватить. ----- Есть в мире две бесконечные вещи - это Вселенная и человеческая глупость. Но насчет первой .. я не уверен.

DelphinPRO	Отправлено: 20 Мая, 2014 - 09:58:18
Активный участник Покинул форум Сообщений всего: 7187 Дата рег-ции: Февр. 2012 Помог: 353 раз(а)	а я по coding standards сделаю замечание вводная статья - на вики а вот несколько стандартов для PHP - habrahabr[dot]ru/post/38214/ ----- Чем больше узнаю, тем больше я не знаю.

esterio	Отправлено: 20 Мая, 2014 - 12:07:53
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	ну есть же PSR-ы https://github[dot]com/dotzero/fig-s[dot][dot][dot]ster/accepted/ru здесь перевод на русский PSR-0, PSR-1, PSR-2

Поиск в теме | Версия для печати

Страниц (1): [1]

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Вопросы новичков »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.