Использование переменных переменных в простой функции PHP

Question

На днях мне был предоставлен этот огромный набор массивов и он предложил сделать HTML-страницу с пучком или выбрать/радиокнопки, поэтому я написал несколько простых функций для упрощения.

Это моя функция для генерации очень больших HTML-предложений.

function genSelect($name) { 
    $selectReturn = '<select name="'.$name.'">'; 
     foreach(${$name} as $value=>$text){ 
      $selectReturn .= '<option value="'.$value.'"'; 
      if($evalThis->loaded_settings[$name]['value']==$value) 
       $selectReturn .= ' SELECTED '; 
      $selectReturn .= '>'.$text.'</option>'; 
     } 
    $selectReturn .= '</select>'; 
return $selectReturn; 
} 

Это, кажется, не работает, потому что ${$name} просто не вызывает ничего/работу, как я надеюсь, тоже. У меня уже есть работа вокруг, где я просто передаю массив, хотя вызов функции, но он исказил меня, что я делал неправильно с переменными переменными в этом коде.

Редактировать: Чтобы дать некоторый контекст, это загружается в представлении joomla для создания гигантского пользовательского интерфейса для ввода параметров. Эта функция находится в файле lib и загружается с require_once рядом с другим файлом, содержащим все используемые мной массивы. База данных содержит только текущие значения этих параметров, а массивы содержат все возможные варианты для различных меню выбора/радио/выпадающего меню и централизованы в отдельном файле по причинам языка/перевода.

Уточнение некоторых вопросов, заданных людьми, $name просто содержит имя используемого массива, а также имя входа/выбора HTML/флажка. $evalThis - это массив, содержащий значения, имеющиеся в базе данных.

Answer 1

Кажется, массив с именем, хранящимся в $ name, определяется в глобальном контексте, а не в этой функции.

$somearr = array('x', 'y'); 
function genSelect($name) { 
    $selectReturn = '<select name="'.$name.'">'; 

    foreach(${$name} as $value=>$text){ 
     $selectReturn .= '<option value="'.$value.'"'; 
     if($evalThis->loaded_settings[$name]['value']==$value) 
      $selectReturn .= ' SELECTED '; 
     $selectReturn .= '>'.$text.'</option>'; 
    } 

    $selectReturn .= '</select>'; 
    return $selectReturn; 
} 
echo genSelect('somearr'); //doesnt work, somearr is global 


$somearr = array('x', 'y'); 
function genSelect($name) { 
    global $somearr; 
    $selectReturn = '<select name="'.$name.'">'; 

    foreach(${$name} as $value=>$text){ 
     $selectReturn .= '<option value="'.$value.'"'; 
     if($evalThis->loaded_settings[$name]['value']==$value) 
      $selectReturn .= ' SELECTED '; 
     $selectReturn .= '>'.$text.'</option>'; 
    } 

    $selectReturn .= '</select>'; 
    return $selectReturn; 
} 
echo genSelect('somearr'); //works, note 'global $somearr' line at the beginning of genSelect 

Вообще это довольно плохой дизайн, чтобы использовать имена переменных функций/массивов, когда нет необходимости/по крайней мере оправдано. Если на вашем месте я бы просто переписать метод

function genSelect($array, $name) { ... } 

echo genSelect($someArr, 'someArr'); 

Это может показаться дублированием typying, но это лучше, чтобы не иметь ваши методы зависят от глобального масштаба. Btw, ваш $ evalThis var также выходит за пределы области действия.

Answer 2

Проблема в том, что ${$name} не определен внутри области действия. Вы должны были бы либо ссылки на глобальную переменную так:

$GLOBALS[$name] // bad 

Или добавьте это в верхней части функции:

global ${$name}; // worse 

ВНИМАНИЕ: Это ужасно. Не делайте этого :-) Я советую просто передать массив, как вы упомянули ранее.