Ktoś pomoże jak wyfiltrować przez własne zapytania SQL produkty na których nie było żadnych ruchów od np 6 lat (i stan magazynowy jest 0). mam dużo produktów na subiekci które chciałbym wyfiltrować jako nieaktywne..

Masz rację @candy - tutaj wiele brakuje. To moja wina, bo tylko rzuciłem na to okiem (a że już wzrok nie taki...)
Zapytanie zrobił bardzo młody chłopak, który jeszcze kilka dni temu niewiele wiedział co to jest baza danych i jak wyglądają zapytania SQL, nie miał też pojęcia co to jest Subiekt, ale w ramach praktyk szkolnych chce się uczyć i ja to doceniam:)
Chłopak ma jednak potencjał, więc nie będę za niego tego poprawiał, ale obiecuję, że  w poniedziałek poprawi się i napisze poprawne zapytanie

select tw_Symbol Symbol, tw_Nazwa Nazwa, Stan.Stan from tw__Towar t
join(
select st_TowId, sum(st_Stan) Stan from tw_Stan
group by st_TowId
) stan
on stan.st_Towid = tw_Id
left join
(
select ob_towid from dok_Pozycja
join dok__Dokument on dok_Id = ob_DokMagId
where datediff(year, dok_DataWyst, getdate()) <= {I: Nieużywano przez lat: 0}
group by ob_towid
) sprzedaz on sprzedaz.ob_towid=st_TowId
where (stan > 0 or   1 = {DB:select 0, 'Większy od zera' union all select 1, 'Dowolny':Stan magazynowy:1:Dowolny})
and sprzedaz.ob_towid is null


Poprawione, do testowania

@MacKar
Coś mi się wydaje, że ten where datediff nie zadziała tak jak byś chciał 

Mógłbym poprosić, o trochę dokładniejsze wytłumaczenie? Jeżeli faktycznie jest tu błąd to chciałbym go poprawić, ale najpierw muszę go zrozumieć

Ja już dzisiaj po drugiej lampce jestem, ale tak na oko bez ssms wydaje mi się, że podzapytanie ze sprzedażą pokaże towary, które miały ruch magazynowy w okresie z datadiffa. Jeżeli połączmy je lewostronnie z kartoteką towarów i wybierzemy tylko te, które mają "and sprzedaz.ob_towid is null" to mamy towary, które się w danym okresie nie sprzedały.
Jedyne IMHO nie jest potrzebne to "group by ob_towid". Można było zrobić distinct ale nie wiem czy wydajnościowo to miałoby sens.


No chyba, że to wino jest za mocne albo ja już słabszą głowę mam

Ja bym skorzystał z NOT EXISTS.
W końcu nie musimy wyliczac wszystkich pozycji ruchów towarowych, wystarczy stwierdzić że istnieje jakikolwiek ruch w wybranym okresie i już wiemy że warunek niespełniony.

W oryginalnym pytaniu chodziło też w sumie o znalezienie towarów z zerowym stanem a warunek w zapytaniu @MacKar jest na stany większe od zera ale to pierdółka.

Można jeszcze dodatkowo dorzucić tw_zablokowany=0 żeby pominąć to co już jest nieaktywne.


U mnie też not exits i left join też wyszły bardzo podobnie - dużo mniej kartotek bo 13k ale w dok_pozycja ponad 6 mln.
Rzuciłem się jeszcze na tabelę dok_magruch. U mnie wykonało się to nieporównywalnie szybciej od podzapytania z dok_pozycja i dok__dokument.

Z dok_pozycja i dok_dokument jest jeszcze taki myk, że tam mogą być dokumenty i pozycje bez ruchu magazynowego, jakieś ZD, ZK, czy KFs / KFz cenowe i to może zrobić kuku w takim zapytaniu bez odfiltrowania tego odpowiednimi warunkami. Bez sprawdzania nie chcę rzucać tego jako absolutnego pewniaka ale dok_magruch chyba będzie bezpieczniejsze i szybsze.

EDIT: Dopiero teraz skojarzyłem, że odpytanie jest tutaj po ob_DokMagId więc te ZK i inne to nam wytnie ale zawsze zostanie kwestia wzięcia poprawki na wycofany lub odłożony skutek magazynowy.