Как проверить значение в массиве если он пустой?

Question

[Макс В]

Имеется задача:
Написать функцию, которая принимает массив, а возвращает сумму его элементов. Если передан не массив, либо в нём нет элементов, либо некоторые элементы не являются числами, то должно возвращаться false.
Все прекрасно работает, но если оставить значение пустым, то все равно отображается 0, в чем проблема?

<?php
$my = [];
function sum_ar($arr)
{
        $sum = 0;
        foreach ($arr as $value)
        {
            if (is_array($arr) && (is_int($value) && !is_null($value)))
            {
                $sum += $value;
            }
            else
            {
                return false;
            }
        }
        return $sum;
 }
echo sum_ar($my);

Comments

[FanatPHP]

is_array внутри цикла - это глупость несусветная.
а в остальном норм. как ниже написали, только добавить проверку на пустой массив

Answer 1

[Vitsliputsli]

У вас нет проверки на пустой массив, поэтому возвращает 0. Не хватает проверки на float, на null уже нет смысла проверять, а проверять на массив не нужно при каждой итерации.
Т.е. что-то вроде этого должно быть:

function sum($arr)
{
    if (!is_array($arr) || count($arr)===0) {
        return false;
    }
    $sum = 0;
    foreach ($arr as $value) {
        if (!is_int($value) && !is_float($value)) {
            return false;
        }
        $sum += $value;
    }
    return $sum;
}

Comments

[Макс В]

Благодарю, то что нужно!

[Vitsliputsli]

Лентюй, кстати да,
Макс В, если вдруг под числом понимается не тип элемента, а его содержание, то нужно использовать is_numeric.

Answer 2

[Lander]

Ну добавьте условие на !empty($arr)ещё