Не выводится значение переменной в <option>

[max74max]

Здравствуйте! Помогите, пожалуйста разобраться.
Есть функция, которая ищет в директории файлы, которые заканчиваются .txt и формирует их в выпадающее меню. Здесь всё работает как нужно. Но сейчас задача, добавить в <option значение selected, если переменная $file_name равна переменной $file

Проблема заключается в том, что переменная $selected которая находится в <option всегда пустая. Не потому что данного файла нет в директории, а потому что любая переменная после <option будет пустой. В общем, переменную $selected в <option что-то блокирует или очищает значение. Но я не могу разобраться что именно. Прошу помочь

PHP код:

$file_name = "name_of_file";

function my_function(){
$exclude = array("txt");
foreach (new DirectoryIterator('my_folder/') as $file) {
  if ($file->isFile()) {
    $extension = pathinfo($file, PATHINFO_EXTENSION);
    if(in_array($extension, $exclude)) {
        if(md5($file_name) == md5($file)){$selected = 'selected';}
        echo "
    <option ".$selected ." value='".md5($file)."'>$file</option>";
            }
        }
    }
} 


[macomics]

А разве md5 надо считать не для basename? Иначе $file_name должна быть равна

$file_name = "/path_to_file/name_of_file";

И еще. Зачем в цикле пересчитывать кучу раз константное значение md5($file_name) и по 3 раза считать md5($file). Стоит объявить переменные и сохранить результат работы функции в них, чтобы избавиться от повторных вычислений константных значений.

Для того, чтобы понять почему $selected не устанавливается, достаточно просто вывести в option значение md5($file_name) хотя бы в виде комментария, чтобы иметь возможность визуально сравнить значения в value и понять, что нет совпадений.

PHP код:

$file_name = "name_of_file";
function my_function(){
    $md5file_name = md5($file_name);
    $exclude = array("txt");
    foreach (new DirectoryIterator('my_folder/') as $file) {
        if ($file->isFile()) {
            $extension = pathinfo($file, PATHINFO_EXTENSION);
            if(in_array($extension, $exclude)) {
                $md5file = md5($file);
                if($md5file_name == $md5file){
                    $selected = ' selected';
                } else {
                    $selected = '';
                }
                echo "<!--$md5file_name  == $md5file-->
<option$selected value='$md5file'>$file</option>";
            }
        }
    }
} 


[max74max]

Спасибо, большое. Это то что нужно.
Но только есть один нюанс.
Почему-то переменная $file_name = "name_of_file"; прописанная до function my_function() пустая, а если прописать её после function my_function() то всё ок. Почему так происходит и как задать значение переменной до функции?

[ADSoft]

почитайте про видимость переменных... функция по сути отдельная программа и что было вне её не знает - да и не должна.

Для передачи в функцию тех или иных значений используются параметры... в этом как бы и заключается суть функций.... передал в функцию параметр(ы) = получил результат(ы)

в вашем случае -

function  my_function($file_name){..}

а вызов функции с нужным параметром

my_function('name_of_file1');
...
my_function('name_of_file2');

[macomics]

еще можно сделать так

PHP код:

$file_name = "name_of_file";
function my_function(){
    global $file_name;
    $md5file_name = md5($file_name);
    $exclude = array("txt");
    foreach (new DirectoryIterator('my_folder/') as $file) {
        if ($file->isFile()) {
            $extension = pathinfo($file, PATHINFO_EXTENSION);
            if(in_array($extension, $exclude)) {
                $md5file = md5($file);
                if($md5file_name == $md5file){
                    $selected = ' selected';
                } else {
                    $selected = '';
                }
                echo "<!--$md5file_name  == $md5file-->
<option$selected value='$md5file'>$file</option>";
            }
        }
    }
} 


[ADSoft]

можно - но не нужно учить плохому... глобальные переменный в целом - зло

[macomics]

И что в них злого? Глобальные переменные это просто переменные. Ещё одна возможность задавать и хранить значения у программы. Кстати, не будь глобальных переменных, то большинство программ просто не существовало бы. Возможность адресации данных глобально необходима.

Если бы от неё так просто можно было бы отказаться, тогда из php давно бы уже выпилили $GLOBALS

[Valick]

macomics, не выпиливают из-за обратной совместимости. Но использовать $GLOBALS в наше время - это себе дороже.
https://snipp.ru/php/registry

[macomics]

Тогда почему нету пометки, что это устарело. Да и появились $GLOBALS не с самой первой версии. Её ввели, если я правильно помню в 4+ версии или вообще в 5.

Хорошо. Не нравится $GLOBALS, тогда как обойтись без $_GET, $_POST или $_FILES? Это тоже глобальные массивы.

[Valick]

Сообщение от macomics

тогда как обойтись без $_GET, $_POST или $_FILES

С ними тоже уже давно никто не работает напрямую, но вы правы без них ни как.
А ещё и $_SESSION есть))
Но от конструкции global $file_name; больше проблем, чем пользы.