Как переписать этот говнокод?

Question

[Ilya Korablev]

Здравствуйте. Как можно переписать этот говнокод? Можно примеры?

<?php

$mystring = mb_strtolower($_POST['text']);
	
	$findme   = "1";
	$pos1 = strpos($mystring, $findme);
	$findme   = "2";
	$pos2 = strpos($mystring, $findme);
	$findme   = "3";
	$pos3 = strpos($mystring, $findme);
	
	
	if ($pos1 === false or $pos2 === false or $pos3 === false) {
	echo "OK";
	}
	else{
	echo "Ошибка";
	}

Answer 1

[SagePtr]

По сути, там проверяется, содержит ли строка символы 1, 2 или 3.

$mystring = mb_strtolower($_POST['text']);
if (!preg_match("/[123]/u", $mystring)){
  echo "OK";
} else {
  echo "Ошибка";
}

Comments

[Ilya Korablev]

А если надо проверить слова. Например, желтый, синий и красный? И если хотя бы одно слово есть в $_POST, то вывод OK?

[SagePtr]

Ilya Korablev: тогда так:

if (preg_match("/(синий|желтый|красный)/u", $mystring)){
  echo "OK";
} else {
  echo "Ошибка";
}

или даже так (чтобы ё и регистр воспринимало):

if (preg_match("/(синий|ж[её]лтый|красный)/ui", $mystring)){

Answer 2

[Виталий Инчин ☢]

Регулярки нагружают сервер, не то чтобы слишком, но я считаю, что не нужно их юзать, когда можно пойти другим легким путем:

function if_str($str, $arr){
   foreach($arr as $val){
      if(substr($str, $val)!==false){
          return true;
      }
   }
   return false;
}

if(if_str("1234545", array(1, 2, 3))){
   echo "OK";
}