Extracting double/decimal number from string

Question

3.00/5 (1 vote)

See more:

I need to extract the 33345.002 from the following string:

"ABC(MAX)(33345.002)"

How can I perform this in c#?
I tried handling it in SQL but was picking up the (MAX) too so now I'm gonna try c#.
Thanks

Posted 3-Nov-11 14:19pm

Mastersev

Add a Solution

Comments

hzawary 3-Nov-11 21:27pm

Solved the issue?

Mastersev 3-Nov-11 21:31pm

nope :(

hzawary 3-Nov-11 21:39pm

Did you try my solution!

Mastersev 3-Nov-11 21:41pm

string value = "White BOX, Inc. (037913.00)";
string[] parts = value.Split(new char[] {'(', ')'}, System.StringSplitOptions.RemoveEmptyEntries);
string numeric = parts[parts.Length-1]; //sic!
double d = double.Parse(numeric);

This returns a double ---> 37913
I need --> 037913.00

hzawary 3-Nov-11 21:46pm

While converted to double left zeros to be removed, so you must save your number in string before to convert it!

Mastersev 3-Nov-11 21:50pm

Ok I tried this and now I got 37913.00

return Convert.ToDecimal(numeric).ToString("#0.00");

Note: The left side is always 6 digits if less then zeros must be added to the front
The right side is always 2 digits

Mastersev 3-Nov-11 21:57pm

Looks like I got it:
string[] parts = value.Split(new char[] {'(', ')'}, System.StringSplitOptions.RemoveEmptyEntries);
string numeric = parts[parts.Length-1];
numeric = Convert.ToDecimal(numeric).ToString("#0.00");
while(numeric.Length < 9)
{
numeric = "0" + numeric;
}
return numeric;

LanFanNinja 3-Nov-11 22:21pm

Glad you figured it out sorry I could not have been more helpfull.

5 solutions

Solution 3

I would go for a regular expression.

You can find some informations here.

And there is a tool, Expresso, most useful to build your expressions and test them against as much entries as you want/need to.

Having a look at regular expressions is worth the additional learning time ; you will find they can be handy in many situations.

For your question, it would be (updated for variable lengths and any alphanumeric character) :

^(?:[\w]+$[\w]+$)$(?<NamedValue>[\d]+(?:\.[\d]+)?)$$

Which describes the chain as :
Begin of line - One or more letters, followed by one or more letters enclosed in parenthesis, followed by a named capture group enclosed in parenthesis. This named capture group is composed of one or more digits, followed or not by a point and one or more digits - End of line

You can obtain the string representation of your double in NamedValue.

You can convert it to double by using the method hzawary told you, but passing only the NamedValue you got. It can be great to use Double.TryParse() to handle problematic entries.

double d;
bool ok = double.TryParse(yourMatch.Group["NamedValue"].Value, out d);

Hope this helps.

Posted 3-Nov-11 14:54pm

phil.o

Updated 3-Nov-11 15:22pm

v6

Comments

phil.o 3-Nov-11 22:33pm

Thank you for the 1 vote Oo

Mastersev 4-Nov-11 15:31pm

I'll vote 5 because you helped me.

Solution 4

Try this it also uses a regex:

C#

string temp = "YYY(33345.002)(gg)YYYY";
temp = Regex.Replace(temp, "[^.0-9]", "");
double num;
bool success = Double.TryParse(temp, out num);

if (success)
{
    //do what ever to the number
}

Posted 3-Nov-11 15:05pm

LanFanNinja

Comments

Mastersev 3-Nov-11 21:11pm

Thanks Ninja your answer is the best answer so far. But there is something new I forgot to mention. Some of the numbers have zeros before them. i.e. 00333.33

They are IDs :(

phil.o 3-Nov-11 21:11pm

There would be a problem if there were to be a point anywhere else than in the numeric part.

Mastersev 3-Nov-11 21:14pm

Yea in case of "White BOX, Inc. (037913.00)" it does not work.

Solution 1

C#

double x = Convert.ToDouble("ABC(MAX)(33345.002)".Substring(9, 9));

Try this one and just you input the characters to remove!

C#

double x = Convert.ToDouble("ABC(MAX)(33345.002)".ToUpper().Trim(new char[] { 'A', 'B', 'C', 'M', 'X', '(', ')' }));

Posted 3-Nov-11 14:26pm

hzawary

Updated 3-Nov-11 15:20pm

v3

Comments

Mastersev 3-Nov-11 20:41pm

Sorry for the confusion but it may be "BADLands(O)(33345.002)". No fix length.

hzawary 3-Nov-11 20:47pm

is fixed "ABC(MAX)("?

Mastersev 3-Nov-11 20:48pm

no can be anything

hzawary 3-Nov-11 20:55pm

Ok! so you want double only!?

Mastersev 3-Nov-11 20:55pm

Double and int

Solution 2

C#

string[] MyString = Strings.Split("ABC(MAX)(33345.002)", "(");
MyString(2) = MyString(2).Remove(MyString(2).IndexOf(")"));
//Then call MyString(2)

Posted 3-Nov-11 14:47pm

WhiteKnightBRASIL

Updated 3-Nov-11 14:52pm

v3

Comments

Mastersev 3-Nov-11 20:49pm

What if it's "YYY(33345.002)(gg)YYYY"?

WhiteKnightBRASIL 3-Nov-11 20:52pm

There, it's updated ;)

WhiteKnightBRASIL 3-Nov-11 20:54pm

But you need to fixate where the data is. So in THAT example ("YYY(33345.002)(gg)YYYY") you need to use MyString(1) instead of MyString(2)

Mastersev 3-Nov-11 20:56pm

I have 130000 records I cannot check them one by one

WhiteKnightBRASIL 3-Nov-11 21:04pm

Will the numbers you need ALWAYS have a "(" before them?

Mastersev 3-Nov-11 21:06pm

Yes

Sergey Alexandrovich Kryukov 3-Nov-11 21:05pm

Sorry, White Knight, this won't even compile. Please, don't provide an answer if you are not sure. (And in this case you could be sure if you check it up in writing actual code; if I'm not 100% sure I always do it.) Also, using Split could make IndexOf (which is also less reliable) redundant.

So, I just had to vote 1, sorry.
--SA

WhiteKnightBRASIL 3-Nov-11 21:21pm

Yea about that... I don't have c# on this PC, so I had to use a browser vb.net, c# converter :/
Sorry!

Sergey Alexandrovich Kryukov 3-Nov-11 21:11pm

Please see the fixed solution I provided...
--SA

Mastersev 3-Nov-11 21:21pm

It just grabs the 37913 from "White BOX, Inc. (037913.00)"

WhiteKnightBRASIL 3-Nov-11 21:25pm

Thats because 037813.00 is the SAME as 037913 if it were a number/integer/decimal. If you want it to come with the .00 you need to use it as string, i think.

Mastersev 3-Nov-11 21:31pm

well in my case these are IDs. I think there is no way to achieve this :(

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Accepted Answer · 2011-11-03T15:10:00

Solution 5

Here is the corrected solution based on Split:

C#

static double ExtractFrom(string value) {
   string[] parts = value.Split(new char[] {'(', ')'}, System.StringSplitOptions.RemoveEmptyEntries);
   string numeric = parts[parts.Length-1]; //sic!
   return double.Parse(numeric);
}

See also the Regex-based solution by Phil. It can be correct, I did not test it.

—SA

Posted 3-Nov-11 15:10pm

Sergey Alexandrovich Kryukov

Updated 3-Nov-11 15:27pm

v2

Comments

hzawary 3-Nov-11 21:12pm

Dear SAKryukov, May you see... 'Why "this.DragMove()" hold up some of other "events"? Not Solved Yet!' http://www.codeproject.com/Questions/277094/Why-DragMove-hold-up-some-of-other-event

Sergey Alexandrovich Kryukov 3-Nov-11 21:21pm

Yes, I know the problem, thanks for reminding it to me. I did not try yet, but I recently tried DragMove -- it works. Mark Salsbery correctly mentioned that this API blocks some events (actually just consumes them), that's the problem. You should explain what else do you want to achieve as DragMove just works without isMouseDown. So, do you need anything else?
--SA

Mastersev 3-Nov-11 21:20pm

It just grabs the 37913 from "White BOX, Inc. (037913.00)"

Sergey Alexandrovich Kryukov 3-Nov-11 21:24pm

No! You did something wrong. It grabs the last term in brackets, which is 33345.002. Do you want me to test it? Did you add a separator '.', by any chance? Did you Parse by the method of double?
--SA

Sergey Alexandrovich Kryukov 3-Nov-11 21:31pm

No! I tried not to believe my eyes (who knows?) so I just tested it -- it works correctly.
Now, you try.

When you are convince yourself, please accept the solution formally (green button) -- thanks.
--SA

Mastersev 3-Nov-11 21:28pm

Code:
string value = "White BOX, Inc. (037913.00)";
string[] parts = value.Split(new char[] {'(', ')'}, System.StringSplitOptions.RemoveEmptyEntries);
string numeric = parts[parts.Length-1]; //sic!
double d = double.Parse(numeric);

Mastersev 3-Nov-11 21:29pm

it grabs the 37913

Sergey Alexandrovich Kryukov 3-Nov-11 21:33pm

Sorry, no. Every onecan try it and confirm.
--SA

phil.o 3-Nov-11 21:38pm

Normal : your number is 037913.00
When you convert it to double, depending on how you print your value, non significant digits can be pulled out.
Try value.ToString("#0.00");

Mastersev 3-Nov-11 21:44pm

string numeric = parts[parts.Length-1]; //sic!
return numeric.ToString("#0.00");

Error: Argument 1: cannot convert from 'string' to 'System.IFormatProvider'
The best overloaded method match for 'string.ToString(System.IFormatProvider)' has some invalid arguments

Mastersev 3-Nov-11 21:51pm

Ok I tried this and now I got 37913.00

return Convert.ToDecimal(numeric).ToString("#0.00");

Note: The left side is always 6 digits if less then zeros must be added to the front
The right side is always 2 digits

Mastersev 3-Nov-11 21:57pm

string[] parts = value.Split(new char[] {'(', ')'}, System.StringSplitOptions.RemoveEmptyEntries);
string numeric = parts[parts.Length-1];
numeric = Convert.ToDecimal(numeric).ToString("#0.00");
while(numeric.Length < 9)
{
numeric = "0" + numeric;
}
return numeric;

Mastersev 3-Nov-11 22:19pm

Many thanks to phil.o and Sakryukov

Sergey Alexandrovich Kryukov 3-Nov-11 22:26pm

You are welcome.
--SA

hzawary 3-Nov-11 22:28pm

While Moving it, the window do skipping to around!

in http://www.codeproject.com/Questions/277732/How-do-I-moving-an-irregularly-wpf-window-in-deskt

Sergey Alexandrovich Kryukov 3-Nov-11 23:07pm

Could you please go to that page and write your comments there? I answered to both of your questions related to dragging of WPF Window. Why are you trying to contaminate this page with unrelated comments?
--SA