[ugotowany_kamien]

#csharp #naukaprogramowania jak zrobić zapytanie linq ktore usunie mi ze stringa wszystkie znaki ktore nie sa cyframi np

var query = ( for x in tabelazbazydanych
where x.jakaskolumna == 0
select x)

chcę żeby wyciełko z jakas_kolumna wszystkie znaki po za cyframi

[LeniwiecPracoholik]

Na cholerę linq?
https://msdn.microsoft.com/en-us/library/xwewhkd1(v=vs.110).aspx
Zrób to poprawnie jak człowiek

[ugotowany_kamien]

@LeniwiecPracoholik: bo mam całe zapytanie do bazy danych i chce zeby mi ucinalo te znaki w jednej z kolumn

[nykaz]

var result = jakas_kolumna.Where(v => v.Any(c => Char.IsDigit(c))).ToList();

[LeniwiecPracoholik]

@nykaz nie ucz go złych praktyk. Taki kod potem jest nieczytelnym gównem.

@ugotowany_kamien to wciąż nie jest usprawiedliwienie dla użycia linq. Linq jest do przeszukiwania kolekcji. W ten sposób stworzysz #!$%@? kod, który w przyszłości ktoś będzie musiał przeczytać i zrozumieć. To się nazywa dobre praktyki programistyczne.
Pamiętaj żeby mieć nawyk pisania kodu tak by ktoś inny, kto przejmie go po tobie mógł się w nim szybko połapać. Dotyczy to też samego

[ugotowany_kamien]

@LeniwiecPracoholik: tylko że ja dostałem taki kod i muszę tylko jedną klauzule w tym zapytaniu przerobic

[LeniwiecPracoholik]

@ugotowany_kamien to go popraw. Bądź dobrym programistą. Nie powielaj gówna.
Żadne usprawiedliwienie nie wyjaśnia tworzenia gówna

[ugotowany_kamien]

@nykaz: wyskakuje mi blad ze ta kolmna nie jest kolekcja

[LeniwiecPracoholik]

@ugotowany_kamien var query = ( for x in tabelazbazydanych where x.jakaskolumna == 0 select Regex.Replace(x.kolumna, "[a-zA-Z]*", string.Empty)); Nie jest czytelniej?

[bi-tek]

@LeniwiecPracoholik: Kolega @ugotowany_kamien: Dobrze Ci mówi, nie pisz gówno kodu bo potem nie idzie się w tym połapać albo masz kod z milionem komentarzy i zastanawiasz się WTF? Jak to działa w ogóle. Spaghetti nie jest fajne.

[ugotowany_kamien]

@LeniwiecPracoholik: rozumiem ze tu "[a-zA-Z]*] ma byc tez cudzyslow na koncu?
Dzieki za pomoc

[LeniwiecPracoholik]

@ugotowany_kamien źle się pisze na telefonie, sorki

[ugotowany_kamien]

@LeniwiecPracoholik: tak gwiazdka też uwzgledni znaki specjalne?

[ugotowany_kamien]

@LeniwiecPracoholik: The expression is not supported in the Where clause: Replace(l.IDAA.NRBUD, "[a-zA-Z]*]", String.Empty)
( ͡° ʖ̯ ͡°)

[krzysio19]

@ugotowany_kamien: * to dowolna ilość

[nykaz]

Nie wiedziałem, że zapytanie wykonywane jest przez linqToSql / EF, które nie tłumaczy wielu metod (w tym regexów) do SQL-a. W tym przypadku, aby wykonało się to po stronie serwera, musisz ratować się czymś w stylu:

where jakaskolumna.Replace("-", "").Replace(" ", "") == 0

czyli spaghetti code (lepiej to rozwiązać w inny sposób). Więcej tutaj: https://msdn.microsoft.com/en-us/library/bb882672%28v=vs.100%29.aspx?f=255&MSPPError=-2147217396

@ugotowany_kamien BTW: Jeżeli chodzi o operacje w pamięci, moim zdaniem i tak powinno się wyodrębnić extension

[LeniwiecPracoholik]

@ugotowany_kamien nie znasz regex? Gwiazdka oznacza od 1 do więcej znaków

[nykaz]

@LeniwiecPracoholik: Nie zgadzam się, że zamiana na regex uczyni kod bardziej czytelnym. Powinnyśmy wyodrębnić osobną metodę. Poza tym w przypadku linq to sql, obie implementacje i tak nie zostaną zamienione na poprawny SQL, co wyjaśniłem wyżej.

[andrzej-kopara]

@LeniwiecPracoholik: Dziwnie wygląda krytyka z Twojej strony, wychwalanie Regexa i wyśmiewanie innych od nieznajomości go, gdy sam masz z nim problem.

Gwiazdka oznacza 0 lub więcej, to plus oznacza 1 lub więcej dopasowań.

[LeniwiecPracoholik]

@andrzej-kopara: Gdzie wychwalam regex? I przede wszystkim, gdzie wyśmiałem kogokolwiek z powodu nieznajomości wyrażeń regularnych? Spytałem tylko o to.
I owszem nie jestem mistrzem regex, bo z niego za często nie korzystam. Dlatego zwykle korzystam z konstrukcji {1,} zamiast * i +, które notorycznie mylę, co widać w mojej odpowiedzi. ¯\_(ツ)_/¯
A co do krytyki.
LinQ służy do operacji na kolekcjach. Jak ktoś zobaczyłby kod używający linq do wycinania z stringa