Перейти до вмісту
Пошук в
  • Детальніше...
Шукати результати, які ...
Шукати результати в ...

На сколько этот говнокод плох?


Recommended Posts

Привет!

 

Оцените, пожалуйста, насколько плох такой код:

<select name="category_id" class="form-control">
<?php
$arrayss = [
    0 => "Все категории",
    "5" => "Категория 1",
    "3" => "Категория 2",
    "14" => "Категория 3",
];
?>
<option value="0">По всем разделам</option>
            
<?php foreach ($arrayss as $key => $value ) { ?>        
            
<?php if ($key == $category_id) { ?>
<option value="<?php echo $key; ?>" selected="selected"><?php echo $value; ?></option>
<?php } else { ?>
<option value="<?php echo $key; ?>"><?php echo $value; ?></option>
<?php } ?>
<?php } ?>

</select>

 

Данный код написал в файл search.tpl... option value соответствует реальной категории на сайте. 

 

Написал это для того, чтобы исключить некоторые категории из выбора для поиска, а также переименовать так, как мне надо. Проверил - всё работает... категории выбираются, ищется по выбранной категории.

 

Интересует такой момент - насколько это всё ужасно? Или более-менее пойдёт?

 

Не хотелось лезть в модели, чтобы исключить из выборок ненужные мне категории..

Змінено користувачем Skunk
[code]
Надіслати
Поділитися на інших сайтах

офигенскй костыль. тем более, что работает :)

зачем лезть в модель, чтобы исключить что-то из поиска? пусть она все выдаст, а контроллер сортирует, как угодно. 

на нагрузку это повлияет аж никак.

  • +1 1
Надіслати
Поділитися на інших сайтах


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

  • +1 1
Надіслати
Поділитися на інших сайтах

16 минут назад, Otvet сказал:

правильнее делать unset в контроллере

 

да, думал и так сделать - unset_ом исключить категории  из массива, но мне надо было ещё и переименовать эти категории. Подумал, мол, вдруг это ещё что-то заденет.

Надіслати
Поділитися на інших сайтах

Добрый день, возможно не в тему, но что то похоже. Только осваиваю opencart, установил версию 2.3.0.2.1. Выдает ошибку на только что развёрнутом сайте. Установил версию старее 2.1.0.2.1 - всё нормально.

 

Вопрос: Старая версия уязвима по безопасности, и вообще какие сюрпризы ждут, если поставлю сайт на 2.1.0.2.1?

 

Спасибо, извините, если не в тему.

 

Надіслати
Поділитися на інших сайтах


Супер костыль - так точно не делают.

 

"Если в доме есть грабли - рано или поздно на них наступишь" (законы "жизни" Марка)

 

1. А если измените ID категорий ;) (даже случайно это бывает, по прошествии времени когда забываешь что у тебя "грабли" в коде)

2. В tpl используют языковые переменные а не пишут "кириллицей" 

  • +1 2
Надіслати
Поділитися на інших сайтах

Створіть аккаунт або увійдіть для коментування

Ви повинні бути користувачем, щоб залишити коментар

Створити обліковий запис

Зареєструйтеся для отримання облікового запису. Це просто!

Зареєструвати аккаунт

Вхід

Уже зареєстровані? Увійдіть тут.

Вхід зараз
  • Зараз на сторінці   0 користувачів

    • Ні користувачів, які переглядиють цю сторінку
×
×
  • Створити...

Important Information

На нашому сайті використовуються файли cookie і відбувається обробка деяких персональних даних користувачів, щоб поліпшити користувальницький інтерфейс. Щоб дізнатися для чого і які персональні дані ми обробляємо перейдіть за посиланням . Якщо Ви натиснете «Я даю згоду», це означає, що Ви розумієте і приймаєте всі умови, зазначені в цьому Повідомленні про конфіденційність.