?
Solved

Convert VBA Code to C# - How to find a phrase in a string and then parse everything to the left of that value?

Posted on 2015-02-02
7
Medium Priority
?
366 Views
Last Modified: 2015-02-03
Please note the following code looks for a phase/value called (Phone: ) and once it's found in a string grabs all of the text to the left of it and returns the value:

I'm a lot more familiar with VBA than C# and was wondering if someone could help me with the code below that would be it's equivalent.   MID function does a pretty good job of identifying text in a string if you give it a start value and LEFT I know C#'s equivalent is SubString... but if someone could convert the code to actual C# - it would be much appreciated.

I tried http://www.developerfusion.com/tools/convert/vb-to-csharp and they are still stumped and have not returned a reply.  I'm not trying to be guided to articles where I can read up on how to figure it out.  I'm doing other programming stuff - this is just a little nugget that I'm stuck on and would ideally like some code to try and see if it's doing the same thing in Visual C# .NET.  

Thank you in advance.

Public Function NoPhoneNumber(sValue as String)

Dim newvalue As String
Dim i as Integer

For i = 1 to len(sValue)

If len(sValue) > 8 Then

If mid(sValue, i, 9) = "(Phone: )" Then

  newvalue = Left(sValue, i-9)
  Exit For

End if

End if

Next i

NoPhoneNumber = sValue

End Sub

Open in new window

0
Comment
Question by:stephenlecomptejr
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 2
  • 2
7 Comments
 
LVL 23

Expert Comment

by:Michael Fowler
ID: 40585287
Here is code which should achieve what you are looking for

Public string NoPhoneNumber(string input)
{
   If (input.IndexOf("(Phone: )") == -1)
   {
      return "";
   }
   
   return input.Substring(0, input.Length - input.IndexOf("(Phone: )"));
}

Open in new window

0
 
LVL 86

Expert Comment

by:Mike Tomlinson
ID: 40585324
Another one:
        private string NoPhoneNumber(string phone)
        {
            return phone.Contains("(Phone: )") ? phone.Substring(0, phone.IndexOf("(Phone: )")) : "";
        }

Open in new window


That could also be written as:
        private string NoPhoneNumber(string phone)
        {
            int i;
            return (i = phone.IndexOf("(Phone: )")) != -1 ? phone.Substring(0, i) : "";
        }

Open in new window

0
 
LVL 1

Author Comment

by:stephenlecomptejr
ID: 40585365
Michael74

I have for my string input:  "GE Healthcare - Anesthesia Delivery  (Phone: ) CAD ID: ANE0034"
And  yet the result using your code comes out to :  "GE Healthcare - Anesthesi"

... so Anesthesia Delivery is cut off.
0
Windows Server 2016: All you need to know

Learn about Hyper-V features that increase functionality and usability of Microsoft Windows Server 2016. Also, throughout this eBook, you’ll find some basic PowerShell examples that will help you leverage the scripts in your environments!

 
LVL 1

Author Comment

by:stephenlecomptejr
ID: 40585369
Mike Tomlinson, that code worked but what if I want to trouble shoot each character in the string and see it getting to the point of 9 characters as it moves from the beginning character to the end... that's what the VBA enables me to see.

With the above code, I can't see the result as it goes through the process of finding it.  

I guess the benefit is that it's faster than looping through each character in the string - is that correct?
0
 
LVL 23

Assisted Solution

by:Michael Fowler
Michael Fowler earned 600 total points
ID: 40585380
Sorry about that. Use

Public string NoPhoneNumber(string input)
{
   If (input.IndexOf("(Phone: )") == -1)
   {
      return "";
   }
   
   return input.Substring(0, input.IndexOf("(Phone: )")-1).Trim()
}

Open in new window


Note: I added trim to strip the whitespace. If this was wrong just remove the Trim() function from the call
0
 
LVL 86

Accepted Solution

by:
Mike Tomlinson earned 1400 total points
ID: 40585564
"I guess the benefit is that it's faster than looping through each character in the string - is that correct?"

I'm not sure what the underlying code for IndexOf actually does.  It might be looping for you.

If you really want to loop, then:
        private string NoPhoneNumber(string phone)
        {
            string lookFor = "(Phone: )";
            if (phone.Length > lookFor.Length)
            {
                for(int i = 1; i <= (phone.Length - lookFor.Length); i++)
                {
                    if (phone.Substring(i, lookFor.Length).Equals(lookFor))
                    {
                        return phone.Substring(0, i);
                    }
                }
            }
            return "";
        }

Open in new window

0
 
LVL 1

Author Closing Comment

by:stephenlecomptejr
ID: 40586076
Appreciate all the replies.   Mike gets more points for taking the time to give me exactly what I wanted but both deserve 'A's.  Thank you sincerely.
0

Featured Post

Efficient way to get backups off site to Azure

This user guide provides instructions on how to deploy and configure both a StoneFly Scale Out NAS Enterprise Cloud Drive virtual machine and Veeam Cloud Connect in the Microsoft Azure Cloud.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This article describes two methods for creating a combo box that can be used to add new items to the row source -- one for simple lookup tables, and one for a more complex row source where the new item needs data for several fields.
Instead of error trapping or hard-coding for non-updateable fields when using QODBC, let VBA automatically disable them when forms open. This way, users can view but not change the data. Part 1 explained how to use schema tables to do this. Part 2 h…
What’s inside an Access Desktop Database. Will look at the basic interface, Navigation Pane (Database Container), Tables, Queries, Forms, Report, Macro’s, and VBA code.
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…
Suggested Courses

719 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question