Obter Substring - tudo antes de determinado caractere

Estou tentando descobrir a melhor maneira de obter tudo antes do caractere - em uma string. Alguns exemplos de strings estão abaixo. O comprimento da corda anterior - varia e pode ter qualquer comprimento

223232-1.jpg
443-2.jpg
34443553-5.jpg

então eu preciso do valor que é do índice inicial de 0 para a direita antes de -. Portanto, as substrings seriam 223232, 443 e 34443553

Exemplo de .Net Fiddle

class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("223232-1.jpg".GetUntilOrEmpty());
        Console.WriteLine("443-2.jpg".GetUntilOrEmpty());
        Console.WriteLine("34443553-5.jpg".GetUntilOrEmpty());

        Console.ReadKey();
    }
}

static class Helper
{
    public static string GetUntilOrEmpty(this string text, string stopAt = "-")
    {
        if (!String.IsNullOrWhiteSpace(text))
        {
            int charLocation = text.IndexOf(stopAt, StringComparison.Ordinal);

            if (charLocation > 0)
            {
                return text.Substring(0, charLocation);
            }
        }

        return String.Empty;
    }
}

Resultados:

223232
443
34443553
344

34

Use a função de divisão .

static void Main(string[] args)
{
    string s = "223232-1.jpg";
    Console.WriteLine(s.Split('-')[0]);
    s = "443-2.jpg";
    Console.WriteLine(s.Split('-')[0]);
    s = "34443553-5.jpg";
    Console.WriteLine(s.Split('-')[0]);

Console.ReadKey();
}

Se sua string não tiver um -, você receberá toda a string.

String str = "223232-1.jpg"
int index = str.IndexOf('-');
if(index > 0) {
    return str.Substring(0, index)
}

As coisas mudaram um pouco desde o início desta discussão.

Agora você pode usar

string.Concat(s.TakeWhile((c) => c != '-'));

Uma maneira de fazer isso é usar String.Substringjunto com String.IndexOf:

int index = str.IndexOf('-');
string sub;
if (index >= 0)
{
    sub = str.Substring(0, index);
}
else
{
    sub = ... // handle strings without the dash
}

Começando na posição 0, retorne todo o texto até, mas não incluindo, o traço.

Com base na resposta do BrainCore:

    int index = 0;   
    str = "223232-1.jpg";

    //Assuming we trust str isn't null 
    if (str.Contains('-') == "true")
    {
      int index = str.IndexOf('-');
    }

    if(index > 0) {
        return str.Substring(0, index);
    }
    else {
       return str;
    }

Você pode usar expressões regulares para esse fim, mas é bom evitar exceções extras quando a string de entrada não corresponder à expressão regular.

Primeiro, para evitar dores de cabeça extras ao escapar para o padrão regex - poderíamos simplesmente usar a função para esse fim:

String reStrEnding = Regex.Escape("-");

Eu sei que isso não faz nada - como "-" é o mesmo que Regex.Escape("=") == "=", mas fará diferença, por exemplo, se o personagem é @"\".

Então, precisamos corresponder do início da string ao final da string ou, alternativamente, se o final não for encontrado - não corresponda a nada. (String vazia)

Regex re = new Regex("^(.*?)" + reStrEnding);

Se o seu aplicativo for crítico em termos de desempenho - separe uma linha para o novo Regex, se não estiver - você poderá ter tudo em uma linha.

E, finalmente, combine com a sequência e extraia o padrão correspondente:

String matched = re.Match(str).Groups[1].ToString();

Depois disso, você pode escrever uma função separada, como foi feita em outra resposta, ou escrever uma função lambda embutida. Eu escrevi agora usando ambas as notações - função lambda inline (não permite parâmetro padrão) ou chamada de função separada.

using System;
using System.Text.RegularExpressions;

static class Helper
{
    public static string GetUntilOrEmpty(this string text, string stopAt = "-")
    {
        return new Regex("^(.*?)" + Regex.Escape(stopAt)).Match(text).Groups[1].Value;
    }
}

class Program
{
    static void Main(string[] args)
    {
        Regex re = new Regex("^(.*?)-");
        Func<String, String> untilSlash = (s) => { return re.Match(s).Groups[1].ToString(); };

        Console.WriteLine(untilSlash("223232-1.jpg"));
        Console.WriteLine(untilSlash("443-2.jpg"));
        Console.WriteLine(untilSlash("34443553-5.jpg"));
        Console.WriteLine(untilSlash("noEnding(will result in empty string)"));
        Console.WriteLine(untilSlash(""));
        // Throws exception: Console.WriteLine(untilSlash(null));

        Console.WriteLine("443-2.jpg".GetUntilOrEmpty());
    }
}

Btw - alterar o padrão regex para "^(.*?)(-|$)"permitirá pegar até o "-"padrão ou se o padrão não foi encontrado - pegar tudo até o final da string.

A maneira LINQy

String.Concat ("223232-1.jpg" .TakeWhile (c => c! = '-'))

(Mas você precisa testar se é nulo;)