Как переписать код в более простой и понятный вид?

Question

[Павел]

есть такой отрывок кода

foreach ($result as $i) {
                    if ($i['name'] == $name && $i['date'] == $day && print("<td>{$i['sub_hours']}</td>"))
                        continue 2;
                }

не пойму как переписать код внутри if чтоб он был "по обычному" написан ))

Comments

[Дмитрий Гординский]

Для чего вы засовывает print в условие?
Вы точно понимаете разницу между логическими операторами
'AND' и '&&'?

[Павел]

DmitriyGordinskiy, это не я делал и именно этот код мне не ясен, но он работает как надо, хотелось бы переписать чтоб был понятен, поэтому и спрашиваю у специалистов

Answer 1

[Ильдар Сарибжанов]

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

foreach ($any as $result) {
    foreach ($result as $i) {
        if (isCheck($i, $name, $day)) {
            print("<td>{$i['sub_hours']}</td>");

            continue 2;
        }
    }
}

/**
 * Проверка условия 
 * 
 * @param $itm
 * @param $name
 * @param $day
 *
 * @return bool
 */
function isCheck($itm, $name, $day)
{
    if ($itm['name'] != $name) {
        return false;
    }

    return ($itm['date'] == $day);
}

Answer 2

[Дмитрий Гординский]

Как переписать код в более простой и понятный вид?

Не использовать функции вывода или функции меняющие состояние внутри выражения условия.

И уж тем более не делать это в условиях с 'ленивыми' логическими операторами.

Comments

[Павел]

так, а мне делать то что?)

[Дмитрий Гординский]

Павел, вынести вывод из выражения в тело.

[Павел]

DmitriyGordinskiy, я не понимаю код который тут написан, поэтому прошу помощи,
если вынесу
&& print("<td>{$i['sub_hours']}</td>")
в тело условия {}
то код перестает работать

[FanatPHP]

Павел, а && ты тоже оставляешь?

[Павел]

FanatPHP, нет, это же И , что оно там будет делать )) выдало бы ошибку, но ошибок нет, а код не работает, где то логика ломается

[FanatPHP]

а ты небось под continue перенес?

[Павел]

FanatPHP, ну нет я не настолько новичок,
после continue уже никакой код не выполнится

[FanatPHP]

Ну тогда отлаживайся дальше

[Павел]

FanatPHP, ок будем искать с перламутровыми ))
спасибо