|
Regex should be your friend.
Edit:
string filename = "123_456.txt" ;
System.Text.RegularExpressions.Regex reg = new System.Text.RegularExpressions.Regex
( @"^(?'ID0'\d+)_(?'ID1'\d+)\.(?'Ext'.+)$" ) ;
int id = System.Int32.Parse ( reg.Matches ( filename ) [ 0 ].Groups [ "ID1" ].Value ) ;
modified 12-Sep-14 13:51pm.
|
|
|
|
|
You'd think so, wouldn't you?
Surprisingly, that turns out not to be the case...
Counting Lines in a String[^]
Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...
|
|
|
|
|
I know.
|
|
|
|
|
I'm noticing your wording my friend.
|
|
|
|
|
Your length is off by one. The decimal point is included. I prefer int.TryParse over Convert.
|
|
|
|
|
And we have a winner.
Amazing that the decimal point is making the conversion more than 3000 times slower.
But since it didn't throw an exception but rather make a correct conversion I actually doubt that TryParse would have made a difference.
|
|
|
|
|
Jörgen Andersson wrote: doubt that TryParse would have made a difference.
Fewer puppies would have been killed. Don't use Convert.
|
|
|
|
|
If I don't expect the conversion to fail I want an exception.
When failure is expected I use TryParse.
There is a reason both exists.
|
|
|
|
|
Yes, in this case Parse is more appropriate. (I just updated my earlier response with an example.)
|
|
|
|
|
Jörgen Andersson wrote: But since it didn't throw an exception but rather make a correct conversion
What version of .NET? I've just tried Convert.ToInt32("42.") in LINQPad for 3.5 and 4.5, and both versions throw a FormatException .
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
|
Weird. If I try Convert.ToInt32("42.") in .NET 4.5, I get:
FormatException: Input string was not in a correct format.
Are you sure you're not doing something else in your code that's hiding the error?
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Odd, I'have to check on monday when I'm back at work.
|
|
|
|
|
Jörgen Andersson (is this you "Neo"?!?) sounds scandinavian and they probably have a different decimal separator...
Robert
|
|
|
|
|
Good point. However, Convert.ToInt32("42,") still produces a FormatException .
Trying "42." with the "sv-SE" culture, which uses "," as the decimal separator and "." as the group separator, also produces a FormatException .
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
3000 times slower? I noticed the extra character about 15 mins after you started this thread, but didn't think it relevant.
I'll have to keep an eye out for that... Almost as bad as the exponent operator in VB.NET... (Try benchmarking 2^2 vs Math.Pow(2,2))
|
|
|
|
|
hmm FileName.IndexOf('_') is calculated twice too
|
|
|
|
|
Well, a little late to the party, but this is why I have a whole bunch of string extensions, like this "Between" and "ToInt32":
FileName.Between('_', '.').ToInt32()
And then you would never have included the '.' by accident.
Marc
|
|
|
|
|
Just put that on my todo list.
|
|
|
|
|
Send me a direct email and I'll send you the .cs file of my extensions.
There's this[^] too.
Marc
|
|
|
|
|
very cool (aka )
'g'
|
|
|
|
|
Whereas the number of threads should be known in advance, the architecture of this application is so bad, that this check has been added before launching new threads, to avoid the ThreadPool to explode :
var count = Process.GetCurrentProcess().Threads.Count;
if (count > maxApplicationThreadCount) return;
Cheers
|
|
|
|
|
Well, having a lot of threads isn't necessarily a bad thing. It depends on what they're doing. If they're sitting around waiting for some async process to complete, then should be OK. But if they're all competing for CPU time doing heavy processing, then yeah, that's not good.
Marc
|
|
|
|
|
Also nice to note that it prevents the creation of new threads only after the maximum number has been exceeded...
|
|
|
|
|
...which also implies that there is no feedback to the calling function whether the new thread has been started, or not. Interesting, at least.
~RaGE();
I think words like 'destiny' are a way of trying to find order where none exists. - Christian Graus
Entropy isn't what it used to.
|
|
|
|