[Решено] Прочитать Nullable<long> из строки

А что именно нужно пояснить? код в целом и без комментариев понятен, единственное что я бы оставил комментарий про распознавание в первых цифрах до пробела и ещё что за входные данные такие странные потому что не ясно почему именно так они парсятся. Ну и внешний if не имеет смысла, т.к. и без него будет работать так же. З.ы. ветка с присвоением null также не нужна, т.к. в начале функции переменная уже проинициализирована null-ом

Да вроде всё и так понятно. Даже мне, который на C# не кодит.

А что именно нужно пояснить?

Нужно сделать так, чтоб не пришлось пояснять. Чтобы код читаемым был. Или так сойдет?

Ну и внешний if не имеет смысла, т.к. и без него будет работать так же.

Как это? Если val == null, а я вызываю val.Split()?

long? catCode = null;
if (!(String.IsNullOrWhiteSpace(val)))
{
    // распознаем в первых цифрах до пробела целое 64-битное число.
    if (Int64.TryParse(val.Split(" ")[0], out long cc))
    {
        catCode = cc;
    }
}

А в шарп не завезли какой-нибудь аналог оператора «?.»

Этот? Есть.

Не, такое https://stackoverflow.com/questions/4390141/java-operator-for-checking-null-what-is-it-not-ternary позволяет компактнее записывать вот эти все проверки на null

К сожалению нольпойнтерофобия создателей дотнета сильно загаживает подобные трансформации. Я давно не трогал дотнет, но сделал бы как-то так.

public static string LeftDigits(this string str)
{
    if (str == null) return null;
    if (!str.contains(" ") return null;
    string left = str.Split(" ")[0];
    if (! left.All(Char.IsDigit){
        return null;
    }
    return left;
}

//... 

string digits = LeftDigits(val);
long? catCode = digits != null ? Int64.Parse(digits) : null;

Но я не шарпей, может там чего адекватного подвезли. Ну и я бы весь парсинг отдельно в метод унёс.

Пояснил комментариями

Структурируй код так, чтобы не надо было комментировать. Я не умею в шарп, но в целом нужно сделать так

Это отдельная функция, например getValues

val.Split(" ")

Это тоже отдельная функция, например getFirst

[0]

Ну и это отдельная функция, например parseLongElseNull

Int64.TryParse

Итого должно получиться типа

catCode = parseLongElseNull(getFirst(getValues(val)))

Если этот твой long имеет какой-то конкретный смысл, например если это длина (чего-то), то вместо getFirst назови getWidth и т.п.

В современном C# для таких вещей надо спаны использовать.

public static long? ReadLong(string str)
{
	ReadOnlySpan<char> span = str.AsSpan();
	int spaceIndex = span.IndexOf(' ');
	ReadOnlySpan<char> slice = span.Slice(0, spaceIndex == -1 ? span.Length : spaceIndex);
	return long.TryParse(slice, out long number) ? number : null;
}

Есть конечно. Только, где его здесь применить?

Это не оно

По твоей ссылке в вопросе именно оно спрашивается. Но null-conditional operator тоже есть.

Во, теперь оно

И как его здесь применить?

ya-betmen, no-such-file, amm, но это же всё еще страшнее, чем было в начале.

В итоге сделал так.

long? catCode = null;
if (!(String.IsNullOrWhiteSpace(val)))
{
    // распознаем в первых цифрах до пробела целое 64-битное число.
    if (Int64.TryParse(val.Split(" ")[0], out long cc))
    {
        catCode = cc;
    }
}

Я б тебя выгнал за необучаемость.

А почему у вас тут одни токсики на форуме?

Для кода «страшнее» это не про длину а про понятность. А ты даже комент решил оставить в качестве социального дезодоранта.

ЗЫ. А за иф внутри ифа бесполезный можно руки по задницу отрывать.

Сделай бенчмарки, потом скажешь, что страшно, а что нет.

Внешний if() убрать

long? catCode = null;
// распознаем в первых цифрах до пробела целое 64-битное число.
if (Int64.TryParse(val?.Split(" ")?[0], out long cc))
{
        catCode = cc;
}

И зачем лишний раз TryParse вызывать?

А в чем проблема?