Как передать массив через функцию?

Question

[Александр Сисюкин]

Помогите с функцией

function file_error($avtor, $file)
{
    $error = array();
    if (empty($avtor))
    {
        $error[] = "введите имя автора!";
        
    }
    if (empty($file))
    {
        $error[] = "Выберите фаил";
        
    }
    return $error ;


}

Пытаясь вывести результаты массива

$oshibka = file_error($_POST['avtor'], $_FILES['uploadfile']['name']);
   if (count($oshibka)== 0){
        
        
        upload();
        bd_insert($_FILES['uploadfile']['name'], $_POST['avtor']);
        } else{
             print "<b>При загрузке произошли следующие ошибки:</b><br>";
        foreach($error AS $err)
        {
            print $err."<br>";
        }
        }

получаю ошибку категории
Warning: Invalid argument supplied for foreach()

Не судите строго только начал изучать php.

Answer 1

[Александр Сисюкин]

решилил проблему вот так:

function file_error($avtor, $file)
{
    
    if (empty($avtor))
    {
        $error1 = "введите имя автора!";
        
    }
    if (empty($file))
    {
        $error2 = "Выберите фаил";
        
    }
    return $error = array($error1,$error2);


}

но все же как передать массив который не надо прописывать вручную return $error = array($error1,$error2); с автоматическим добавлением типа $error[] = "текст ошибки";??

Comments

[Silm]

То что вы написали, не решило бы вашу ошибку. В функции file_error() все было ок, как вам и написали у вас 2 переменные, в одну получили результат, во второй его искали.

Замените в первой и второй строке переменную $oshibka на $error и все будет ок.

Answer 2

[Алексей Уколов]

Вы уж определитесь $oshibka у вас или $error. Пишете в одну переменную, читаете из другой - вот и не работает.

Ну и немного здравого смысла:

• Конечно, не нужно использовать транслитерацию, пишите обычным английским языком.
  
• Если у вас массив ошибок, то и переменная должна называться во множественном числе.
  
• Отформатируйте код и пользуйтесь IDE, это уменьшит количество глупых ошибок.
  

Comments

[Александр Сисюкин]

спасибо) учту ваши советы.

Answer 3

[Uwe_Boll]

данная ошибка говорит что в цикл foreach() передан НЕ массив
что бы решить проблему вам нужно изменить код вот так

function file_error($avtor, $file){
    $error = array();
    if (empty($avtor)){
      return  $error[] = "введите имя автора!";
    }
    if (empty($file)){
       return $error[] = "Выберите фаил";
    } 
}

$error= file_error($_POST['avtor'], $_FILES['uploadfile']['name']);
   if (count($oshibka)== 0){
        upload();
        bd_insert($_FILES['uploadfile']['name'], $_POST['avtor']);
        } else{
             print "<b>При загрузке произошли следующие ошибки:</b><br>";
        foreach($error AS $err) {
            print $err."<br>";
        }
   }

Comments

[red_led]

Там не из за этого ошибка. Посмотрите внимательно на вызов функции и цикл.

[Александр Сисюкин]

Заменил код так как вы посоветовали. результат тот же(

[red_led]

А функцию file_error не надо было вообще трогать. Вы изменили её смысл.