多次包含php文件的foreach循环-可以接受吗？

Question

I have this function "processMessage($msg)" that processes string based on the first few words in it (prefixes).

I pull lots of rows from the database and pass each $msg string through said function...

The caveat here is that, the function doesn't have the "if($prefix=='blah')" conditions, but rather it includes a bunch of php files that contains these conditions.

Why?

Because instead of hard coding a bunch of conditions in one function, I wanted to organize each condition through separate php files for portability, customization etc (Long story here)

So it basically looks like this (keeping code simple):

Main script loads rows from database and puts messages in an array $msg_r then loops through each msg as:

foreach (msg_r as $key=>$msg){
    processMsg($msg);
}

the actual Processor function :

function processMsg($msg){

    $msg_r = explode(" ",$msg); // break apart message based on spaces .. eg. "reboot machine 30"
    //prepare prefixes
    $prefix1 = $msg_r[0];// reboot
    $prefix2 = $msg_r[1];// machine
    $prefix3 = $msg_r[3];// 30

    //process the above prefixes.. but instead of hard coding multiple if conditions here, load if else conditions from files. 
    require("condition1.php");
    require("condition2.php");
    require("condition3.php");
    //my actual require code is in a loop that loads all files found in a target directory


}

the conditions files are basically just php code of an if else condition for each purpose, eg:

if($prefix1 == 'reboot' and $prefix2 == 'machine') {
// do something
}

So it's a simple setup that seems to work during my tests, but I wonder if this is a "normal" or "acceptable" strategy, or if ya'll can suggest a different approach?

Regards to all

Answer 1

Use an array of functions, and have each include file define a function and push it onto the array. So the main script would contain:

$test_array = array();
require ("condition1.php");
require ("condition2.php");
...

and the include files would do:

$test_array[] = function($prefix1, $prefix2, $prefix3) {
    if ($prefix1 == 'reboot' && $prefix2 == 'machine') {
        // do something
    }
};

And your main function would be:

function processMsg($msg){
    global $test_array;

    $msg_r = explode(" ",$msg); // break apart message based on spaces .. eg. "reboot machine 30"
    //prepare prefixes
    $prefix1 = $msg_r[0];// reboot
    $prefix2 = $msg_r[1];// machine
    $prefix3 = $msg_r[3];// 30

    foreach ($test_array as $test) {
        $test($prefix1, $prefix2, $prefix3);
    }
}