Forum PHP.pl > [PHP]Generator sumy kontrolnej

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]Generator sumy kontrolnej

Durkane

28.07.2015, 12:18:06

Witam. Otóż mam mały problem ze zrobieniem takiego generatora. Podaje kod poniżej, wydaje mi się, że moja funkcja liczy źle modulo : / Jak ustawie w funkcji zamiast $num $acc to jaki numer konta nie podam to ciągle wylicza sumę kontrolną 98. Zaznaczam, że dopiero uczę się programowania w php (jestem testerem i dostałem takie zadanie domowe od programistów php)

[PHP] pobierz, plaintext 
<?php
$n_acc = (string)$_POST["n_acc"];
$acc = $n_acc."252100";
 
    if (strlen($n_acc) != 20) 
    {
        echo "Numer konta jest za krotki lub za dlugi. Podaj 20 cyfr.";
    } else {
 
    function mod ($num)
    {
       $acc = $n_acc."252100";
 
        for ($i=0; $i<strlen($num);$i++) 
        {
           $CheckSum = "";
           $CheckSum .= $num[$i];
           $CheckSum = (string)((int)$CheckSum%97); 
        } 
 
        if ($CheckSum < 10) 
        {
           $CheckSum = "0".$CheckSum;
        } 
           return $CheckSum;
    }    
 
$CheckSum = mod($n_acc);
$CheckSum = 98 - (int)$CheckSum; 
 
echo "Podany numer konta : ", wordwrap($acc, 4, " ", 1),"<br>";
echo "Suma kontrolna : ".$CheckSum."<br>";
 
 
$CheckAcc = $n_acc."2521".$CheckSum;
$Check = mod($CheckAcc);
 
echo "Sprawdzanie : ".$Check."<br>"; 
 
    if ($Check == 1) 
    {                                                      
        echo "Suma kontrolna prawidlowa.<br>";         
    } else { 
        echo "Suma kontrolna nieprawidlowa.<br>" ;
    }  
$Account = $CheckSum." ".$acc;  
echo  "Numer konta z suma kontrolna : ", wordwrap($Account, 4, " ", 1),"<br>";       
    }  
?>
[PHP] pobierz, plaintext

Tutaj podaje do html'a kod jakby ktoś potrzebował.

[HTML] pobierz, plaintext 
<html>
<head></head>
<body>
<form action ="Nowy2.php"  method ="POST">
 
 <div align="center">
 <br><br>
   <h3> Wpisz 20 cyfrowy numer konta</h3>
 
    <input type="text" name="n_acc" </input><br><br>
    <input type="submit"  value="Generuj"> <br>
 
  </div>
 
</form>
</body>
</html>
[HTML] pobierz, plaintext

Jarma

28.07.2015, 13:32:42

Nie możesz odnieść się do i-tego znalu ciągu za pomocą $num[$i];
Stwórz najpierw zmienną która będzie tablicą. Pomocne może być str_split

PS.
Po co jest to:

[PHP] pobierz, plaintext 
if ($CheckSum < 10) 
        {
           $CheckSum = "0".$CheckSum;
        } 
[PHP] pobierz, plaintext

Skoro później robisz to

[PHP] pobierz, plaintext 
(int)$CheckSum
[PHP] pobierz, plaintext

Durkane

28.07.2015, 14:56:07

[PHP] pobierz, plaintext 
if ($CheckSum < 10) 
        {
           $CheckSum = "0".$CheckSum;
        } 
[PHP] pobierz, plaintext

Suma kontrolna musi mieć 2 cyfry, więc jeżeli wychodzi od 0 do 9 to musimy z przodu dodać 0. Wiem, że jest do tego funkcja odpowiednia

Ale w tym momencie niestety nie pamiętam jej nazwy.

Później daje

[PHP] pobierz, plaintext 
(int)$CheckSum
[PHP] pobierz, plaintext

po to, aby mieć pewność, że odejmując

[PHP] pobierz, plaintext 
98 - $CheckSum
[PHP] pobierz, plaintext

to moje $CheckSum będzie intem (nie ufam za bardzo temu, że PHP samo zamienia na odpowiedni typ).

Czyli jeżeli dobrze rozumiem to użyć str_split na

[PHP] pobierz, plaintext 
$num[$i]
[PHP] pobierz, plaintext

CuteOne

28.07.2015, 18:20:11

Ta funkcja nie ma sensu. Każdy obrót pętli, to nadpisanie poprzedniej wartości w CheckSum - wynieś inicjalizowanie zmiennej z pętli i podmień

[PHP] pobierz, plaintext 
$CheckSum .= $num[$i];
           $CheckSum = (string)((int)$CheckSum%97); 
[PHP] pobierz, plaintext

[PHP] pobierz, plaintext 
$CheckSum .= $num[$i] % 97;
[PHP] pobierz, plaintext

Durkane

28.07.2015, 19:46:43

Ale to będzie brać po 1 liczbie i robić modulo dla jednej liczby. A z tego co mi tłumaczył programista algorytm to ma robić modulo i brać kolejną liczbę i dodawać do wyniku z modulo tzn. np. z modulo mamy 3, a następna liczba to 2, więc kolejny krok to 32%97. Czy to co podałeś będzie tak robić?
Bo to co wkleiłem na początku cały czas podaje mi różne wartości natomiast jeżeli usunę

[PHP] pobierz, plaintext 
$acc = $n_acc."252100";
[PHP] pobierz, plaintext

to wtedy suma kontrolna cały czas jest 98...

Ale jutro sprawdzę Twoim sposobem i zobaczymy jak to wyjdzie

Na razie dziękuję za pomoc.

mls

28.07.2015, 20:35:53

Po pierwsze, numer konta bankowego w Polsce powinien mieć 28 znaków.
Po drugie, wg np. Wikipedii:

Cytat

Algorytm sprawdzania cyfr kontrolnych:

Weź pełny numer konta (razem z kodem kraju), bez spacji.
Sprawdź, czy zgadza się długość numeru dla danego kraju.
Przenieś 4 pierwsze znaki numeru na jego koniec.
Przekształć litery w numerze konta na ciągi cyfr, zamieniając 'A' na '10', 'B' na '11' itd., aż do 'Z' na '35' (dla Polski 2521).
Potraktuj otrzymany ciąg znaków jak liczbę i wylicz resztę z dzielenia przez 97.
Jeśli reszta jest równa 1, to numer konta ma prawidłowe cyfry kontrolne.

Po trzecie - przyjmując, że numer konta jest poprawny (zaczyna się od "PL"):

[PHP] pobierz, plaintext 
function check ($number)
{
	$number = substr($number, 4) . substr($number, 0, 4);
	foreach (str_split(substr(preg_replace_callback('/([A-Z])/', function ($match) { return (ord($match[0]) - 55); }, $number), 0, -2) . '00', 6) as $n)
		$r = (int)((isset($r) ? $r : '') . $n) % 97;
	return strcmp(sprintf('%02d', 98 - $r), substr($number, -2)) == 0;
}
[PHP] pobierz, plaintext

gdzie sprawdzenie może wyglądać np. tak:

[PHP] pobierz, plaintext 
$iban = 'PL91106000760000331000154960';
echo('Numer ' . $iban . ' jest ' . (check($iban) ? 'poprawny' : 'błędny'));
[PHP] pobierz, plaintext

Durkane

30.07.2015, 10:12:49

Dobra dziękuje za pomoc

Już sobie poradziłem i ładnie zlicza

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.