Remover caracteres da string C #


150

Como posso remover caracteres de uma string? Por exemplo: "My name @is ,Wan.;'; Wan".

Gostaria de remover os caracteres '@', ',', '.', ';', '\''dessa string para que ela se torne"My name is Wan Wan"

Respostas:


177
var str = "My name @is ,Wan.;'; Wan";
var charsToRemove = new string[] { "@", ",", ".", ";", "'" };
foreach (var c in charsToRemove)
{
    str = str.Replace(c, string.Empty);
}

Mas posso sugerir outra abordagem, se você deseja remover todos os caracteres que não sejam letras

var str = "My name @is ,Wan.;'; Wan";
str = new string((from c in str
                  where char.IsWhiteSpace(c) || char.IsLetterOrDigit(c)
                  select c
       ).ToArray());

12
Também pode ser feito assim: str = new string (str.Where (x => char.IsWhiteSpace (x) || char.IsLetterOrDigit (x)). ToArray ());
Adnan Bhatti

1
Eu tive que procurar isso, string.Empty não cria uma string para a comparação, portanto é mais eficiente que "". ( stackoverflow.com/questions/151472/… )
Tom Cerul 15/10

6
Eu sou o único que recebe "Argumento 2: não é possível converter de 'string' para 'char'" om string.Empty?
OddDev 25/10

2
@OddDev, você só deve receber esse erro se sua matriz que você percorrer for uma lista de caracteres. Se eles são strings isso deve funcionar
Newteq desenvolvedor

3
Além disso, observe que, para que a função "str.Replace" funcione corretamente, o primeiro parâmetro deve ser uma "string" se você quiser usar string.Empty como o segundo parâmetro. Se você usar um caractere (ou seja, 'a') como o primeiro parâmetro, também precisará de um caractere como o segundo. Caso contrário, você obterá o "Argumento 2: Não é possível converter de 'string' para 'char'" erro mencionado por @OddDev acima
Leo

68

Simples:

String.Join("", "My name @is ,Wan.;'; Wan".Split('@', ',' ,'.' ,';', '\''));

64

Parece um aplicativo ideal para o RegEx - um mecanismo projetado para manipulação rápida de texto. Nesse caso:

Regex.Replace("He\"ll,o Wo'r.ld", "[@,\\.\";'\\\\]", string.Empty)

3
Parece que isso seria muito mais eficiente do que uma abordagem baseada em iterador, especialmente se você puder usar um Regex compilado;
Ade Miller

Essa deve ser a resposta aceita, especialmente porque, como o @AdeMiller disse, será muito mais eficiente.
Obsidian

14
Isso não é mais rápido que o loop, é um equívoco comum que os regex sejam sempre mais rápidos que os loops. As regexs não são mágicas; elas devem, em algum momento, percorrer a string para executar suas operações, e podem ser muito mais lentas com as despesas gerais da própria regex. Eles realmente se destacam quando se trata de manipulação extremamente complexa, onde dezenas de linhas de código e vários loops seriam necessários. Testando a versão compilada desse regex em um loop simples e não otimizado 50000 vezes, o regex é 6X mais lento.
Tony Cheetham

E quanto à eficiência da memória? As expressões regulares não serão mais eficientes no sentido de novas alocações de strings?
Marek19:

2
Talvez eu tenha me enganado quando afirmei que o RegEx é rápido. A menos que isso esteja no centro de um loop muito apertado, outras considerações, como legibilidade e facilidade de manutenção, provavelmente dominam o desempenho de uma pequena operação como essa.
John Melville

21

Menos específico para sua pergunta, é possível remover a pontuação ALL de uma sequência de caracteres (exceto o espaço) listando os caracteres aceitáveis ​​em uma expressão regular:

string dirty = "My name @is ,Wan.;'; Wan";

// only space, capital A-Z, lowercase a-z, and digits 0-9 are allowed in the string
string clean = Regex.Replace(dirty, "[^A-Za-z0-9 ]", "");

Observe que há um espaço após esse 9 para não remover espaços da sua frase. O terceiro argumento é uma string vazia que serve para substituir qualquer substring que não pertence à expressão regular.


19

Comparando várias sugestões (bem como comparando no contexto de substituições de caracteres únicos com vários tamanhos e posições do alvo).

