有没有更好的方法来编写这个IF语句？ [关闭]

Question

if($colour && 
      ((strlen($colour) == 7 && preg_match("/#[0-9a-fA-F]{6}/",$colour)) ||
       (strlen($colour) == 4 && preg_match("/#[0-9a-fA-F]{3}/",$colour)))) {

• If $colour has a value AND
  • $colour is a 7 digit string and conforms to the 7 digit hex colour format OR
  • $colour is a 4 digit string and conforms to the 4 digit hex colour format

(String lengths include hash, hence 3 & 6 become 4 & 7)

Answer 1

Yes, there are. If you use RegEx, you can just as well use one regex, like this. The length can be skipped too, by using start or string/end of string anchors.

if ($colour && preg_match("/^#([0-9a-fA-F]){3}(([0-9a-fA-F]){3})?$/", $colour)
{
}

I think the 'check for value' can be skipped to, but that might make sense as an optimization.

[edit]

It turns out to be an example on the RegEx cheat sheet.