删除字符串的最佳方法是什么？

Question

I need ideas with the best performance to remove/filter strings

I have:

string Input = "view('512', 3, 159);";

What's the best performance way to remove "view(" and ");" and the quotes? I can do this:

Input = Input.Replace("view(","").Replace("'","").Replace("\"","").Replace(");",""); 

but it seems rather inelegant.

Input.Split('(')[1].Split(')')[0].Replace("'", "");

it seems rather better

I want no do it by using regular expression; I need make the faster application what I can. Thanks in advance: :)

Answer 1

You could use a simple linq statement:

string Input = "view('512', 3, 159);";

string output = new String( Input.Where( c => Char.IsDigit( c ) || c == ',' ).ToArray() );

Output: 512,3,159

If you want the spaces, just add a check in the where clause.

Answer 2

You could use just a Substring to remove the view( and ); :

Input.Substring(5, Input.Length - 7)

Other than that it looks reasonably efficient. Plain string operations are pretty well optimised.

So:

Input =
  Input.Substring(5, Input.Length - 7)
  .Replace("'", String.Empty)
  .Replace("\"", String.Enmpty);

Answer 3

char[] Output = Input.SkipWhile(x => x != '(') // skip before open paren
                     .Skip(1)                  // skip open paren
                     .TakeWhile(x => x != ')') // take everything until close paren
                     .Where(x => x != '\'' && x != '\"') // except quotes
                     .ToArray();
return new String(Output);

Answer 4

Hope this helps

Regex.Replace("view('512', 3, 159);",@"[(view)';]","")

Answer 5

Use the following:

            System.Text.StringBuilder sb=new System.Text.StringBuilder();
        int state=0;
        for(var i=0;i<Input.Length;i++){
            switch(state){
                case 0: // beginning
                    if(Input[i]=='('){
                        state=1; // seen left parenthesis
                    }
                    break;
                case 2: // seen end parentheses
                    break; // ignore
                case 1:
                    if(Input[i]==')'){
                        state=2; // seen right parentheses
                    } else if(Input[i]!='\''){

                        sb.Append(Input[i]);
                    }
                    break;
            }
        }
        Console.WriteLine(sb.ToString());

Answer 6

IndexOf, LastIndexOf, and Substring are probably fastest.

string Input = "view('512', 3, 159);"; 
int p1 = Input.IndexOf('(');
int p2 = Input.LastIndexOf(')');
Input = Input.Substring (p1 + 1, p2 - p1 - 1);

Answer 7

    var result = new string(Input.ToCharArray().
SkipWhile (i => i != '\'').
TakeWhile (i => i != ')').ToArray());

Answer 8

Why don't you want to use regular expressions? Regular expressions are heavily optimised and will be much faster than any hand written hack.

This is java (as I run linux and can't run c# as a result), but I hope you get the idea.

input.replace("view(","").replace("'","").replace("\"","").replace(");",""); 

A million repetitions of the above runs in about 6 seconds on my computer. Whereas, the regular expression below runs in about 2 seconds.

// create java's regex matcher object
// matcher is looking for sequences of digits (valid integers)
Matcher matcher = Pattern.compile("(\\d+)").matcher(s);
StringBuilder builder = new StringBuilder();
// whilst we can find matches append the match plus a comma to a string builder
while (matcher.find()) {
    builder.append(matcher.group()).append(',');
}
// return the built string less the last trailing comma
return builder.substring(0, builder.length()-1);

If you want to find valid decimals as well as integers then use the following pattern instead. Though it runs slightly slower than the original.

"(\\d+(\\.\\d*)?)"

Answer 9

fastest way would be Input = Input.Substring(5, Input.Length - 7)

Answer 10

More generic

void Main()
{
    string Input = "view('512', 3, 159);";
    var statingPoint = Input.IndexOf('(') + 1;

    var result = Input.Substring(statingPoint, Input.IndexOf(')') - statingPoint);
}