Nesse caso em particular, a divisão nos destinos e a junção nas substituições (nesse caso, cadeia vazia) é a mais rápida em pelo menos um fator de 3. Por fim, o desempenho é diferente dependendo do número de substituições, em que as substituições estão a fonte e o tamanho da fonte. #ymmv

Resultados

(resultados completos aqui )

| Test                      | Compare | Elapsed                                                            |
|---------------------------|---------|--------------------------------------------------------------------|
| SplitJoin                 | 1.00x   | 29023 ticks elapsed (2.9023 ms) [in 10K reps, 0.00029023 ms per]   |
| Replace                   | 2.77x   | 80295 ticks elapsed (8.0295 ms) [in 10K reps, 0.00080295 ms per]   |
| RegexCompiled             | 5.27x   | 152869 ticks elapsed (15.2869 ms) [in 10K reps, 0.00152869 ms per] |
| LinqSplit                 | 5.43x   | 157580 ticks elapsed (15.758 ms) [in 10K reps, 0.0015758 ms per]   |
| Regex, Uncompiled         | 5.85x   | 169667 ticks elapsed (16.9667 ms) [in 10K reps, 0.00169667 ms per] |
| Regex                     | 6.81x   | 197551 ticks elapsed (19.7551 ms) [in 10K reps, 0.00197551 ms per] |
| RegexCompiled Insensitive | 7.33x   | 212789 ticks elapsed (21.2789 ms) [in 10K reps, 0.00212789 ms per] |
| Regex Insentive           | 7.52x   | 218164 ticks elapsed (21.8164 ms) [in 10K reps, 0.00218164 ms per] |

Arnês de teste (LinqPad)

(nota: as extensões de tempoPerf e Vssão que escrevi )

void test(string title, string sample, string target, string replacement) {
    var targets = target.ToCharArray();

    var tox = "[" + target + "]";
    var x = new Regex(tox);
    var xc = new Regex(tox, RegexOptions.Compiled);
    var xci = new Regex(tox, RegexOptions.Compiled | RegexOptions.IgnoreCase);

    // no, don't dump the results
    var p = new Perf/*<string>*/();
        p.Add(string.Join(" ", title, "Replace"), n => targets.Aggregate(sample, (res, curr) => res.Replace(new string(curr, 1), replacement)));
        p.Add(string.Join(" ", title, "SplitJoin"), n => String.Join(replacement, sample.Split(targets)));
        p.Add(string.Join(" ", title, "LinqSplit"), n => String.Concat(sample.Select(c => targets.Contains(c) ? replacement : new string(c, 1))));
        p.Add(string.Join(" ", title, "Regex"), n => Regex.Replace(sample, tox, replacement));
        p.Add(string.Join(" ", title, "Regex Insentive"), n => Regex.Replace(sample, tox, replacement, RegexOptions.IgnoreCase));
        p.Add(string.Join(" ", title, "Regex, Uncompiled"), n => x.Replace(sample, replacement));
        p.Add(string.Join(" ", title, "RegexCompiled"), n => xc.Replace(sample, replacement));
        p.Add(string.Join(" ", title, "RegexCompiled Insensitive"), n => xci.Replace(sample, replacement));

    var trunc = 40;
    var header = sample.Length > trunc ? sample.Substring(0, trunc) + "..." : sample;

    p.Vs(header);
}

void Main()
{
    // also see /programming/7411438/remove-characters-from-c-sharp-string

    "Control".Perf(n => { var s = "*"; });


    var text = "My name @is ,Wan.;'; Wan";
    var clean = new[] { '@', ',', '.', ';', '\'' };

    test("stackoverflow", text, string.Concat(clean), string.Empty);


    var target = "o";
    var f = "x";
    var replacement = "1";

    var fillers = new Dictionary<string, string> {
        { "short", new String(f[0], 10) },
        { "med", new String(f[0], 300) },
        { "long", new String(f[0], 1000) },
        { "huge", new String(f[0], 10000) }
    };

    var formats = new Dictionary<string, string> {
        { "start", "{0}{1}{1}" },
        { "middle", "{1}{0}{1}" },
        { "end", "{1}{1}{0}" }
    };

    foreach(var filler in fillers)
    foreach(var format in formats) {
        var title = string.Join("-", filler.Key, format.Key);
        var sample = string.Format(format.Value, target, filler.Value);

        test(title, sample, target, replacement);
    }
}

