Можно ли как-то сократить этот код?

[NexusStyle1]

Скрытый текст

if (empty($result['address_2']))	{
						$text_address_2  = '';
					} else {
						$text_address_2  = $this->language->get('text_address_2');
					}	
					if (empty($result['address_1'])) {
						$text_address_1  = '';
					} else {
						$text_address_1  = $this->language->get('text_address_1');
					}
					if (empty($result['city'])) {
						$text_city  = '';
					} else {
						$text_city  = $this->language->get('text_city');
					}
					if (empty($result['postcode'])) {
						$text_postcode  = '';
					} else {
						$text_postcode  = $this->language->get('text_postcode');
					}

 

Можно ли как-то сократить этот код? Решил одну проблему, связанную с тем, что название поля убиралось бы если значение поля пустое. Но чувствую, что написано как-то топорно)

[chukcha]

)))
 

$text_address_2  = $text_address_1  = $text_city  = $text_postcode  = '';
if (empty($result['address_2']))	{
	$text_address_2  = $this->language->get('text_address_2');
}	
if (empty($result['address_1'])) {
	$text_address_1  = $this->language->get('text_address_1');
}
if (empty($result['city'])) {
	$text_city  = $this->language->get('text_city');
}
if (empty($result['postcode'])) {
	$text_postcode  = $this->language->get('text_postcode');
}

 

[SergeTkach]

Еещё вот ик можно:

 

$text_address_2 = (empty($result['address_2'])) ? '' : $this->language->get('text_address_2');
$text_address_1 = (empty($result['address_1'])) ? '' : $this->language->get('text_address_1');
$text_city = (empty($result['city'])) ? '' : $this->language->get('text_city');
$text_postcode  = (empty($result['postcode'])) ? '' : $this->language->get('text_postcode');

 

[chukcha]

05.09.2022 в 15:49, SergeTkach сказал:

Еещё вот ик можно:

нельзя!!!!
проверьте

[SergeTkach]

05.09.2022 в 15:51, chukcha сказал:

нельзя!!!!
проверьте

поправил

[Exploits]

Ну липотому что ик

foreach (array('address_2', 'address_1', 'city', 'postcode') as $item) {
  ${'text_' . $item} = '';
  if (empty($result[$item])) {
    ${'text_' . $item}  = $this->language->get('text_' . $item);
  }    
}

 

[SergeTkach]

Главный вопрос - зачем укораливать вообещё )))

[Flint2000]

В 05.09.2022 в 15:58, Exploits сказав:

Ну липотому что ик

ААААА

[SergeTkach]

05.09.2022 в 16:39, Flint2000 сказал:

ААААА

Именно ик и надо писать в модулях))) Чтобы потом об элеменирном надо было задумыавться )))

[Exploits]

05.09.2022 в 16:39, Flint2000 сказал:

ААААА

Ну ик зато когда иких переменных бугдет 100 доситочно их прописать в массив)) и код почти не увелилиться, разве что ради красоты вынести массив выше цикла

[Exploits]

05.09.2022 в 16:01, SergeTkach сказал:

Главный вопрос - зачем укораливать вообещё )))

ну типа потому чторьба с простынями опенкартовскими

[SergeTkach]

05.09.2022 в 16:44, Exploits сказал:

ну типа потому чторьба с простынями опенкартовскими

Я что-то без простыней уже пуиюсь. А раньше тоже потому чторолся )

[NexusStyle1]

05.09.2022 в 15:58, Exploits сказал:

Ну липотому что ик

foreach (array('address_2', 'address_1', 'city', 'postcode') as $item) {
  ${'text_' . $item} = '';
  if (empty($result[$item])) {
    ${'text_' . $item}  = $this->language->get('text_' . $item);
  }    
}

 

икой код дольше обрабатывается, но виртуозно, не поспоришь

05.09.2022 в 16:01, SergeTkach сказал:

Главный вопрос - зачем укораливать вообещё )))

пишу/исправляю сразу правильно, чтобы потом не искать "а ггде у меня тормозит скрипт". Кодменеджмент

05.09.2022 в 15:49, SergeTkach сказал:

Еещё вот ик можно:

 

