C# Hvordan kalkulere Modulus 10 check digit?

codeslinger

Det kan lønne seg å teste ja. Fant to feil i koden - den første if-testen første til at first og second ble byttet om, og n-variabelen ble inkrementert en ekstra gang før den ble returnert.

Kode

        // Version by Jannis with two bugfixes by tormaroe
        public int GetMod10Digit(string barCode)
        {
            int first = 0;
            int second = 0;
            int i = 0;
            foreach (char num in barCode)
            {
                i++;
                double floating = (double)i / 2;
                int floating2 = i / 2;
                if (floating != floating2)
                {
                    first += int.Parse(num.ToString());
                }
                else
                {
                    second += int.Parse(num.ToString());
                }
            }
            first *= 3;
            int result = first + second;
            bool run = true;
            int n = 0;
            while (run)
            {
                n++;
                if ((result + n).ToString().Last() == '0') run = false;                
                if (n > 10000) return -1;
            }
            return n;
        }

Jeg synes det er ganske morsomt at du ikke har brukt modulo-operatoren, ettersom det er det det dreier seg om her. På tide å lære noe nytt! Ta en titt på min versjon:

Kode

        // Refactored to use modulo
        public int GetMod10Digit_v2(string barCode)
        {
            int first = 0;
            int second = 0;
            int i = 0;
            foreach (char num in barCode)
                if (++i % 2 != 0)
                    first += int.Parse(num.ToString());
                else
                    second += int.Parse(num.ToString());
            int result = (first * 3) + second;
            int n = 0;
            while (true)
                if ((result + ++n) % 10 == 0) 
                    return n;
        }

Trådstarter

Sitat av tormaroe

Det kan lønne seg å teste ja. Fant to feil i koden - den første if-testen første til at first og second ble byttet om, og n-variabelen ble inkrementert en ekstra gang før den ble returnert.

Kode

        // Version by Jannis with two bugfixes by tormaroe
        public int GetMod10Digit(string barCode)
        {
            int first = 0;
            int second = 0;
            int i = 0;
            foreach (char num in barCode)
            {
                i++;
                double floating = (double)i / 2;
                int floating2 = i / 2;
                if (floating != floating2)
                {
                    first += int.Parse(num.ToString());
                }
                else
                {
                    second += int.Parse(num.ToString());
                }
            }
            first *= 3;
            int result = first + second;
            bool run = true;
            int n = 0;
            while (run)
            {
                n++;
                if ((result + n).ToString().Last() == '0') run = false;                
                if (n > 10000) return -1;
            }
            return n;
        }

Jeg synes det er ganske morsomt at du ikke har brukt modulo-operatoren, ettersom det er det det dreier seg om her. På tide å lære noe nytt! Ta en titt på min versjon:

Kode

        // Refactored to use modulo
        public int GetMod10Digit_v2(string barCode)
        {
            int first = 0;
            int second = 0;
            int i = 0;
            foreach (char num in barCode)
                if (++i % 2 != 0)
                    first += int.Parse(num.ToString());
                else
                    second += int.Parse(num.ToString());
            int result = (first * 3) + second;
            int n = 0;
            while (true)
                if ((result + ++n) % 10 == 0) 
                    return n;
        }

Vis hele sitatet...

Veldig kult. Jeg skjønner at dette kanskje var litt vanskeligere enn jeg først forventet, men jeg ser det som god læring å kunne få til noe slikt på egenhånd.

Siden jeg ennå ikke er en racer på dette (jeg lærer best av eksempler), hvordan vil jeg kunne bruke metoden ved en button_click? Jeg trenger ikke nødvendigvis koden, men hva må jeg lese meg opp på for å forstå hvordan jeg kan bruke snippets som dette i en button click action?

codeslinger

Sitat av bronsky

hvordan vil jeg kunne bruke metoden ved en button_click?

Vis hele sitatet...

Tja, antar at du har et windows forms prosjekt i Visual Studio, og et Form.

Dra en textbox, en knapp og en label ut på formet
Dobbeltklikk på knappen (du blir da tatt til kodefilen for formet, hvor det har blitt generert en tom metode for deg som vil bli kalt når knappen trykkes på)
Lim inn mod10-metoden nedenfor click-metoden
Inne i click-metoden kaller du den, f.eks. slik:

Kode

Label1.Text = GetMod10Digit(TextBox1.Text);

Done!

Trådstarter

Tusen takk for hjelpen så langt!

Jeg får error "Cannot implicitly convert type "int" to "string".

Kode

private void button1_Click(object sender, EventArgs e)
        {

            
            label1.Text = GetMod10Digit(txtNumber.Text);
            
        }


        public int GetMod10Digit(string barCode)
        {
            int first = 0;
            int second = 0;
            int i = 0;
            foreach (char num in barCode)
                if (++i % 2 != 0)
                    first += int.Parse(num.ToString());
                else
                    second += int.Parse(num.ToString());
            int result = (first * 3) + second;
            int n = 0;
            while (true)
                if ((result + ++n) % 10 == 0)
                    return n;
        }

