вы не прерываете цикл, если найдете домен, поэтому то, что вы получаете, на самом деле является результатом только проверенной ПОСЛЕДНЕЙ строки. просто добавьте break; после $output = TRUE;

Из официального документа strpos:

Предупреждение

Эта функция может возвращать логическое значение FALSE, но также может возвращать нелогическое значение, которое оценивается как FALSE. Пожалуйста, прочтите раздел о логических значениях для получения дополнительной информации. Используйте оператор === для проверки возвращаемого значения этой функции.

И не забудьте добавить разрыв после того, как вы установили $output в true.

Это хорошая функция для использования оператора ===, поскольку она гарантирует, что значение и тип равны (1==true, но 1!==true).

if (strpos( $email, $domain )!==false) {
    $output = TRUE;
}

Изменять

if ( $pos ) {

to

if ( $pos !== false) {

это связано с тем, что is strpos возвращает 0, что будет равнозначно false, даже если строка была найдена.

Вот два прямых/распространенных метода, которые имеют разные преимущества:

Способ 1: подход без регулярных выражений

function check_email1($email){
    $whitelist_domains=['@domain1.com','@domain2.com','@domain3.com'];
    foreach($whitelist_domains as $domain){
        if(strpos($email,$domain)!==false){
            return true;  // allow quick return (exit loop & function asap)
        }
    }
    return false;  // default response
}

Способ 2: регулярное выражение

function check_email2($email){
    $whitelist_pattern='/@(?:domain1\.com|domain2\.com|domain3\.com)$/';  // condense if possible /@domain[123]\.com$/
    return (bool)preg_match($whitelist_pattern,$email);  // convert 0 or 1 output to boolean (false/true)
}

Ссылка на демонстрацию

Ввод/вызов функции:

$emails=['[email protected]','[email protected]'];
foreach($emails as $email){
    echo "$email\n";
    var_export(check_email1($email));
    echo "\n";
    var_export(check_email2($email));
    echo "\n\n";
}

Вывод:

[email protected]
true
true

[email protected]
false
false

Преимущества/недостатки:

• strpos() в большинстве случаев превосходит функции регулярных выражений. Ваш метод по умолчанию должен заключаться в использовании строковых функций и переходе на регулярное выражение только тогда, когда строковые функции менее эффективны или слишком запутаны для кода. Связанная страница: Что более эффективно, PHP строковые функции или регулярное выражение в PHP?

• Зацикливание $whitelist_domains в № 1 делает блок кода более неуклюжим по сравнению с № 2 (который можно сжать до однострочного, если вы запишете шаблон непосредственно в preg_match()).

• Простые/распространенные ошибки иногда случаются при работе с strpos(). Эти ошибки могут включать:

  • not checking for false in the if condition
  • написание haystack и needle в неправильном порядке

• # 2 требует некоторых знаний о регулярных выражениях (экранирование, классы символов, альтернативы и т. д.), что может быть сдерживающим фактором для неопытных кодеров. В зависимости от того, как вы напишете свой шаблон регулярного выражения и сколько доменов будет занесено в белый список, № 2, вероятно, будет сложнее поддерживать, чем № 1.

• #2 имеет дополнительное преимущество, заключающееся в возможности проверки появления подстроки domain.com в конце слова с помощью метасимвола $. По этой причине регулярное выражение предлагает более сильную проверку.

Вы должны изменить свой код в этом:

function check_email($email) {
    $whitelist_domains = array(
        '@domain1.com',
        '@domain2.com',
        '@domain3.com'
    );

    foreach ($whitelist_domains as $domain) {   
        if ( strpos( $email, $domain ) !== false ) {
            return true;
        }
    }
    return false;
}

Документация strpos

Цитата из руководства (http://php.net/manual/en/function.strpos.php):

Также можно использовать оператор !==. Использование != не будет работать должным образом, поскольку позиция 'a' равна 0. Оператор (0 != false) оценивается как false.

Пример кода

<?php
$mystring = 'abc';
$findme   = 'a';
$pos = strpos($mystring, $findme);

// The !== operator can also be used.  Using != would not work as expected
// because the position of 'a' is 0. The statement (0 != false) evaluates 
// to false.
if ($pos !== false) {
     echo "The string '$findme' was found in the string '$mystring'";
         echo " and exists at position $pos";
} else {
     echo "The string '$findme' was not found in the string '$mystring'";
}
?>