Задать вопрос
mixtape774
@mixtape774
Занимаюсь Web, Desktop разработкой и реверсингом
php

Можно ли так сократить код PHP if (!empty($product_info) && $product_info['status'] == 1) ?

Обнаружил такой код у заказчика... не понимаю почему так написано:
if (!empty($product_info))
			{
				if ($product_info['status'] == 1)
				{					
					$this->data['product_name'] = $product_info['name'];
					$this->data['product_image'] = $product_info['image'];
					$this->data['product_price'] = $product_info['price'];
									
					if (isset($this->request->get['count']))
						$this->data['product_count'] = $this->request->get['count'];
				}
				else
					$this->data['isWrongProduct'] = true;
			}
			else
				$this->data['isWrongProduct'] = true;


Можно ли сократить код до: 
if (!empty($product_info) && $product_info['status'] == 1)
			{
					$this->data['product_name'] = $product_info['name'];
					$this->data['product_image'] = $product_info['image'];
					$this->data['product_price'] = $product_info['price'];
									
					if (isset($this->request->get['count']))
						$this->data['product_count'] = $this->request->get['count'];
			}
			else
				$this->data['isWrongProduct'] = true;

или могут возникнуть ошибки undefined?
  • Вопрос задан
  • 2719 просмотров
1 комментарий
Подписаться 2 Оценить 1 комментарий
Решения вопроса 1
sim3x
@sim3x 
if (
// есть ли переменная; если не проверить кинет notice
    isset($product_info) &&  

// есть ли в массиве; если не проверить кинет notice
    isset($product_info['status']) &&  

// во избежании выпадение волос -- используй сравнение без приведения типов    
   $product_info['status'] === 1)


Действует аналогично плюсам

Есть and, со своими тараканами
stackoverflow.com/questions/2803321/and-vs-as-oper...
Ответ написан
1 комментарий
1 комментарий
Пригласить эксперта
Ответы на вопрос 2
@Grag
По логике можно.
Ответ написан
Комментировать
Комментировать
miraage
@miraage
Старый прогер
Еще более короткая запись.
В isset можно передавать любое условие с любым количество вложенностей.
Так же можно передавать несколько переменных, и вернет true, только если все отличны от null.

if (isset($product_info['status']) && $product_info['status'] === 1)
Ответ написан
Комментировать
Комментировать
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы
Вакансии с Хабр Карьеры
PHP Developer
YCLIENTS • Москва
от 200 000 до 350 000 ₽
PHP разработчик
Ведисофт • Екатеринбург
от 25 000 ₽
Midlle PHP developer (backend)
ИТЦ Аусферр • Магнитогорск
от 100 000 до 160 000 ₽
Ещё вакансии
Заказы с Хабр Фриланса
Ошибка в 1с при выгрузке товаров на сайт на Битриксе
25 апр. 2024, в 03:08
3000 руб./за проект
Создание и публикация короткого (reels) UGC-видео на YouTube-канале
25 апр. 2024, в 01:46
500 руб./за проект
Настроить перехват https-трафика для android-приложения
25 апр. 2024, в 01:02
10000 руб./за проект
Ещё заказы