Форум PHP программистов > Помогите спростить код

[ Поиск ] - [ Пользователи ] - [ Календарь ]

BzenkoSergey

5.10.2009 - 23:08

Доброе время суток, помогите пожалуйста спростить код:

PHP

<?php
$n_kiev = "/12/index.php?do=cat&category=kiev";
$n_kiev_club = "/12/index.php?do=cat&category=kiev-club";
$n_kiev_magazin = "/12/index.php?do=cat&category=kiev-magazin";
$n_kiev_banki = "/12/index.php?do=cat&category=kiev-banki";
$n_kiev_restorani = "/12/index.php?do=cat&category=kiev-restorani";
$n_kiev_oteli = "/12/index.php?do=cat&category=kiev-oteli";

$n_kiev_u = $_SERVER['REQUEST_URI'];
if($n_kiev == $n_kiev_u or $n_kiev_club == $n_kiev_u or $n_kiev_magazin == $n_kiev_u or $n_kiev_banki == $n_kiev_u or $n_kiev_restorani == $n_kiev_u or $n_kiev_oteli == $n_kiev_u) 
  {
  echo "<li><a href='index.php?do=cat&category=kiev-club' class='menuClas'>Клубы</a></li><li><a href='index.php?do=cat&category=kiev-magazin' class='menuClas'>Магазины</a></li><li><a href='index.php?do=cat&category=kiev-banki' class='menuClas'>Банки</a></li><li><a href='index.php?do=cat&category=kiev-restorani' class='menuClas'>Рестораны</a></li><li><a href='index.php?do=cat&category=kiev-oteli' class='menuClas'>Отели</a></li><li><a class=menuClass2 onclick=\"show('test');\" href=\"#\"><img src='{THEME}/images/plus.gif' title='&nbsp;показать ещё&nbsp;'></a></li>";
  }

Спустя 5 минут, 11 секунд (5.10.2009 - 22:13) vasa_c написал(а):

Чего сделать?

Спустя 3 минуты, 48 секунд (5.10.2009 - 22:17) UnWind написал(а):

BzenkoSergey Опиши подробнее то, что нужно сделать пожалуйста. А то я мало понимаю, чего ты хочешь из слова "Спростить".

Спустя 4 минуты, 54 секунды (5.10.2009 - 22:22) sergeiss написал(а):

BzenkoSergey - тебе нужно срочно разобраться с массивом $_GET и использовать его! Этот массив содержит данные, передаваемые в URI после знака вопроса. В виде ассоциативного массива.

Спустя 9 минут, 48 секунд (5.10.2009 - 22:32) twin написал(а):

Ну если только спростить...

PHP

$url = array(
                "/12/index.php?do=cat&category=kiev",
                "/12/index.php?do=cat&category=kiev-club",
                "/12/index.php?do=cat&category=kiev-magazin",
                "/12/index.php?do=cat&category=kiev-banki",
                "/12/index.php?do=cat&category=kiev-restorani",
                "/12/index.php?do=cat&category=kiev-oteli",
             );
$n_kiev_u = $_SERVER['REQUEST_URI'];
    if(in_array($n_kiev_u, $url))
    {
    .
    .

А вообще прислушайся к совету sergeiss'a. Он дело речет.

Спустя 6 минут, 19 секунд (5.10.2009 - 22:38) BzenkoSergey написал(а):

Если выбрана категория к примеру киев ссылка получаеться
"/12/index.php?do=cat&category=kiev"
в категории есть под категории клуб к ссылке добавляеться приставка "-club"
"/12/index.php?do=cat&category=kiev-club"

мне нужно если посетитель выбрал категорию киев или любую из его подкатегорий вывести определённый код

PHP

<li><a href='index.php?do=cat&category=kiev-club' class='menuClas'>Клубы</a></li><li><a href='index.php?do=cat&category=kiev-magazin' class='menuClas'>Магазины</a></li><li><a href='index.php?do=cat&category=kiev-banki' class='menuClas'>Банки</a></li><li><a href='index.php?do=cat&category=kiev-restorani' class='menuClas'>Рестораны</a></li><li><a href='index.php?do=cat&category=kiev-oteli' class='menuClas'>Отели</a></li><li><a class=menuClass2 onclick="show('test');\" href=\"#\

Я только начинаю учиться php и етот код сам написал как видете довольно приметивно, если у меня не одна категория киев с её подкатегориями то оч много кода получаеться. Помогите упростить код, оптимизировать.

Спустя 6 минут, 34 секунды (5.10.2009 - 22:45) sergeiss написал(а):

Цитата (BzenkoSergey @ 5.10.2009 - 23:38)

Помогите упростить код, оптимизировать.

Так вот я тебе и говорю: почитай про массив $_GET, в который попадают все данные, которые были переданы в адресной строке после знака вопроса.

Тогда ты сможешь сравнивать не весь URI, а его отдельные части. Поверь - этим массивом пользуются все программеры ПХП

Например, при входе с такими параметрами

Цитата (BzenkoSergey @ 5.10.2009 - 23:08)

"/12/index.php?do=cat&category=kiev"

массив $_GET (глобальный, который будет составлен без твоего участия) будет состоять из следующих элементов:
$_GET['do']='cat'
$_GET['category']='kiev'
Ты просто берешь эти элементы и работаешь с ними.

Далее ты можешь сравнивать эти части с чем-то, не таская весь URI за собой.

Ну, и совсем далее, когда сделаешь вышесказанное, то можно будет упрощать вывод данных

Там тоже есть что оптимизировать, но лучше делать всё по очереди.

Спустя 1 час, 3 минуты, 9 секунд (5.10.2009 - 23:48) BzenkoSergey написал(а):

Как писать код чтобы

PHP

$_GET['category']=

само определило чему оно ровно на даной странице?

Спустя 5 минут, 45 секунд (5.10.2009 - 23:54) sergeiss написал(а):

"Почти" так и написать

Только тебе лично не надо ничего определять. Этот массив создается автоматически. На основании данных из URI.
Ты должен сделать что-то типа

PHP

$category=isset($_GET['category']) ? $_GET['category'] : false;

Спустя 23 минуты, 46 секунд (6.10.2009 - 00:17) BzenkoSergey написал(а):

Цитата (sergeiss @ 5.10.2009 - 20:54)

"Почти" так и написать

PHP

$category=isset($_GET['category']) ? $_GET['category'] : false;

Огромное спасибо тебе за помощь, сделал так

PHP

$category=isset($_GET['category']) ? $_GET['category'] : false;
if($category == "kiev") 
  {
  echo "rthrytuiu";
  }

1-н вопрос:
Что дописать к "kiev" чтобы его считывало как "kiev" и любые символы если они есть?

Спустя 10 часов, 42 минуты, 51 секунда (6.10.2009 - 11:00) BzenkoSergey написал(а):

Всё сделал спасибо всем за помощь!

Быстрый ответ:

Здесь расположена полная версия этой страницы.