C# Onåbar kod. Varför?

Tjena här får du lite kod för det du vill göra...jag har inte lagt ner en massa tid på det flyger i alla fall...
Jag packade allting i en klass, satte namespacet verifiers på den så kan du ju samla dina verifiers där

Here we go:

using System;
using System.Collections.Generic;

namespace verifiers
{
public class PersonalNumberVerifier
{
//Method to calculate is personalnumber is valid
public bool VerifyPersonalNumber(Int64 personalNumber)
{
int multiplier = 2;
int carrier = 0;

foreach (int i in GetIntArray(personalNumber))
{
if (multiplier == 2)
{
if (i * multiplier > 9)
{
carrier = carrier + GetSplitAndSum(i * multiplier, carrier);
multiplier = 1;
}
else
{
carrier = carrier + i * multiplier;
multiplier = 1;
}
}

else
{
if (i * multiplier > 9)
{
carrier = carrier + GetSplitAndSum(i * multiplier, carrier);
multiplier = 2;
}
else
{
carrier = carrier + i * multiplier;
multiplier = 2;
}
}
}

//Check if divisible by ten
if (carrier % 10 == 0)
{
return true;
}
else
{
return false;
}
}

//Split Int64 into an Array of Int64´s and add the members together
private int GetSplitAndSum(int numberToProcess, int carrierSum)
{
int carrierSummarized = carrierSum;
foreach (int y in GetIntArray(numberToProcess))
{
carrierSummarized = carrierSummarized + y;
}
return carrierSummarized;
}

//Split Int64 into an Array of Int64´s
private Int64[] GetIntArray(Int64 num)
{
List<Int64> listOfInts = new List<Int64>();
while (num > 0)
{
listOfInts.Add(num % 10);
num = num / 10;
}
listOfInts.Reverse();
return listOfInts.ToArray();
}
}
}

//Slut på klassen

Såhär instansierar du (antar att du redan vet hur, men slänger in det ändå):

bool verificationValue = new bool();
verificationValue= personalNumberVerifier.VerifyPersonalNumber(personnummer);

personnummer i parantesen ovan ersätter du förståss med det personnummer du skall kolla...

Hoppas det hjälper något...

tänkte lite vidare du får kasta in lite try catch block för att fånga eventuella fel och hantera dem, det måste du göra i hela din kodmassa så du bygger in undantagshantering direkt så att applikationen du bygger kan hantera sådana situation och act accordingly istället för att cracha...

have a nice one ....

Problemet är

if (controlNumber != personalStringVektor[9])
{
      return false;
}

return true; <-- Denna

Koden ser ut såhär

for (int i=0; i<10; i++)
{
......
      return true;
}

Det betyder att i++ kommer aldrig att kallas eftersom i första loopen av din for så kommer return kallas, och programmet kommer avsluta funktionen.

Jag skulle gissa att koden ska se ut något liknande detta

if (controlNumber != personalStringVektor[9])
{
      return false;
}
else
{
      return true;
}

Edit: Såg att någon redan svarat.

No probs...behöver du hjälp med något så hör av dig...
Detta är ett bra ställe både för tutorials och inspiration:
http://msdn.microsoft.com/sv-se/ms348103