1
Finalmente alguns números! Bom trabalho @drzaus!
Marek



6

Outra solução simples:

var forbiddenChars = @"@,.;'".ToCharArray();
var dirty = "My name @is ,Wan.;'; Wan";
var clean = new string(dirty.Where(c => !forbiddenChars.Contains(c)).ToArray());

5
new List<string> { "@", ",", ".", ";", "'" }.ForEach(m => str = str.Replace(m, ""));

4

Uma string é apenas uma matriz de caracteres; portanto, use o Linq para substituir (semelhante ao Albin acima, exceto que usa uma instrução linq contém para fazer a substituição):

var resultString = new string(
        (from ch in "My name @is ,Wan.;'; Wan"
         where ! @"@,.;\'".Contains(ch)
         select ch).ToArray());

A primeira string é a string para substituir os caracteres e a segunda é uma string simples que contém os caracteres


A solução Linq da Albin é provavelmente melhor, a menos que haja caracteres adicionais que você deseja filtrar (não cobertos por espaços em branco, letras e dígitos).
alistair

3

Eu poderia muito bem jogar isso aqui.

Faça uma extensão para remover caracteres de uma sequência:

public static string RemoveChars(this string input, params char[] chars)
{
    var sb = new StringBuilder();
    for (int i = 0; i < input.Length; i++)
    {
        if (!chars.Contains(input[i]))
            sb.Append(input[i]);
    }
    return sb.ToString();
}

E é utilizável assim:

string str = "My name @is ,Wan.;'; Wan";
string cleanedUpString = str.RemoveChars('@', ',', '.', ';', '\'');

Ou assim:

string str = "My name @is ,Wan.;'; Wan".RemoveChars('@', ',', '.', ';', '\'');

Essa é a melhor solução, pois gera o menor número de alocações de memória. Eu também definiria o comprimento da string original como a capacidade inicial do construtor de strings, como: new StringBuilder (input.Length) com esse objetivo de ter o menor número de alocações de memória.
treaschf 27/04

3

Parece que o caminho mais curto é combinar o LINQ e string.Concat:

var input = @"My name @is ,Wan.;'; Wan";
var chrs = new[] {'@', ',', '.', ';', '\''};
var result = string.Concat(input.Where(c => !chrs.Contains(c)));
// => result = "My name is Wan Wan" 

Veja a demonstração do C # . Observe que string.Concaté um atalho parastring.Join("", ...) .

Observe que ainda é possível criar dinamicamente o uso de um regex para remover caracteres conhecidos individuais, embora se acredite que o regex seja mais lento. No entanto, aqui está uma maneira de criar um regex dinâmico (onde tudo que você precisa é de uma classe de personagem):

var pattern = $"[{Regex.Escape(new string(chrs))}]+";
var result = Regex.Replace(input, pattern, string.Empty);

Veja outra demonstração em C # . A regex será semelhante [@,\.;']+(uma correspondência ou mais ( +) ocorrências consecutivas @, ,, ., ;ou 'caracteres) onde o ponto não tem que ser escapado, mas Regex.Escapeserão necessários para escapar outros caracteres que devem ser escaparam, como \, ^, ]ou -cuja posição dentro da classe de personagem que você não pode prever.



3

Aqui está um método que escrevi que tem uma abordagem um pouco diferente. Em vez de especificar os caracteres a serem removidos, digo ao meu método quais caracteres quero manter - ele removerá todos os outros caracteres.

