Вот моя функция, к удивлению рабочая

function hq_io_check($value,$mode) 
{
function check ($value) {if ((strlen($value)<255) && (strlen($value)>0))
 {return $value;} else {return 2;}}
if ($value=='') {return 2;}
if ($mode=='') $mode='S';
$mode=strtoupper($mode);
switch ($mode){
case 'S':{
if (!preg_match("/[^a-zA-Z0-9]/", $value))
{$a=check($value);
if ($a!=2) {return $value;} else {return 2;}} else {return 2;}
break;
}
case 'A':{
if (!preg_match("/[^a-zA-Z0-9,]/", $value)) 
{$str=explode(",",$value);unset($off);
for ($x=0;$x<count($str);$x++) {if (check($str[$x])==2) $off++;}
if ($off>0) {return 2;} else 
{
for ($m=0;$m<count($str);$m++) 
{for ($n=0;$n<count($str);$n++){if ($m!=$n) {if (strtolower($str[$m])==strtolower ($str[$n])) {$off++;}}}}
if ($off>0) {return 2;} else  {return $value;}
}
} 
else
{return 2;}
break;
}
}
}

Смысл ее фильтрования данных, вызываться фун-ция может в двух режимах
1.Одиночное значение (ключ S)

hq_io_check('table','S')

2.Список значений через запятую (ключ A)

hq_io_check('mama,papa','A')

Значения могут состоять только из латинских букв и цифр 0-9. Если задается список значений, то он проверяется повторение значений.

Покажите где можно упростить и оптимизировать фун-цию