2012-02-07 24 views
5

Mam problem z potwierdzeniem wkładu pieniężnego. Użyłem niektórych wskazówek z innych moich pytań, aby napisać lepszy kod. Oto, czego używam do sprawdzenia, czy dane wejściowe są pieniędzmi.Sprawdzanie poprawności moich pieniędzy

static void Main(string[] args) 
{ 
    string myTest1 = "$1,234.56"; 
    string myTest2 = "$1.00"; 
    string myTest3 = "$1000.01"; 
    string myTest4 = "$1,234,567.89"; 

    myIsMaybeMoneyValidator Miimv = new myIsMaybeMoneyValidator(); 

    bool myResult1 = Miimv.isMaybeMoney(myTest1); 
    bool myResult2 = Miimv.isMaybeMoney(myTest2); 
    bool myResult3 = Miimv.isMaybeMoney(myTest3); 
    bool myResult4 = Miimv.isMaybeMoney(myTest4); 
} 

public bool isMaybeMoney(object theirMaybeMoney) 
{ 
    string myMaybeMoney = theirMaybeMoney.ToString(); 

    if (myMaybeMoney.StartsWith("-")) 
    { 
     myMaybeMoney.Remove(0, 1); 
    } 

    if (!myMaybeMoney.StartsWith("$")) 
    { 
     return false; 
    } 

    myMaybeMoney.Remove(0, 1); 

    string[] myMaybeMoneyStringArray = myMaybeMoney.Split('.'); 

    string myMaybeMoneyDollars = myMaybeMoneyStringArray[0]; 
    string myMaybeMoneyCents = myMaybeMoneyStringArray[1]; 

    if (!isDollars(myMaybeMoneyDollars)) 
    { 
     return false; 
    } 

    if (!isCents(myMaybeMoneyCents)) 
    { 
     return false; 
    } 

    return true; 
} 

private bool isDollars(string theirMaybeMoneyDollars) 
{ 
    if (!isNumber(theirMaybeMoneyDollars)) 
     return false; 

    try 
    { 
     int myMaybeDollars = Convert.ToInt32(theirMaybeMoneyDollars); 

     if (myMaybeDollars < 1) 
      return false; 

     return true; 
    } 
    catch (Exception) 
    { 
     return false; 
    } 

    return true; 
} 

private bool isCents(string theirMaybeMoneyCents) 
{ 
    if (!isNumber(theirMaybeMoneyCents)) 
     return false; 

    try 
    { 
     int myMaybeCents = Convert.ToInt32(theirMaybeMoneyCents); 

     if (myMaybeCents > 99) 
      return false; 
     if (myMaybeCents < 1) 
      return false; 

     return true; 
    } 
    catch (Exception) 
    { 
     return false; 
    } 

    return true; 
} 

private bool isNumber(object theirMaybeNumber) 
{ 
    return Microsoft.VisualBasic.Information.IsNumeric(theirMaybeNumber); 
} 

Wyniki nie są pieniędzmi, co jest dla mnie dość mylące.

+0

Co to jest "myIsMaybeMoneyValidator"? Gdzie to jest zdefiniowane? – Oded

+0

Oczekiwanie na rozwiązanie regex ... Również dlaczego nie używasz regex Założę się, że istnieje wiele bardzo dobrych przykładów z nich dla USD. –

+0

Musisz nauczyć się wyrażeń regularnych – JoelFan

Odpowiedz

11

Wygląda na to, że istnieje wbudowana walidacja dla pieniędzy w języku C#.

float num; 
bool isValid = float.TryParse(str, 
NumberStyles.Currency, 
CultureInfo.GetCultureInfo("en-US"), // cached 
out num); 

OD: https://stackoverflow.com/a/617847/290822

+0

To działało, dziękuję. –

1

Spróbuj decimal.TryParse (myTest4, NumberStyles.Currency, nowy CultureInfo ("en-us"), z myResult4)

0

Wystarczy użyć:

float value; 
bool isMoney= float.TryParse(str, 
    NumberStyles.Currency, 
    CultureInfo.GetCultureInfo("en-US"), 
    out value); 

Jeśli chcesz się dowiedzieć, co jest nie tak z Twoim kodem, prawdopodobnie jest to Microsoft.VisualBasic.Information. IsNumeric (ichMaybeNumber); Brak obsługi przecinków.

+1

hmm powielenie mojej odpowiedzi ... z wyjątkiem brzęczenia OP .. –

+1

IsNumeric zwraca również wartość True, jeśli wyrażenie zawiera prawidłowe wyrażenie liczbowe rozpoczynające się znakiem + lub - lub zawierające przecinki. –

Powiązane problemy