Новости Joomla

0 Пользователей и 1 Гость просматривают эту тему.
  • 17 Ответов
  • 1345 Просмотров
*

Vasisdas

  • Новичок
  • 5
  • 0 / 0
Добрый день!
Набрасываю небольшой плагин. Суть в том, что для запуска конкретного скрипта мне необходимо, чтобы были активны (или не активны) несколько параметров плагина.

Условие для запуска n-го скрипта, если активны 3 параметра такое:

Код
				if (($this->params->def('param1', '1'))&&($this->params->def('param2', '1'))&&($this->params->def('param3', '1'))) {

$document->addScript(JURI::base($pathonly = true). '/media/plg_blablabla/js/blablabla.js');

}
Вопрос простой: можно ли сократить эту запись?
СПАСИБО.
« Последнее редактирование: 22.12.2019, 23:25:50 от Vasisdas »
*

dmitry_stas

  • Легенда
  • 13151
  • 1234 / 8
не знаю как насчет "простой", но непонятный точно )) что такое "сократить эту запись"?
Тут дарят бакс просто за регистрацию! Успей получить!
Все советы на форуме раздаю бесплатно, то есть даром. Индивидуально бесплатно консультирую только по вопросам стоимости индивидуальных консультаций
*

passer

  • Завсегдатай
  • 1013
  • 75 / 3
Мне одному кажется, что так всегда true будет?
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Мне одному кажется, что так всегда true будет?
Есть такое
Смущает, что если параметр не определен он будет автоматом определен как int(1)

И зачем её сокращать - в данном виде условие вполне читаемо, если правильно всё остальное )))
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

ProtectYourSite

  • Живу я здесь
  • 2356
  • 135 / 4
  • Безопасность вебсайтов
И зачем её сокращать - в данном виде условие вполне читаемо, если правильно всё остальное )))
А если одинаковых условий 10, 100 или больше ? Не думаю, что читабельно будет, тут скорее речь идёт по поиск в массиве.
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
А если одинаковых условий 10, 100 или больше ? Не думаю, что читабельно будет, тут скорее речь идёт по поиск в массиве.
Ну тогда без вопросов, 2\3 условия еще ладно, но десяток - значит у приложения неправильная логика
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

Vasisdas

  • Новичок
  • 5
  • 0 / 0
Спасибо за ответы.

Да, когда я набросал несколько if для всех 3х вариантов параметров, то возникли трудности.

Чуть подробнее:
Есть 3 параметра: param1, param2, param3. Комбинации этих параметров:

все отключены (0 0 0)
1 0 0
1 1 0
1 1 1
0 1 0
0 1 1
0 0 1
1 0 1

В зависимости от выбранной комбинации запускается соответствующий скрипт.
Как правильно сформировать условие?
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Да, когда я набросал несколько if для всех 3х вариантов параметров, то возникли трудности
Значит у вас неправильная логика скрипта, что требует десяти разных условий, нужно копать глубже
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

dmitry_stas

  • Легенда
  • 13151
  • 1234 / 8
Код
$cond = $this->params->get('param1', '1'). $this->params->get('param2', '1'). $this->params->get('param3', '1');
if ($cond == '000') {
} else if ($cond == '001') {
...
}

ну или switch. или определить методы класса func000, func001 и т.д., и вызывать сразу

Код
$func = 'func'.$cond;
$this->$func()
Тут дарят бакс просто за регистрацию! Успей получить!
Все советы на форуме раздаю бесплатно, то есть даром. Индивидуально бесплатно консультирую только по вопросам стоимости индивидуальных консультаций
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
ну или switch. или определить методы класса func000, func001 и т.д., и вызывать сразу
Будет тот же зоопарк, если показ элемента требует проверки 10 условий - значит изначально неправильная логика, ну не должно быть такого
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

robert

  • Живу я здесь
  • 4974
  • 457 / 20
У вас по сути - бинарные числа. Сделайте массив с этими числами (или их десятичными значениями) в качестве ключей, а сами элементы - соответствующие скрипты (ну или наоборот).
Не будь паразитом, сделай что-нибудь самостоятельно!
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
У вас по сути - бинарные числа. Сделайте массив с этими числами (или их десятичными значениями) в качестве ключей, а сами элементы - соответствующие скрипты (ну или наоборот).
Хорошая идея, тогда можно работать через case и запускать скрипт по условию.
Но опять же мы не знаем ТЗ и вообще ничего ))
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

robert

  • Живу я здесь
  • 4974
  • 457 / 20
Никакие кейсы не нужны, есть же ключ.
Не будь паразитом, сделай что-нибудь самостоятельно!
*

passer

  • Завсегдатай
  • 1013
  • 75 / 3
Понятия не имею что там в $this->params. Исхожу из того, что там json. Если ассоциированный массив то декодировать не надо.
Код
$params = json_decode($this->params,1);
$scripts = [
        000=>'script1.js',
        100=>'script2.js',
        110=>'script3.js',
        011=>'script4.js',
        001=>'script5.js',
        101=>'script6.js',
        //......
    ];
$combs = (int)$params['param1'].(int)$params['param2'].(int)$params['param3'];
$scriptByCombination = $scripts[$combs];
*

beliyadm

  • Легенда
  • 9758
  • 1664 / 66
  • Севастополь, Россия
Понятия не имею что там в $this->params. Исхожу из того, что там json.
А если нет (потому что там его нет)?
Все истины, которые я хочу вам изложить, — бесстыдная ложь. Сделать всё хорошо
TLG: @Beliyadm
*

passer

  • Завсегдатай
  • 1013
  • 75 / 3
На этот случай уже подсказали.
Код
$cond = $this->params->get('param1',0). $this->params->get('param2',0). $this->params->get('param3', 0);
Логика то одна и та же. Реализация разная.
*

sivers

  • Живу я здесь
  • 2465
  • 336 / 0
Есть 3 параметра: param1, param2, param3. Комбинации этих параметров:
Код
$file = 'script_'.$param1.'-'.$param2.'-'.$param3.'.js';
$document->addScript(JURI::base($pathonly = true). '/media/plg_blablabla/js/'.$file);
Разместите скрипты с именами соответствующими возможным вариантам. И все. При желании можно добавить проверку существования файла, если не для всех вариантов созданы скрипты.
На связи в telegram @sivers
sivers @ inbox . ru
https://sivers.su/
*

Vasisdas

  • Новичок
  • 5
  • 0 / 0
Ребята, благодарю за активность.
Все варианты не пробовал, рабочим оказался вариант dmitry_stas - мне этого достаточно.

Плюсами отблагодарить не удалось - не хватает кол-ва сообщений. Поэтому просто СПАСИБО всем!

Названия топика поправил, возможно кому-то пригодится.
Чтобы оставить сообщение,
Вам необходимо Войти или Зарегистрироваться
 

Плагин для сброса параметров mod_jshopping_filters

Автор robert

Ответов: 5
Просмотров: 1843
Последний ответ 24.06.2018, 16:42:31
от kr.istina7