Модераторы: skyboy, MoLeX, Aliance, ksnk
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> большой switch или маленький include, что лучше? 
V
    Опции темы
Mpak
Дата 19.4.2009, 13:03 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 445
Регистрация: 12.11.2006

Репутация: нет
Всего: 7
Не знаю к чему этот вопрос больше относится, думаю скорее к оптимизации.
какой вариант лучше? рассмотрим на простейших примерах "добавление новости":
1.
Код

switch ($_GET["page"])
 { case "news": $action      = "NewsAdd.php"; // обратим сюда внимание
                $button      = "Добавить новость";
                $autor       = $_COOKIE['user_name'];
                // ...
                include("NewsAddForm.php"); 
   break; 
 } 

и будет несколько мелких файлов для каждого типа page. Или же
2.
Код

$action = "add.php"; // !!
switch ($_GET["page"])
 { case "news": $button      = "Добавить новость";
                $autor       = $_COOKIE['user_name'];
                // ...
                include("NewsAddForm.php"); 
   break; 
 } 

и будет один большой файл с большим свич-кейсом?


Это сообщение отредактировал(а) Mpak - 19.4.2009, 13:21


--------------------
В любой откомпилированной программе есть, по крайней мере, одна ошибка...
P.S. А у меня их минимум две...
PM MAIL ICQ   Вверх
bars80080
Дата 19.4.2009, 21:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


прапор творюет
****
Награды: 1



Профиль
Группа: Завсегдатай
Сообщений: 12022
Регистрация: 5.12.2007
Где: Königsberg

Репутация: 71
Всего: 315
что-то нефига не понятно. опишите подробнее
PM MAIL WWW   Вверх
Djinn
Дата 19.4.2009, 23:33 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 250
Регистрация: 18.5.2007

Репутация: 1
Всего: 1
Действительно совсем не ясно, опишите задачу и проверте весь ли нужный код вы показали ? мне кажется вы что-то упустили!
PM MAIL   Вверх
Strilo4ka
Дата 20.4.2009, 01:47 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 97
Регистрация: 27.2.2009

Репутация: нет
Всего: нет
да не ясно что то....smile
PM ICQ   Вверх
Sanchezzz
  Дата 20.4.2009, 06:07 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1670
Регистрация: 19.11.2006
Где: Voronezh

Репутация: 41
Всего: 60
сделай следующие 

case "news": 
 include 'action/news.php';
break;
case "pages": 
 include 'action/pages.php';
break;

...

Это сообщение отредактировал(а) Sanchezzz - 20.4.2009, 06:10


--------------------
Понравился ответ "+" по репе, не забываем закрывать тему, заказы в LS.
PM MAIL Skype GTalk   Вверх
ksnk
Дата 20.4.2009, 08:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


прохожий
****


Профиль
Группа: Комодератор
Сообщений: 6855
Регистрация: 13.4.2007
Где: СПб

Репутация: 96
Всего: 386
А зачем вообще switch?
Почитать по MVC, и сделать как там предлагается...

Или вот,  на коленке смастерил:
Код

class application {
    $actions=array();
...
    function act($action){
        if (in_array($action,$this->action) 
             return call_user_function($this->action[$action]) ;
        else
             return error(404);
   }
}

class news {
   function init(&$application){
       $application-> actions['newsAdd']=array($this,'add_news');
      ...
   }
   ...
   function add_news (){
      ...
   }
}

...

$application= new application();
$news= new news(); $news->init($application);
...

$application->act($_GET['page']);



. Понятно, что класс application надо бы для кошерности снабдить гетами/сетами для action, но я для простоты и компактности многие ООП'ности сократил. 

Класс news, очевидно, удобнее выкинуть в отдельный файл и модифицировать по мере необходимости.

Это сообщение отредактировал(а) ksnk - 20.4.2009, 08:29


--------------------
Человеку свойственно ошибаться, программисту свойственно ошибаться профессионально ! user posted image
PM MAIL WWW Skype   Вверх
Mpak
Дата 20.4.2009, 10:02 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 445
Регистрация: 12.11.2006

Репутация: нет
Всего: 7
Приношу свои извинения, что дествительно не совсем понятно оформил топик =)
Но ksnk понял вопрос абсолютно верно, за что ему огромный + ;)

Моя работа далека от принципов ООП, надо действительно пересмотреть некоторые места...
Вопрос решен, воспользуюсь вариантом ksnk

P.S. ООП - великое изобретение человечества!  smile 


--------------------
В любой откомпилированной программе есть, по крайней мере, одна ошибка...
P.S. А у меня их минимум две...
PM MAIL ICQ   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "PHP"
Aliance
IZ@TOP
skyboy
SamDark
MoLeX

Новичкам:

  • PHP редакторы собираются и обсуждаются здесь
  • Электронные книги по PHP, документацию можно найти здесь
  • Интерпретатор PHP, полную документацию можно скачать на PHP.NET

Важно:

  • Не брезгуйте пользоваться тегами [code=php]КОД[/code] для повышения читабельности текста/кода.
  • Перед созданием новой темы воспользуйтесь поиском и загляните в FAQ
  • Действия модераторов можно обсудить здесь

Внимание:

  • Темы "ищу скрипт", "подскажите скрипт" и т.п. будут переноситься в форум "Web-технологии"
  • Темы с именами: "Срочно", "помогите", "не знаю как делать" будут УДАЛЯТЬСЯ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, IZ@TOP, skyboy, SamDark, MoLeX, awers.
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | PHP: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0501 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.