Regex problem i C#.net

public const string NonHyphenSpecialChar = @\.|,|\||\?;

public const string Hyphen = "-";

public const string ExtendedSpecialChar = @!|@|#|$|%|\^|&|\*|=|_|\+|[|]|{|}|\|;|<|>|~|\|; 

public const string NumericChar = "[0-9]";

public const string AlfabeticUpperCase = "[A-Z]";

public const string AlfabeticLowerCase = "[a-z]";

public const string SpecialCharecter = NonHyphenSpecialChar + "|" + Hyphen;

 public const string DataCode = SingleDataCodeCharacter + "|(" + FirstLastDataCodeChar + DataCodeChar + 
            "*" + FirstLastDataCodeChar + ")";

public const string DataCodeChar = FirstLastDataCodeChar + "|" + BlankChar;

public const string FirstLastDataCodeChar = SingleDataCodeCharacter + "|" + Hyphen;

public const string SingleDataCodeCharacter = AlfabeticUpperCase + "|" + AlfabeticLowerCase + "|" +
            NumericChar + "|" + NonHyphenSpecialChar + "|" + ExtendedSpecialChar;

public static void Main(string[] args)
        {
            string test = "d9s";
            Regex r = new Regex(@^ + DataCode + @$);
            Match m = r.Match(test);
            Console.WriteLine("m: " + m.ToString());
            Console.WriteLine("t: " + test);
            Console.Read();
        }

Kan du inte göra en if-sats som kollar längden på strängen och sen stoppa in rätt regexp-uttryck?

DataCode = "^((" + SingleDataCodeCharacter + ")|(" + FirstLastDataCodeChar + DataCodeChar + "*" + FirstLastDataCodeChar + "))$";

Enligt mig så använder du "^" och "$" på fel sätt. När du har två eller fler uttryck i din regexp så behöver du det där för varje uttryck.
Exempel:
^[a-z]|[a-z][A-Z]$
ger inte samma resultat som
^[a-z]$|^[a-z][A-Z]$

Om du hade lyckats klämma in "^" och "$" på varje uttryck så hade det fungerat.

Personligen så hade jag gjort en RegexpHelper-klass där man kan hämta rätt dynamisk regular expression.
Ibland vill man ha "^" och "$" på varje uttryck, ibland vill man ha det runt hela regexpen, etc.

Men det kanske är för stort steg att ta nu, baserat på hur långt du redan har använt det du har?

Godmorgon på dig.

Varför man väljer regexp är för att det finns ett bestämt mönster av tecken och ja, det är lite svårt att göra regexp när man inte vet kraven.

Du ska egentligen kontrollera om det är ett decimaltal?
Jag tror att jag hade kört Decimal.TryParse() eller något annat.

Hur som helst...
Kommatecknet får inte heller komma först eller sist(?)
Det finns inte alltid ett kommatecken(?)

Regex:
      (första)       (andra)
^[0-9]+,{0,1}[0-9]+$|^[0-9]+$

Test och resultat:
123123,123123   //Match (första)
123,123123123   //Match (första)
123123,
,123123
124124          //Match (första)
1               //Match (andra)
1,
,1

Testverktyg:
http://gskinner.com/RegExr/

Istället för {0, 1} kan man använda ?

Snyggt!

Precis.

if (Decimal.TryParse(userInputData))
{
       doMagic();
}

Denna lösning oändligt bättre, enligt mig.
Men det betyder inte att det du har gjort är onödigt. Du har ju lärt dig lite mer om Regex och det är väl aldrig fel?

Hur kan du tycka att det är jobbigare att skriva EN rad kod som för övrigt är lättläst jämfört med Regexp som består av flera rader och kräver regexp tolkning?
Jag vet inte vad majoriteten använder men jag hoppas och chansar på att det är "vanlig kod". Regexp är oftast (om inte alltid) långsammare än "vanlig kod". Regexp använder jag bara när det inte "blir enkelt" att skriva "vanlig kod", om mönstret man letar efter är för besvärligt alltså.

Regexp är ju också en form av kodning, likaså all kod runt själva regexp-uttrycket.

I ditt nya fall:
Du menar, du har en sträng som ser ut på det där viset och du vill ha ut alla tal mellan kommatecknen?

/,?([0-9]+),?/

Då måste du loopa MatchCollection från ditt Regex-objekt då varje tal kommer att vara en egen match.

Men, samma sak här.....
Strängen går att splitta på "," så får du en string-array med dina värden.

var userInputData = "111,222,333";
string[] splitResult = userInputData.Split(',');
//splitResult är nu en string-array med tre strängar.

Även detta är betydligt bättre än att använda regexp... förutsatt att jag förstod ditt problem korrekt.

Japp, allt är svårt innan man behärskar det. Regexp är ju en lösning som fungerar väldigt lika i olika språk. Därför kanske det känns lättare att använda det. Har du använt det i andra språk innan du började med C#?

Lite rolig läsning, om inte annat:
http://www.codinghorror.com/blog/2008/06/regular-expressions-...