исключая значения из цикла foreach

Question

У меня есть следующий код ... и я знаю, что, вероятно, все неправильно, но я раньше не имел дело с циклами foreach.

$last_names = regapiGetLastNames( NULL, -1 );
foreach ($last_names as $name => $last_name_id)
    $exclude = array('11196','11195','11198','11197');
    if(!in_array($name->last_name_id, $exclude)):
    print '<option value="'.$last_name_id.'">'.$name.'</option>';

Очевидно, что где-то что-то идет не так, любая помощь, пожалуйста?

Answer 1

Если идентификаторы являются значениями массива, вы также можете использовать array_diff для их фильтрации:

$last_names = regapiGetLastNames( NULL, -1 );

$exclude = array('11196','11195','11198','11197');
$last_names = array_diff($last_names, $exclude);

foreach ($last_names as $name => $last_name_id) {
    print '<option value="'.$last_name_id.'">'.$name.'</option>';
}

Answer 2

$last_names = regapiGetLastNames( NULL, -1 );
$exclude = array('11196','11195','11198','11197');
foreach ($last_names as $name => $last_name_id)
{
    if(!in_array($name->last_name_id, $exclude))
        print '<option value="'.$last_name_id.'">'.$name.'</option>';
}

Вам нужны фигурные скобки для многострочного цикла. также переместите объявление массива за пределы цикла

Answer 3

Я, вероятно, написал бы немного по-другому, это не правильно ($ name не является объектом)

$name->last_name_id

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

$last_names = regapiGetLastNames( NULL, -1 );
$exclude = array('11196','11195','11198','11197');

foreach ($last_names as $name => $last_name_id) {
    if(!in_array($last_name_id, $exclude)) print '<option value="'.$last_name_id.'">'.$name.'</option>';
}

Answer 4

Вы можете превратить это в ...

$last_names = regapiGetLastNames( NULL, -1 );

$last_names = array_filter($last_names, function($value) {
    return in_array($value, array('11196','11195','11198','11197'));
});

foreach ($last_names as $name => $last_name_id) {
    print '<option value="'.$last_name_id.'">'.$name.'</option>';
}

Когда вы приступаете к выполнению цикла, вы перебираете только тот набор, который хотите. Это полезно для разделения ваших бизнес-правил (отбрасывание определенных идентификаторов) и представления (повторение HTML).