[C#] Getalvalidatie adhv CultureInfo class

[Roxors]

Goedemiddag,

Ik probeer functionaliteit te ontwikkelen die de input valideert op geldige Nederlandse getallen.
Zo is 12.34 niet goed, en 12.345,67 weer wel natuurlijk.Dit probeer ik te verwezenlijken door gebruik te maken van de CultureInfo class, zodat dit in het geval van een engelse omgeving ook goed gaat.
Hierto heb ik een test console programma'tje geschreven, echter deze geeft onverwachte resultaten.
De code:

C#:

namespace NumberValidation
{
    class Program
    {
        public static string[] testArray = new string[] {"1.2.3.4.5", "1.2.3.4.5,67", "123.45", "12.345", "123.45,67", "12.345,67", "123.00", "12.345,00", "123.45,67", "12.345,67" };

        static void Main(string[] args)
        {
            System.Globalization.CultureInfo cInfo = Thread.CurrentThread.CurrentCulture;
            System.Globalization.NumberFormatInfo nInfo = cInfo.NumberFormat;
            decimal value = 0;
            bool result = true;

            Console.WriteLine("Specified culture is: " + cInfo.DisplayName);
            Console.WriteLine();
            Console.WriteLine();

            foreach (string str in testArray)
            {
                try
                {
                    value = decimal.Parse(str, nInfo);
                }
                catch
                {
                    result = false;
                }

                Console.WriteLine(string.Format("Trying to format : {0}", str));
                Console.WriteLine(string.Format("Was the conversion succesfull: {0}", result));
                Console.WriteLine(string.Format("The formatted result is: {0}", value));
                Console.WriteLine();
                Console.WriteLine();
            }
        }
    }
}

Zoals gezegd staat de culture op Dutch. Echter, alle getallen in de testArray worden goedgekeurd. Kan iemand me vertellen wat ik verkeerd doe?

[RobIII]

Kijk eens naar de NumberStyles parameter en bijbehorende enum

[ Voor 34% gewijzigd door RobIII op 05-05-2009 13:09 ]

[Woy]

En met deze overload van TryParse ( Parse heeft eenzelfde soort overload ) http://msdn.microsoft.com/en-us/library/ew0seb73.aspx hoef je ook geen exception te vangen, maar kan gewoon checken of een number te parsen is.

[Roxors]

Ik heb de try-catch vervangen door onderstaande code maar ik krijg nog steeds dezelfde resultaten? Alles wordt nog steeds goedgekeurd??

C#:

result = decimal.TryParse(str, NumberStyles.Number, nInfo, out value);

[RobIII]

Roxors schreef op dinsdag 05 mei 2009 @ 14:12:
Ik heb de try-catch vervangen door onderstaande code maar ik krijg nog steeds dezelfde resultaten? Alles wordt nog steeds goedgekeurd??

C#:

result = decimal.TryParse(str, NumberStyles.Number, nInfo, out value);

Wat versta je onder "goedgekeurd"?

[Roxors]

De regel

C#:

Console.WriteLine(string.Format("Was the conversion succesfull: {0}", result)); 

retourneert voor alle waarden true, terwijl ik toch echt verwacht dat "1.2.3.4.5" geen geldige waarde is.
Van het NumberFormatInfo object lijkt alles te kloppen, dat wil zeggen, de waarden in de properties komen overeen met de Nederlandse stijl, het lijkt alleen alsof de TryParse zich hier niets van aantrekt.
Zo is bijvoorbeeld de property NumberGroupSizes ingesteld op 3, wat volgens mij wil zeggen dat het aantal posities links van de DecimalSeperator (",") 3 moet zijn voordat de ThousandSeperator (".") voor mag komen. Of zie ik het nou verkeerd?

[RobIII]

AFAIK zal (Try)Parse niet afdwingen dat 'cijfergroeperingssymbolen' op de juiste plek staan; ze zijn puur esthetisch om getallen voor ons mensen leesbaar(der) te maken en verder betekenisloos. Het lijkt er op dat (Try)Parse het dus verder ook zal roesten waar ze staan. Ik vrees dat je controle of er 'voldoende' cijfers in de groep zitten dus zelf zult moeten doen.

Dat 123.45 wordt "goedgekeurd" is logisch; je staat immers cijfergroeperingssymbolen toe; je zou wellicht 123,45 bedoelen maar 't wordt (correct) geïnterpreteerd als cijfergroeperingssymbool en dus wordt 123.45 -> 12345

code:

Specified culture is: Nederlands (Nederland)


Trying to format : 1.2.3.4.5
Was the conversion succesfull: True
The formatted result is: 12345


Trying to format : 1.2.3.4.5,67
Was the conversion succesfull: True
The formatted result is: 12345,67


Trying to format : 123.45
Was the conversion succesfull: True
The formatted result is: 12345


Trying to format : 12.345
Was the conversion succesfull: True
The formatted result is: 12345


Trying to format : 123.45,67
Was the conversion succesfull: True
The formatted result is: 12345,67


Trying to format : 12.345,67
Was the conversion succesfull: True
The formatted result is: 12345,67


Trying to format : 123.00
Was the conversion succesfull: True
The formatted result is: 12300


Trying to format : 12.345,00
Was the conversion succesfull: True
The formatted result is: 12345,00


Trying to format : 123.45,67
Was the conversion succesfull: True
The formatted result is: 12345,67


Trying to format : 12.345,67
Was the conversion succesfull: True
The formatted result is: 12345,67

In bovenstaande redenering kloppen deze resultaten dus allemaal.

[ Voor 87% gewijzigd door RobIII op 05-05-2009 14:55 ]

[Roxors]

Ja, daar was ik al bang voor. Ik hoopte echter dat ik iets miste, aangezien er een IFormatProvider object meegegeven kan worden. Het lijkt er dus op dat een CultureInfo object als IFormatProvider meegeven niks toevoegt.

Jammer, dan maar een Regex om correcte getalinvoer af te dwingen

Bedankt voor de hulp!

[RobIII]

Roxors schreef op dinsdag 05 mei 2009 @ 15:02:
Het lijkt er dus op dat een CultureInfo object als IFormatProvider meegeven niks toevoegt.

Jawel; zo kun je namelijk een afwijkende cultuur-afhankelijke formattering specificeren onafhankelijk van de thread's cultuur-afhankelijke formattering.

[ Voor 6% gewijzigd door RobIII op 05-05-2009 15:08 ]

[Roxors]

Heh, ik zei het misschien een beetje kort door de bocht inderdaad, maar dat is wat ik bedoelde. De CultureInfo wordt niet gebruikt om te kijken naar het aantal cijfers voor of na de decimaal/duizendtal separator, waar ik wel van uit ging eigenlijk. Assumptions are the mother of all f...ups