Как написать linq выражение?

Question

[kek123]

Есть строковый список со значениями "Нет", "20000", "30000", "45000", "50000", "75000"
Нужно отсеять значения, которые больше значения maxSum = 60000.
Результат должен быть такой: "Нет", "20000", "30000", "45000", "50000"
Как правильно написать linq запрос?

Comments

[Толстый Лорри]

Where (x => ....), где попытаться спарсить значение в int, если парсится, то сравнить с maxSum.

[kek123]

Толстый Лорри: это понятно, проблема как раз в конвертации в инт, и в том, что делать с "Нет"

[leremin]

В c# 7.0 можно, например, так

Where(t => !int.TryParse(t, out int i) || int.Parse(t) < 60000);

[Роман]

Проще простого

int maxSum=49000;
string[] input = { "Нет", "20000", "30000", "45000", "50000", "75000" };
var output = input.Where(x => int.TryParse(x, out int t) ? t > maxSum : true);

это для C# 7 версии, для других просто объявить переменную t выше.

[Толстый Лорри]

kek123: Ничего не делать - вернуть true. Where перебирает все члены последовательности и для каждого задает вашему предикату вопрос: "Оставим?". То есть предикат должен вернуть true для всего, кроме того, что парсится в int и больше порогового значения.

Answer 1

[Роман]

string[] input = { "Нет", "20000", "30000", "45000", "50000", "75000" };
var output = input.Where(x => int.TryParse(x, out int t) ? t < 60000 : true);

Answer 2

[Дмитрий Еремин]

List<string> lst = new List<string> { "Нет", "20000", "30000", "45000", "50000", "75000" };

List<string> res = lst.Where(x => !int.TryParse(x, out int n) || int.Parse(x) < maxSum ).ToList();

Comments

[yorick_kiev_ua]

int.TryParse(x, out n) - явно лишний. :)

[Дмитрий Еремин]

yorick_kiev_ua: дап, ты прав)

Answer 3

[yorick_kiev_ua]

string[] arr = { "Нет", "20000", "30000", "45000", "50000", "75000" };
int i;
//var x = arr.Where(_ => !int.TryParse(_, out i) || int.Parse(_) < 60000).ToList();
var x = arr.Where(_ => !int.TryParse(_, out i) || i < 60000).ToList();

Comments

[Роман]

а зачем 2 раза преобразование то вызывать? оно в i будет после TryParse.

[yorick_kiev_ua]

Роман: Да, arr.Where(_ => !int.TryParse(_, out i) || i < 60000).ToList() разумеется корректнее.

[Станислав Силин]

Использование имени переменной "_" подразумевает, что она вам не нужна, а вы ее используете..

[yorick_kiev_ua]

Станислав Силин: Это просто имя переменной, причём переменной, которая исспользуется "по месту", т.е. по-факту оно может быть любым и сколь-либо бессмысленным.

В вашем коде вы вольны исспользовать любые правила именования и даже дать ей какое-то осмысленное имя. Если вы подразумеваете, что такое имя что-то там подразумевает, то не исспользуйте его. Возмите "i", ведь i подразумевает integer, это будет очень академично.

[Станислав Силин]

yorick_kiev_ua:

Возмите "i", ведь i подразумевает integer, это будет очень академично.

C "i" совсем другая история, это может быть итератор, индекс и т.д., а вот "_" в лямбде с параметром, в академических кругах значить только одно - входой параметер не нужен для вычеслений, он игнорируется

[yorick_kiev_ua]

Станислав Силин: В академических кругах - там вообще чёрт знает что твориться может, у них там своя атмосфера. :)
Идея о том, что _ - это неисспользуемая переменная(и вообще что-то показывать именем переменной) возникла для случаев, когда выражение большое и это исспользование неочеведно с первого взгляда.

Поэтому у суровых прагматичных программистов пишущих linq queries - для очевидных случаев типа нашего подходит любе имя .

И, насколько мне известно, никаких жестких требований рекомендаций от MS по поводу "_" в лямбдах не существует, поэтому вообще непонятно чего вы прицепились.

Answer 4

[Tom Nolane]

string[] input = { "Нет", "20000", "30000", "45000", "50000", "75000" }; int t; const int maxSum = 60000;
var output = (from f in input where int.TryParse(f, out t) ? t < maxSum : true select f).ToArray();