Форум программистов
 

Восстановите пароль или Зарегистрируйтесь на форуме, о проблемах и с заказом рекламы пишите сюда - alarforum@yandex.ru, проверяйте папку спам!

Вернуться   Форум программистов > IT форум > Помощь студентам
Регистрация

Восстановить пароль

Купить рекламу на форуме - 42 тыс руб за месяц

Ответ
 
Опции темы Поиск в этой теме
Старый 02.09.2011, 11:34   #1
Излом
Новичок
Джуниор
 
Регистрация: 02.09.2011
Сообщений: 2
По умолчанию Правильно ли организован код?

Взялся за написание сайта. Написал пару страниц и сценарии к ним. Вроде все работает нормально, но код какойто громоздкий получается. Вы не могли ьы посмотреть и посоветовать, в чем надо исправляться.

Заранее спасибо.

P.S. Только, пожауйста, не надо ответов типа: читай книги и т.д. Укажите конкретно, что не так
Вложения
Тип файла: rar www.rar (12.2 Кб, 10 просмотров)
Излом вне форума Ответить с цитированием
Старый 02.09.2011, 11:54   #2
Jakethefish
Форумчанин
 
Регистрация: 13.11.2009
Сообщений: 121
По умолчанию

По размеру кода ничего не скажу, но могу посоветовать как оптимизировать.
Если вы делаете сайт для себя и друзей, то предлагаю забыть про всякие MVC и ORM фреймворки, хотя бы на первых порах, больше времени потратите, если же сайт на продажу, то только туда и дорога. Хотя кто-то сейчас кинет камень и скажет - без фреймворков и архитектуры - говнокод.
Могу предложить правда поставить движок CMS, наподобие Wordpress или cmsms. Но там вам кодить не придется по большому счету.

Если же вы хотите "Сами с усами", то
1. сделайте класс для мускула. Поищите тут или тут или напишите свой.
2. Для всех всех запросов сделайте ещё одни класс или вставьте в класс номер 1 все ваши запросы. В идеале вы должна получить что-то такое:
PHP код:
<?php 
$db 
= new MySqlClass();
$articles = array();
$articles $db->getAllArticles(); ?>
//Где-то сверху

<?php foreach($articles as $article): ?>
    <td>$article['name']</td>
    <td>$article['content']</td>
<?php endforeach; ?>
3. предлагаю заменить if(){<>} на if():<>endif; ,также форич и другие конструкции, но это уже на самом деле дело привычки для большей читабельности кода.
4. делайте вставки пхп в хтмл а не наоборот. Т.Е.:

не
PHP код:
 echo "<ul class='$var'>$text</ul>"
а
PHP код:
<ul class="<?php echo $var?>"> <?php echo $text ?></ul>
ИМХО.
Jakethefish вне форума Ответить с цитированием
Старый 02.09.2011, 12:21   #3
Излом
Новичок
Джуниор
 
Регистрация: 02.09.2011
Сообщений: 2
По умолчанию

Спасибо за замечания.

Вот только я не понял, что вот это:
Цитата:
Сообщение от Jakethefish Посмотреть сообщение
предлагаю забыть про всякие MVC и ORM фреймворки
Излом вне форума Ответить с цитированием
Старый 02.09.2011, 12:27   #4
Jakethefish
Форумчанин
 
Регистрация: 13.11.2009
Сообщений: 121
По умолчанию

MVC - Model View Controller. Подход к написанию кода, при котором разделяется представление(View), данные(Model), управляющие функции(Controller). Используется для того что бы можно было поменять допустим хтмл, без ущерба для основной логики программы.
ORM - Object-relational mapping. Фреймворки которые написаны с целью упрощения работы с БД, при которой вы уже можете не писать запрос напрямую, а использовать средства самого фреймворка для загрузки данных. Самый простой бонус от применения: вы можете изменить движок БД и скрипт будет продолжать также работать.

Как-то так. Подробнее в гугле.
Jakethefish вне форума Ответить с цитированием
Ответ


Купить рекламу на форуме - 42 тыс руб за месяц



Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Правильно написать код zoom711 Помощь студентам 1 24.02.2011 12:21
Как правильно оформить код? NiKiToZZ- Помощь студентам 5 28.12.2010 12:26
C++,есть код,как правильно создать очередь? Aleksandr_Yanov Помощь студентам 0 19.08.2010 14:22
Есть код но почему то не правильно работает -ushёl- Помощь студентам 1 10.05.2010 22:27