Объединение функций PHP

Question

Я унаследовал некоторый php, который анализирует XML-файл, чтобы заполнить страницу, полную неупорядоченных списков, и мне интересно, есть ли способ консолидировать функции php, чтобы сделать их более эффективными.

Есть 25 или так функционирует, как следующее:

function oaAccounting(){ 

    // load SimpleXML 
    $term = new SimpleXMLElement('training_list.xml', null, true); 
    echo <<<EOF 
    <ul> 
EOF; 

    foreach($term as $term) 
    { 
    if(preg_match("/accounting/i", $term->keyword)){ 
     echo <<<EOF 
     <li>{$term->name}</li> 
EOF; 
    } 
    } 
    echo '</ul>'; 
} 

каждый просматривает файл XML для термина/ключевого слова, это поиск и добавляет этот термин в качестве элемента списка на неупорядоченный список специфического для данного функция. Следующая функция выполняет одно и то же, но для другого ключевого слова term/и добавляет его в отдельный неупорядоченный список.

Есть ли способ объединить все это, чтобы избежать необходимости делать foreach и если 25 раз подряд?

Спасибо!

Answer 1

Эта пересмотренная функция принимает один параметр, слово для поиска, а затем использует его в функции preg_match(). Поэтому, чтобы использовать все, что вам нужно сделать, это позвонить ему с термином, который вы ищете. Это должно заменить все жестко закодированные функции.

function searchXML($search){ 

    // load SimpleXML 
    $terms = new SimpleXMLElement('training_list.xml', null, true); 
    echo <<<EOF 
    <ul> 
    EOF; 
    foreach($terms as $term) 
    { 
     if(preg_match("/$search/i", $term->keyword)){ 
      echo <<<EOF 
      <li>{$term->name}</li> 
     EOF; 
    }} 
    echo '</ul>'; 
} 

Answer 2

Аннотация ваша функция.

Пропустите файл и условия поиска как аргументы функции, и работать с этим.

Что-то вроде:

function parse_xml_file($file, $terms) 

Затем используйте проход $file и $terms при вызове функции.

Для получения дополнительной информации см. Teh big bad manual.