Mangler det en

Kode

Convert.ToInt32

eller noe et sted?

Prøvde dette, men det funket jo ikke som jeg ville, selvfølgelig:

Kode

string test = txtNumber.Text;

            Convert.ToInt32(test);

            GetMod10Digit(test);


            label1.Text = test.ToString();

Sist endret av bronsky; 17. februar 2011 kl. 15:11.

codeslinger

Ah, GetMod10Digit returnerer en int, og da feiler dette:

Kode

label1.Text = GetMod10Digit(txtNumber.Text);

Resultatet fra GetMod10Digit (int) tilordnes Text-propertien til label1, som er en string. Og C# vil ikke konvertere fra int til string implisitt.., man må være eksplisitt. Så da må du gjøre f.eks. slik:

Kode

label1.Text = GetMod10Digit(txtNumber.Text).ToString();

Kaller altså metoden ToString() (som alle objekter har) på resultatet fra GetMod10Digit før verdien tilordnes Text-propertien.

Si fra hvordan det går..

Her er en annen variant av mod10-funksjonen, for dem som har lyst til å utvide sin C#-kunnskap:

Kode

        public int GetMod10Digit_v3(string barCode)
        {
            var digits = barCode.ToCharArray().Select(c => int.Parse(c.ToString()));
            var first = digits.Where((num, index) => index % 2 == 0).Sum();
            var second = digits.Where((num, index) => index % 2 > 0).Sum();
            int result = (first * 3) + second;
            int n = 0;
            while ((result + ++n) % 10 != 0) ;
            return n;
        }

De første linjene som definerer digits, first og second ved hjelp av Linq er deklerative og mye mere elegante enn de tidligere eksemplene. Dette er eksempler på funksjonell programmering.

While-løkken uten body derimot er et eksempel på stygg kode med nesten usynlige sideeffekter. Å kunne forstå den er derimot viktig.

Trådstarter

18. februar 2011

Kode

label1.Text = GetMod10Digit(txtNumber.Text).ToString();

Dette fungerte veldig bra. Men det er noe galt i kalkulasjonen. Jeg får nemlig ikke riktig kontrollsiffer.

Jeg har lagt kilden i min DropBox, hvis noen vil ta en titt:

http://dl.dropbox.com/u/10581843/ModCalc.zip

codeslinger

18. februar 2011

Kjørte koden din, og kode 12345678912 gir som forventet kontrollsiffer 8 - det samme som mine tre versjoner av algoritmen. Får ikke du det? Har du andre eksempler på koder og sifre?

Trådstarter

18. februar 2011

Sitat av tormaroe

Kjørte koden din, og kode 12345678912 gir som forventet kontrollsiffer 8 - det samme som mine tre versjoner av algoritmen. Får ikke du det? Har du andre eksempler på koder og sifre?

Vis hele sitatet...

Riktig kontrollsiffer for 12345678912 skal være 2.

2*2= 4
1*1= 1
9*2 = 18= 9
8*1 =8
7*2 = 14 = 5
6*1 = 6
5*2 = 10 = 1
4*1 = 5
3*2 = 6
2*1 = 2
1*2 = 1

= 48 (nærmeste opp til hele 10, altså 50, er 2) ergo kontrollsiffer 2.

Bekreftes også av denne kalkulatoren:

http://bavister.org/tools/genLuhn.php

On second thought:

Nærmeste hele 10 fra 8 er jo også 2, så kan det være et eller annet som mangler på slutten av utregningen?

DumDiDum

2.084 3.081

20. februar 2011

Sitat av bronsky

Vis hele sitatet...

Dette eksempelet ignorer ikke Check digit iallefall (siste sifferet).

Sitat av bronsky

Vis hele sitatet...

Her ganger du med to istedenfor 3 som eksempelet i første posten din sier.

Dette er forøvrig en one-liner i Python som er konsistent med forklaringa i første eksempelet (inkludert at siste sifferet ignoreres):

Kode

10-(sum(map(int,list(str(n)))[:-1:2])*3+sum(map(int,list(str(n)))[1::2]))%10

Edit: så på kalkulatoren du postet i siste innlegg Bronsky: den ganger annenhvert tall med to, og fjerner ikke siste tallet. Samme som din utregning i siste innlegget, men ikke samme som du postet i første innlegget.

Sitat av bronsky

2*2= 4
1*1= 1
9*2 = 18= 9
8*1 =8
7*2 = 14 = 5
6*1 = 6
5*2 = 10 = 1
4*1 = 5
3*2 = 6
2*1 = 2
1*2 = 1

Vis hele sitatet...

Denne er heller ikke konsekvent. Du kjører modulo 9 på alle resultat tydeligvis, men 18%9 er faktisk 0, ikke 9.

Sist endret av DumDiDum; 20. februar 2011 kl. 11:40.

Trådstarter