Certamente alguém virá com uma solução de RegEx, que em tese é melhor odo que está fazendo (gera várias alocações de memória sem necessidade e pressiona o GC), mas pra mim fazer na mão é imbatível.
Usei o StringBuilder
para evitar múltiplas alocações. Eu converti para ToString()
no final e ali no WriteLine()
nem precisava porque ele faz automático, mas deixei porque no seu código precisará fazer a conversão.
A regra básica seria varrer toda a string e ir copiando cada caractere, poderia até usar LINQ, filtrando quando não for dígito e trocar pelo ;
(eu entendi que qualquer coisa que não seja dígito deveria converter, mas a pergunta não deixa isso tão claro). Até aí é fácil, o problema é que não pode repetir o ;
se ele já está presente. Então tem que montar uma máquina de estado.
Criei uma variável de controle que indica se o último caractere analisado é um dígito. Começo considerando que é.
Sempre que for dígito e o anterior não era um dígito deve concatenar o ;
, afinal se era um caractere irregular antes e agora vou começar colocar dígito, preciso marcar com o caractere especial escolhido. E obviamente que se é um dígito tenho que dizer que o atual é esse, pra no próximo passo o algoritmo saber o que fazer.
Quando não for um dígito é melhor fazer nada, afinal você tem que sumir com esse caractere, ele não pode ser mostrado. A única ação é identificar que agora não é um dígito na variável de controle.
O segredo é matar todo caractere inválido sem exceção, e antes que começar concatenar uma sequência de dígitos (pode até ser um único na sequência), colocar o caractere especial separador escolhido, mas nunca logo que começa.
Para criar uma máquina de estados tem que pensar como uma funcionaria. Que decisão tomar com cada caractere sendo analisada. É como um compilador funciona.
Se entendi a regra corretamente o código seria este (dá para encapsular em um método genérico:
using static System.Console;
using System.Text;
public class Program {
public static void Main() {
var hs = "/4587 / 5458/";
var texto = new StringBuilder(hs.Length);
var digito = true;
foreach (var chr in hs) {
if (char.IsDigit(chr)) {
if (!digito) texto.Append(';');
texto.Append(chr);
digito = true;
} else digito = false;
}
WriteLine(texto.ToString());
}
}
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
Talvez dê para otimizar ainda mais com o novo Span<T>
, mas não vou arriscar essa complicação para você.