c++, kontroll av personnummer

Borde inte de här fallen vara omkastade? Tänk på att när ni loopar över siffrorna så har den första siffran i personnumret index i == 0.

måste det inte vara en klass eller struktur för att man ska kunna använda this?

checkPnr skulle kunna vara en medlemsfunktion någonstans, men this är ändå en pekare och fungerar därför dåligt med punktoperatorn.

public bool checkPnr()
{
	int len, pnummer;

	String pnr = this.pnumber;

Var finns pnumber, och är checkPnr en medlemsfunktion i en klass?

Titta också på felmeddelandena; känner kompilatorn till String, eller borde du skriva

System::String^ pnr

this är också en pekare, så det borde nog vara

    System::String^ pnr = this->pnumber;

(Jag antar alltså att du kör c++/cli)

jag kan inte c++ men jag gjorde en liknande grej i javascript. kolla källkoden för lite tips om du vill. det här kan säkerligen göras väldigt mycket bättre och jag vet inte om min metod är 100%-ig heller men ändå. här e den:
http://chal.nu/personnummer/

För att testa hur kort det kunde bli så skrev jag algoritmen för att kontrollera att det är ett giltigt personnummer i C# med LINQ:

string pn = "7205181212";

bool valid = pn.Length == 10 && pn.Select((c, i) => {
    int n = Int32.Parse(c.ToString()) * (2 - (i & 1));
    return n < 10 ? n : n - 9;
}).Sum() % 10 == 0;

Funktionell programmering FTW

string pn = "7205181212";

bool valid = pn.Length == 10 && pn.Select((c, i) => {
    int n = Int32.Parse(c.ToString()) * (2 - (i & 1));
    return n < 10 ? n : n - 9;
}).Sum() % 10 == 0;

snyggt!
hur skulle det där se ut i JS?

Obskyrare är roligare. Naiv scheme-tolkning:

((λ (f n)
   (and (> n 1000000000) (< n 9999999999) (=(remainder (f f n 0) 10) 0)))
 (λ (f n x)(let ((k (remainder n 10)) (x1 (+ x 1)))
     (if(= n 0)
        0
        (+(remainder (* x1 k) 10) (quotient (* x1 k) 10) (f f (quotient n 10) (remainder x1 2)))))) 
 7205181212)

Nice! Jag hade inte sett ditt inlägg förrän nu.

Här kommer en konvertering av din kod till F#:

let valid n =
    let rec f n x = 
        match n with
        | 0UL -> 0UL
        | n ->  let k = n % 10UL
                let x1 = x + 1UL
                (x1 * k) % 10UL +
                (x1 * k) / 10UL +
                f (n / 10UL) (x1 % 2UL)
    n >= 1000000000UL &&
    n <= 9999999999UL &&
    (f n 0UL) % 10UL = 0UL

let pn = 7205181212UL

System.Console.WriteLine (pn.ToString () + " is " + if (valid pn) then "valid" else "invalid")

Fast det blev namngivna funktioner istället för anonyma.

Bara för att lägga till ännu ett exempel, skrivet i PHP för nått år eller två sen:

$personnummer = 'yymmddxxxx';
$summa = 0;
for($i = 1; $i < 10; $i++) {
    $del = $personnummer[$i - 1] * ($i % 2 + 1);
    if($del > 9) $summa += 1 + $del % 10;
    else $summa += $del;
}
if(($summa + $personnummer[9]) % 10) echo 'Ej giltligt.';
else echo 'Giltligt.';

Ser genast sätt att göra det mindre, som vanligt när man kollar på lite äldre kod

Schyst! Uppdatera gärna med förbättringarna du tänker på, om du har lust. Det kan vara illustrativt att se hur man skriver en sån här (förvisso enkel) grej i olika språk på smidigaste sätt.

function validateSSN($personnummer = '0000000000') { // yymmddxxxx 
    $summa = 0;  
    for($i = 1; $i < 10; $i++) { 
        $del = $personnummer[$i - 1] * ($i % 2 + 1); 
        $summa += ($del > 9 ? 1 + $del % 10 : $del); 
    }   
    return !(($summa + $personnummer[9]) % 10);
}

Omskriven till funktion och lite mindre, om än inte så mycket. Orkar inte riktigt göra någon stor overhaul och tänka om