[smalczyk1]

#csharp #dotnet #naukaprogramowania

Zerknie ktoś bo ślepy jestem i nie widzę błędu a dostaje zwrotkę "Nie domknięty cudzysłów"

 var prices = _db.Database
 .SqlQuery(
 $"select * from CenyDlaKontrahenta({user_id}) where twr_kod in ('" + string.Join(",", jsonCart.products.Select(_ => "'" + _.code + "'")).ToList() + ")").ToList();

[auCwSlorpoaSh5TGXogPfUDhO8s9EJ99ecW]

twr_kod in ('"

@smalczyk1: tu masz apostrofe i cudzyslow moze to

[smalczyk1]

no też tak myślałem ale nie :( nadal to samo

[b.....a]

@smalczyk1: twr_kod in (' - brakuje zamkniecia tego

[Kazaar]

@smalczyk1: nawiasy ci sie p------y chyba
po co ci ToList() na string.Join?

string.Join(",", jsonCart.products.Select( => "'" + .code + "'")).ToList()

string.Join(",", jsonCart.products.Select( => "'" + .code + "'").ToList())

[Goglez]

@smalczyk1: Tak to poukładałem
https://pastebin.com/AXqURKzA
Chyba brakuje domknięcia apostrofu z 6 linii. W 9 linii ten plus i cudzysłów tak mają być?

[Sierpik21]

@smalczyk1: nigdy, nigdy,...., ale, ale.... nie ważne dlaczego, NIGDY tak nie rób, poczytaj co to jest sql injection!

[p.....s]

@smalczyk1: jak już musisz to tak robić, to chociaż zrób procedury skłądowane bo jak @sierpik21 napisał, to jest dramat

[smalczyk1]

@piepies: @Sierpik21: @Goglez: @Kazaar: @bacteria: @bacteria: @auCwSlorpoaSh5TGXogPfUDhO8s9EJ99ecW:

Dziękuje wszystkim za pomoc. Poprawieęteż kod zgodnie z poradami.

Rozwiązanie

https://pastebin.com/LCPp516C