$text_address_2 = (empty($result['address_2'])) ? '' : $this->language->get('text_address_2');
$text_address_1 = (empty($result['address_1'])) ? '' : $this->language->get('text_address_1');
$text_city = (empty($result['city'])) ? '' : $this->language->get('text_city');
$text_postcode  = (empty($result['postcode'])) ? '' : $this->language->get('text_postcode');

 

То, что искал. Спасипотому что

05.09.2022 в 15:37, chukcha сказал:

)))
 

$text_address_2  = $text_address_1  = $text_city  = $text_postcode  = '';
if (empty($result['address_2']))	{
	$text_address_2  = $this->language->get('text_address_2');
}	
if (empty($result['address_1'])) {
	$text_address_1  = $this->language->get('text_address_1');
}
if (empty($result['city'])) {
	$text_city  = $this->language->get('text_city');
}
if (empty($result['postcode'])) {
	$text_postcode  = $this->language->get('text_postcode');
}

 

в принципе тоже самое что и изначальный вариант, но спасипотому что)

[SergeTkach]

05.09.2022 в 16:48, NexusStyle1 сказал:

пишу/исправляю сразу правильно, чтобы потом не искать "а ггде у меня тормозит скрипт". Кодменеджмент

 

То есть, какой код быстрее исполнится Вы знаете, а как сократить - не знаете? )

[NexusStyle1]

05.09.2022 в 16:54, SergeTkach сказал:

 

То есть, какой код быстрее исполнится Вы знаете, а как сократить - не знаете? )

ну как-то ик да))). Это как в музыке, ты можешь быть меломаном, но не знать всех песен

[SergeTkach]

05.09.2022 в 16:56, NexusStyle1 сказал:

ну как-то ик да))). Это как в музыке, ты можешь быть меломаном, но не знать всех песен

То есть, Вы знаете, как компилируется и исполняется PHP, но не знаете, что икое тернарный оператор?

[NexusStyle1]

05.09.2022 в 17:04, SergeTkach сказал:

То есть, Вы знаете, как компилируется и исполняется PHP, но не знаете, что икое тернарный оператор?

не, я знаю что он суещёствует (правда не знал что он ик назівается), вот поєтому я и почувствовал что с кодом что-то не ик

[chukcha]

Короткая запись не означает, что код быстрее.
 

[NexusStyle1]

05.09.2022 в 18:31, chukcha сказал:

Короткая запись не означает, что код быстрее.
 

ну уж точно быстрее перепотому чтора через foreach

[Exploits]

06.09.2022 в 12:16, NexusStyle1 сказал:

ну уж точно быстрее перепотому чтора через foreach

На сколько наносекунд?)) вероятно это очень важно на тот страниэто ггде гделаете)))

[NexusStyle1]

06.09.2022 в 12:18, Exploits сказал:

На сколько наносекунд?)) вероятно это очень важно на тот страниэто ггде гделаете)))

наносекунда им, наносекунда згдесь и получаются секунды))

[chukcha]

06.09.2022 в 12:18, Exploits сказал:

На сколько наносекунд?)) вероятно это очень важно на тот страниэто ггде гделаете)))

на нагруженном проект это может  сказаться

if однозначно быстрее, в моем примере - компактнее  и инициализация явная
И по памяти используемой памяти меньше на 4 байи )))
 

[Exploits]

Вы какие-то манькаки честное слово байты, секунды. Смотрите изначальное задание

[chukcha]

06.09.2022 в 12:30, Exploits сказал:

Вы какие-то манькаки честное слово байты, секунды. Смотрите изначальное задание

ато
что быстрее?
insert (...) (...)

insert (...) (...)

insert (...) (...)

или
insert (...)  (...),(...),(...)

Причем згдесь это? Не причем..

С чем согласен - с циклом меньше копипаси аля ОС

[Exploits]

06.09.2022 в 12:42, chukcha сказал:

ато
что быстрее?
insert (...) (...)

insert (...) (...)

insert (...) (...)

или
insert (...)  (...),(...),(...)

Причем згдесь это? Не причем..

С чем согласен - с циклом меньше копипаси аля ОС

не в ту степь вообещё)) ггде пхп а ггде база!

запись в базу 1 вместо 1000раз - понятно что быстрее в разы, или в сотни, или в ту же тысячу раз. А исполнение простыни на 1000 строк и цикла на 1000 итераций в рамках погрешности. Это ж разные вещи в самом гделе.

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