[Bleistift]

Ich hab mir das Script nicht ganz durchgesehen, aber beim Überfliegen sind mir einige Sachen aufgefallen.

1. PHP-Code:

   // alle leerzeichen entfernen
    $ISBN = str_replace(".", "", $ISBN); 

   "." ist ein Leerzeichen? Der Kommentar ist sowieso relativ überflüssig, da die Zeile Code nicht wirklich weiter beschrieben wird. Etwa das selbe wie:
   
   PHP-Code:

   // X wird 5 zugewiesen
$x = 5; 

2. PHP-Code:

   $z1  = substr($ISBN, 0, 1);
      $z2  = substr($ISBN, 1, 1);
      $z3  = substr($ISBN, 2, 1);
      $z4  = substr($ISBN, 3, 1);
      $z5  = substr($ISBN, 4, 1);
      $z6  = substr($ISBN, 5, 1);
      $z7  = substr($ISBN, 6, 1);
      $z8  = substr($ISBN, 7, 1);
      $z9  = substr($ISBN, 8, 1);
      $z10 = substr($ISBN, 9, 1);
      $z11 = substr($ISBN, 10, 1);
      $z12 = substr($ISBN, 11, 1);
      $z13 = substr($ISBN, 12, 1); 

   Gar nicht schön... Besser:
   
   PHP-Code:

   $z1  = $ISBN[0];
      $z2  = $ISBN[1];
//... 

   Aber wieso überhaupt diese vielen Zuweisungen? Ich würde das so machen:
   
   PHP-Code:

   $pruefziffer = (10-(($ISBN[0]+$ISBN[2]+$foo+$bar))); 

3. PHP-Code:

   // wenn nun die berechnete Prüfziffer mit der übergebenen Prüfziffer identisch ist, dann handelt es sich um eine valide ISBN
      if($z13 == $pruefziffer)
      {
        return true;
      }
      else
      {
        return false; // andernfalls gibt die Prüffunktion ein false zurück
      } 

   Schöner:
   
   PHP-Code:

   return $z13 == $pruefziffer; 

   bzw. eben:
   
   PHP-Code:

   return $ISBN[12] == $pruefziffer; 


Das soll keine Kritik sein... Ich versuche nur, deine Code-Qualität ein bisschen zu steigern

[Basti]

Da seh ich auch noch ein bisserl was:

str_replace() kann mit Arrays:

PHP-Code:

    $ISBN = str_replace(array('.', '-'), '' $ISBN); 


Die ISBN-13-Abfrage kann man kürzer schreiben:

PHP-Code:

if (preg_match('/^97[8|9]\d{10}$/', $ISBN)) 


Zahl (bzw. String) in Array:

PHP-Code:

$aIsbn = str_split($ISBN); 


Berechnung für ISBN-10 könnte dann so aussehen:

PHP-Code:

$iExpChecksum = (int) str_replace(array('x','X'), 10, array_pop($aIsbn));

for ($iSum = $i = 0; $i < 9; $i++)
    $iSum += ($i+1) * (int) $aIsbn[$i];

return $iSum % 11 === $iExpChecksum; 


…allerdings muss dann hier vorher natürlich auch mit einem RegExp geprüft werden, damit ein anderer Buchstabe als x nicht als 0 gecastet wird.

Damit könnte man aber auch gleich die Positionen der Bindestriche überprüfen.

Basti

[Jann Hendrik]

Zitat von Bleistift

Das soll keine Kritik sein...

schade! Ich bin stets an konstruktiver Kritik interessiert - und genau so habe ich den post auch verstanden!

Was die angesprochenen Punkte angeht:
Da ist einiges bei was sehr gut klingt - ich werde das überarbeiten! Bei dem einen oder anderen Vorschlag muss ich selbst erst ein wenig ausprobieren!