[ugotowany_kamien]

#sql #mssql

chcę utworzyć procedurę która przy dodaniu pracownika do tabeli pracownicy da mu 100 zł płacy dodatkowej jeżeli będzie zarabiał poniżej 1000 zł, a jeżeli powyżej to nie daje mu nic. Zrobiłem takie coś ale nie dodaje tych 100zł, gdzie popełniłem błąd

alter procedure podwyzka
(@id_prac int, @nazwisko varchar(15) = null, @etat varchar(10) = null, @id_szefa int = null, @zatrudniony date = null, @placa_pod decimal(6,2) = null, @placa_dod decimal(6,2) = null, @id_zesp int = null)
as
begin
insert into PRACOWNICY(ID_PRAC, NAZWISKO, ETAT, ID_SZEFA, ZATRUDNIONY, PLACA_POD, PLACA_DOD, ID_ZESP)
values (@id_prac, @nazwisko, @etat, @id_szefa, @zatrudniony, @placa_pod, @placa_dod, @id_zesp)
if @placa_pod <1000
begin
set @placa_dod = isnull(@placa_dod, 100)
end
end

[PrawaNogaBoga]

@ugotowany_kamien: ja bym najpierw sprawdzał pensje podstawowa i w zależności czy jest mniejsza czy nie od tego 1000 to dodawał 100 zł i gotowe dane insertowal, po co robic insert a pozniej kolejna operacje na bazie jak mozna załatwić to jedna operacja

[dobry_programista]

@ugotowany_kamien przenieś tego if'a przed insert, powinno być lepiej. Obecnie najpierw robisz insert do bazy a później dopiero ustalasz ile ma być tej dodatkowej pensji...

[ugotowany_kamien]

@dobry_programista: właśnie :P thx :P

[ugotowany_kamien]

@PrawaNogaBoga: @dobry_programista: co teraz robie źle? podkreśla mi @placa_dod po 2 befinie i elsie

[ugotowany_kamien]

alter procedure podwyzka
(@idprac int, @nazwisko varchar(15) = null, @etat varchar(10) = null, @id_szefa int = null, @zatrudniony date = null, @placa_pod decimal(6,2) = null, @placa_dod decimal(6,2) = null, @id_zesp int = null)
as
begin
if @placa_pod <1000
if @placa_dod = null
begin
@placa_dod = isnull(@placadod, 100)
else
@placa_dod = 100
end
insert into PRACOWNICY(IDPRAC, NAZWISKO, ETAT, IDSZEFA, ZATRUDNIONY, PLACAPOD, PLACADOD, IDZESP)
values (@idprac,

[PrawaNogaBoga]

@ugotowany_kamien: po pierwszym ifie tez musisz dać begin, to jest jak nawias

[PrawaNogaBoga]

@PrawaNogaBoga: w zasadzie to usuń ten warunek if placadod == null

[dobry_programista]

@ugotowany_kamien tam gdzie jest isnull? Bo nie wiem co tym isnull chcesz osiągnąć. W MySQL jest chyba takie coś jak ifnull - podejrzewam że to może Ci pomóc.

[ugotowany_kamien]

@PrawaNogaBoga: ok thx

[ugotowany_kamien]

@dobry_programista: zmieniłem na takie coś i jest git

alter procedure podwyzka
(@idprac int, @nazwisko varchar(15) = null, @etat varchar(10) = null, @id_szefa int = null, @zatrudniony date = null, @placa_pod decimal(6,2) = null, @placa_dod decimal(6,2) = null, @id_zesp int = null)
as
begin
insert into PRACOWNICY(IDPRAC, NAZWISKO, ETAT, IDSZEFA, ZATRUDNIONY, PLACAPOD, PLACADOD, IDZESP)
values (@idprac, @nazwisko, @etat, @id_szefa, @zatrudniony, @

[ugotowany_kamien]

@dobry_programista: @PrawaNogaBoga: jednak nie działa to co wyżej 1 post napisałem ;/ dalej daje 0 do placy dod

[PrawaNogaBoga]

@ugotowany_kamien: nie zrozumiałeś, najpierw powinieneś sprawdzać czy płaca podstawowa jest mniejsza niz 1000, jeśli tak to ustaw płace dodatkowa na 100, jeśli nie to nic nie rob (na początku czyli podczas inicjalizacji ustaw ja na 0 a nie nulla), na koniec bez żadnych juz warunków zrób zwykłego inserta do bazy tak jak to teraz robisz na początku

[PrawaNogaBoga]

@PrawaNogaBoga: No i zauważ ze ustawiasz ZMIENNA a nie wartość w bazie, spróbuj usunąć znak małpy w trzeciej linii od dołu po set

[ugotowany_kamien]

@PrawaNogaBoga: działa po usunięciu małpy :D dzieki

[ugotowany_kamien]

@ugotowany_kamien: a co do tego @placa_pod decimal(6,2) = null to się nie powinno chyba pluć bo tu się tylko ustawia wartość domyślną jak nic nie dodam do tej zmiennej

[PrawaNogaBoga]

jasne, teraz zauważyłem że to parametr, zrefaktoryzuj jeszcze te warunki tak jak pisałem wyżej żeby nie robić niepotrzebnego transferu danych