Algoritmo en PHP

[Italico76]

@dashtrash : no tiene codigo innecesario como Ud dice... Ud asume que jamas se le puede suministrar una precondicion correcta a esa funcion ? o tiene problema porque estan en distintas lineas los chequeos de cada una ?

Sino se van cumpliendo las precondiciones lo devuelve con false, eso no esta "mal" solo que como dice @guardamicorreo no cumple la recomendacion PSR de un solo return()

De todas formas podria ir chequeando precondiciones y sino se cumplen, generar las excepciones correspondientes, lo cual no esta mal si se hacen todos los chequeos al inicio de la funcion o metodo y es como esta hecho:

Código PHP:

if (!ctype_digit($str)) return FALSE; 
  $len = strlen($str); 
  if ($len != 15) return FALSE;
 

Por que no agrupar las precondiciones en un solo IF() al comienzo de la funcion ?

Código PHP:

if ( (!ctype_digit($str)) OR  (strlen($len) != 15) ) return FALSE;
 

Seria ineficiente (no mucho pero suponiendo nos importa) porque luego NECESITO la longitud de la cadena para seguir con el algoritmo y deberia volver calcularla!

Otra opcion seria:

Código PHP:

$len = strlen($str); 
 if (!ctype_digit($str)) return FALSE; 
if ($len != 15) return FALSE;
 

<< pero esta peor porque sino son cadenas de numeros para que voy a calcularle la longitud ? encima empujo la verificacion de la primera precondicion mas adentro de la funcion!


Y finalmente... si fuera testarudo y agrupara las verificaciones en una sola linea y luego me dicen que desean que esa funcion genere excepciones me tocaria refactorizar porque son excepciones distintas (si me importa algo claro):

Código PHP:

<?php
 
public static function is_valid($str,$error=false)  
{   
  if (!ctype_digit($str))
    if ($error)
        throw new InvalidArgumentException("La cadena no es numerica!");  
    else return False;  
  
  $len = strlen($str); 
  if ($len != 15)   
    if ($error)
        throw new LengthException("Longitud incorrecta!");  
    else return False;  
    
  $odd = !strlen($str)%2; 
  $sum = 0; 
  for ($i=0; $i<$len; ++$i) { 
    $n = 0 + $str[$i]; 
    $odd = !odd; 
    if ($odd) { 
      $sum += $n; 
    } else { 
      $x=2*$n; 
      $sum += $x>9?$x-9:$x; 
    } 
  } 
  return (($sum%10)==0); 
}