No exemplo do OP, ele só quer manter caracteres e espaços alfabéticos. Aqui está a aparência de uma chamada para o meu método ( demonstração de C # ):

var str = "My name @is ,Wan.;'; Wan";

// "My name is Wan Wan"
var result = RemoveExcept(str, alphas: true, spaces: true);

Aqui está o meu método:

/// <summary>
/// Returns a copy of the original string containing only the set of whitelisted characters.
/// </summary>
/// <param name="value">The string that will be copied and scrubbed.</param>
/// <param name="alphas">If true, all alphabetical characters (a-zA-Z) will be preserved; otherwise, they will be removed.</param>
/// <param name="numerics">If true, all alphabetical characters (a-zA-Z) will be preserved; otherwise, they will be removed.</param>
/// <param name="dashes">If true, all alphabetical characters (a-zA-Z) will be preserved; otherwise, they will be removed.</param>
/// <param name="underlines">If true, all alphabetical characters (a-zA-Z) will be preserved; otherwise, they will be removed.</param>
/// <param name="spaces">If true, all alphabetical characters (a-zA-Z) will be preserved; otherwise, they will be removed.</param>
/// <param name="periods">If true, all decimal characters (".") will be preserved; otherwise, they will be removed.</param>
public static string RemoveExcept(string value, bool alphas = false, bool numerics = false, bool dashes = false, bool underlines = false, bool spaces = false, bool periods = false) {
    if (string.IsNullOrWhiteSpace(value)) return value;
    if (new[] { alphas, numerics, dashes, underlines, spaces, periods }.All(x => x == false)) return value;

    var whitelistChars = new HashSet<char>(string.Concat(
        alphas ? "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ" : "",
        numerics ? "0123456789" : "",
        dashes ? "-" : "",
        underlines ? "_" : "",
        periods ? "." : "",
        spaces ? " " : ""
    ).ToCharArray());

    var scrubbedValue = value.Aggregate(new StringBuilder(), (sb, @char) => {
        if (whitelistChars.Contains(@char)) sb.Append(@char);
        return sb;
    }).ToString();

    return scrubbedValue;
}

Resposta incrível!
edtheprogrammerguy

Muito agradável! a sequência numérica tem 0 duas vezes.
John Kurtz

@JohnKurtz Boa captura - já se foi.
Mass Dot Net

2

Muitas respostas boas aqui, aqui está minha adição, juntamente com vários testes de unidade que podem ser usados ​​para ajudar a testar a correção, minha solução é semelhante à do @ Rianne acima, mas usa um ISet para fornecer tempo de pesquisa O (1) nos caracteres de substituição (e também semelhante à solução Linq do @Albin Sunnanbo).

    using System;
    using System.Collections.Generic;
    using System.Linq;

    /// <summary>
    /// Returns a string with the specified characters removed.
    /// </summary>
    /// <param name="source">The string to filter.</param>
    /// <param name="removeCharacters">The characters to remove.</param>
    /// <returns>A new <see cref="System.String"/> with the specified characters removed.</returns>
    public static string Remove(this string source, IEnumerable<char> removeCharacters)
    {
        if (source == null)
        {
            throw new  ArgumentNullException("source");
        }

        if (removeCharacters == null)
        {
            throw new ArgumentNullException("removeCharacters");
        }

        // First see if we were given a collection that supports ISet
        ISet<char> replaceChars = removeCharacters as ISet<char>;

        if (replaceChars == null)
        {
            replaceChars = new HashSet<char>(removeCharacters);
        }

        IEnumerable<char> filtered = source.Where(currentChar => !replaceChars.Contains(currentChar));

        return new string(filtered.ToArray());
    }

NUnit (2.6+) testa aqui

using System;
using System.Collections;
using System.Collections.Generic;
using NUnit.Framework;

[TestFixture]
public class StringExtensionMethodsTests
{
    [TestCaseSource(typeof(StringExtensionMethodsTests_Remove_Tests))]
    public void Remove(string targetString, IEnumerable<char> removeCharacters, string expected)
    {
        string actual = StringExtensionMethods.Remove(targetString, removeCharacters);

        Assert.That(actual, Is.EqualTo(expected));
    }

    [TestCaseSource(typeof(StringExtensionMethodsTests_Remove_ParameterValidation_Tests))]
    public void Remove_ParameterValidation(string targetString, IEnumerable<char> removeCharacters)
    {
        Assert.Throws<ArgumentNullException>(() => StringExtensionMethods.Remove(targetString, removeCharacters));
    }
}

internal class StringExtensionMethodsTests_Remove_Tests : IEnumerable
{
    public IEnumerator GetEnumerator()
    {
        yield return new TestCaseData("My name @is ,Wan.;'; Wan", new char[] { '@', ',', '.', ';', '\'' }, "My name is Wan Wan").SetName("StringUsingCharArray");
        yield return new TestCaseData("My name @is ,Wan.;'; Wan", new HashSet<char> { '@', ',', '.', ';', '\'' }, "My name is Wan Wan").SetName("StringUsingISetCollection");
        yield return new TestCaseData(string.Empty, new char[1], string.Empty).SetName("EmptyStringNoReplacementCharactersYieldsEmptyString");
        yield return new TestCaseData(string.Empty, new char[] { 'A', 'B', 'C' }, string.Empty).SetName("EmptyStringReplacementCharsYieldsEmptyString");
        yield return new TestCaseData("No replacement characters", new char[1], "No replacement characters").SetName("StringNoReplacementCharactersYieldsString");
        yield return new TestCaseData("No characters will be replaced", new char[] { 'Z' }, "No characters will be replaced").SetName("StringNonExistantReplacementCharactersYieldsString");
        yield return new TestCaseData("AaBbCc", new char[] { 'a', 'C' }, "ABbc").SetName("CaseSensitivityReplacements");
        yield return new TestCaseData("ABC", new char[] { 'A', 'B', 'C' }, string.Empty).SetName("AllCharactersRemoved");
        yield return new TestCaseData("AABBBBBBCC", new char[] { 'A', 'B', 'C' }, string.Empty).SetName("AllCharactersRemovedMultiple");
        yield return new TestCaseData("Test That They Didn't Attempt To Use .Except() which returns distinct characters", new char[] { '(', ')' }, "Test That They Didn't Attempt To Use .Except which returns distinct characters").SetName("ValidateTheStringIsNotJustDistinctCharacters");
    }
}

internal class StringExtensionMethodsTests_Remove_ParameterValidation_Tests : IEnumerable
{
    public IEnumerator GetEnumerator()
    {
        yield return new TestCaseData(null, null);
        yield return new TestCaseData("valid string", null);
        yield return new TestCaseData(null, new char[1]);
    }
}

2

É um método poderoso que eu costumo usar no mesmo caso:

private string Normalize(string text)
{
        return string.Join("",
            from ch in text
            where char.IsLetterOrDigit(ch) || char.IsWhiteSpace(ch)
            select ch);
}

Aproveitar...


1

Old School no local copy / stomp:

  private static string RemoveDirtyCharsFromString(string in_string)
     {
        int index = 0;
        int removed = 0;

        byte[] in_array = Encoding.UTF8.GetBytes(in_string);

        foreach (byte element in in_array)
        {
           if ((element == ' ') ||
               (element == '-') ||
               (element == ':'))
           {
              removed++;
           }
           else
           {
              in_array[index] = element;
              index++;
           }
        }

        Array.Resize<byte>(ref in_array, (in_array.Length - removed));
        return(System.Text.Encoding.UTF8.GetString(in_array, 0, in_array.Length));
     }

Não tenho certeza sobre a eficiência de outros métodos (ou seja, a sobrecarga de todas as chamadas de função e instanciações que ocorrem como efeito colateral na execução de C #).


1

Eu faço isso método de extensão e com matriz de seqüências de caracteres, acho que string[]é mais útil do que char[]porque char também pode ser string:

public static class Helper
{
    public static string RemoverStrs(this string str, string[] removeStrs)
    {
        foreach (var removeStr in removeStrs)
            str = str.Replace(removeStr, "");
        return str;
    }
}

então você pode usá-lo em qualquer lugar:

string myname = "My name @is ,Wan.;'; Wan";
string result = myname.RemoveStrs(new[]{ "@", ",", ".", ";", "\\"});

1

Eu precisava remover caracteres especiais de um arquivo XML. Aqui está como eu fiz isso. char.ToString () é o herói neste código.

string item = "<item type="line" />"
char DC4 = (char)0x14;
string fixed = item.Replace(DC4.ToString(), string.Empty);

1
new[] { ',', '.', ';', '\'', '@' }
.Aggregate("My name @is ,Wan.;'; Wan", (s, c) => s.Replace(c.ToString(), string.Empty)); 

1

Tomando os números de desempenho do @drzaus, aqui está um método de extensão que usa o algoritmo mais rápido.

public static class StringEx
{
    public static string RemoveCharacters(this string s, params char[] unwantedCharacters) 
        => s == null ? null : string.Join(string.Empty, s.Split(unwantedCharacters));
}

Uso

var name = "edward woodward!";
var removeDs = name.RemoveCharacters('d', '!');
Assert.Equal("ewar woowar", removeDs); // old joke
Ao utilizar nosso site, você reconhece que leu e compreendeu nossa Política de Cookies e nossa Política de Privacidade.
Licensed under cc by-sa 3.0 with attribution required.