Copy
function coloringixwi($valll){
$colorsett = '';
$colorl = strlen($valll);
if ($colorl>10) {
if (strpos($valll,'car')||strpos($valll,'bycle')) {
$colorsett = 'bad';
}
} else {
if ($valll!=1000&&$valll!=1) {
$colorsett = 'good';
}
}
return $colorsett;
}if 문이 이중으로 있는데 좀 보기가 그래서요.
이럴때는 어떻게 해야 하나요?
이게 최선인가요??
코드 리뷰해주실분..
|
답변 5개
채택된 답변
+20 포인트
strpos 함수를 사용할때 찾을 단어가 처음에 올경우에는 반환값이 0 이라서 타입비교까지 해야됩니다.
Copy
function coloringixwi($valll){
$colorsett = '';
$colorl = strlen($valll);
// 반환위치가 처음이라면 0을 반환함
$pos = (strpos($valll,'car') !== false || strpos($valll,'bycle') !== false) ? true : false;
if ($colorl>10 && $pos ) {
$colorsett = 'bad';
} else if ($valll!=1000 && $valll!=1){
$colorsett = 'good';
}
return $colorsett;
}
echo coloringixwi('123456789').PHP_EOL;
echo coloringixwi('123456789car').PHP_EOL;
echo coloringixwi('car123456789').PHP_EOL; // 0
echo coloringixwi('12345car6789').PHP_EOL;
echo coloringixwi('123456789bycle').PHP_EOL;
echo coloringixwi('bycle123456789').PHP_EOL; // 0
echo coloringixwi(2);
답변을 작성하려면 로그인이 필요합니다.