Modulo10 rekursiv

[dsxs]

Eine einstellige Prüfziffer wird u.a. bei schweizer Einzahlungsscheinen als letzte Ziffer angefügt, nach der Referenznummer, aus welcher sich diese Ziffer errechnen lässt (nach Modulo 10 - rekursiv)

Hatte Mühe damit, dies in PHP hinzukriegen, hier also der Code, falls mal jemand danach sucht und ordentlich Zeit sparen möchte:

 <?php

function modulo10($ref_nr) {
    $table = array(0,9,4,6,8,2,7,1,3,5);

    for ($i=0; $i<strlen($ref_nr); $i++) {
        $next = $table[($next + substr($ref_nr, $i, 1)) % 10];
    }
    return (10 - $next) % 10;
}

// bsp, ergibt 4
echo modulo10("00000000800514220080540818");

?>  

[Basti]

$next wird nicht initialisiert. Scheinst PHP-Warnungen zu unterdrücken oder übersehe ich grad nur was?

Basti

[WarrenFaith]

Ne da haste recht. Da müsste ne Initialisierung vor die for-Schleife.

[dsxs]

Da habt ihr Recht, danke für den Tipp.
Es funktioniert jedoch auch so - schöner ist aber immer besser ;)


Grüsse

[Basti]

Das error_reporting-Level sollte man immer mind. auf E_ALL setzen. Damit muss man zwar die ein und andere Zeile mehr tippen, aber der Aufwand lohnt sich, da man damit hilfreiche Informationen erhält und ein paar der vielen Type-Casting-Fallen entschärft. Auf dem Produktivserver dann einfach display_errors ausschalten.

"funktioniert jedoch auch so" ist also an eine Bedingung geknüpft, die wohl kaum jemand zu schaffen bereit sein wird.

Basti

[dsxs]

Das error_reporting-Level sollte man immer mind. auf E_ALL setzen. Damit muss man zwar die ein und andere Zeile mehr tippen, aber der Aufwand lohnt sich, da man damit hilfreiche Informationen erhält und ein paar der vielen Type-Casting-Fallen entschärft. Auf dem Produktivserver dann einfach display_errors ausschalten.

"funktioniert jedoch auch so" ist also an eine Bedingung geknüpft, die wohl kaum jemand zu schaffen bereit sein wird.

Basti

Gebe dir da sehr wohl Recht und handhabe die Programmierung in der Regel nicht anders.
Trotzdem sehe ich einen gewaltigen Unterschied zwischen öffentlich zugänglichen Websites mit PHP backend, bei welchen es grob fahrlässig und schlichtweg dumm wäre, nicht vorsichtig und mit System zu coden.
Bei einem gesicherten Adminpanel auf einem geschlossenen Server hingegen, spare ich mir gerne Zeit, indem ich so code, dass es funktioniert, und mich nicht in Kleinigkeiten verliere... Grundsatzfrage, aber danke für deinen Input.

[WarrenFaith]

Jedem sein Stil, aber tu mir den Gefallen und korrigiere deinen Code im ersten Post und initialisiere $next, damit ein durchweg korrektes Codesnippet dort liegt.

Und lasst uns jetzt nicht auf error-Handling/Reportingdiskussionen abschweifen.

Danke

[robo47]

Bei einem gesicherten Adminpanel auf einem geschlossenen Server hingegen, spare ich mir gerne Zeit, indem ich so code, dass es funktioniert, und mich nicht in Kleinigkeiten verliere... Grundsatzfrage, aber danke für deinen Input.

Wenn ich mich nicht ganz arg täusche, merkt PHP intern ja auch auch bei deaktiviertem Errorhandling, bzw. einem niedrigen Error-Level jeden Fehler, was wiederum auch Rechenzeit verbraucht, mag zwar minimal sein, aber zig Notices für unintialisierten variablen können sich vielleicht auch MINIMAL bemerkbar machen