[absynthe]

Martyr2, on 10 Sep, 2009 - 08:00 PM, said:

Well do a process of elimination here. Take out the whole if else statement and see if the trigger still compiles with errors. If it does, then you have an error that isn't that whole big if statement. I would first remove the if else statement and insert the variable v_AdjGrossIncome straight into TaxReturns. See if that goes through smoothly first and then you can tackle the if else statement.

Giving it a try now Thank you so much Martyr2!

[absynthe]

I think it may be copying and pasting from Notepad...getting an error on the simplest form of the trigger now. Let me try inputting it

[mostyfriedman]

i've never heard of anyone using pl sql before anywayz..mysql ftw

[absynthe]

My fingers hurt now and the stupid Compile Error is still there...

I know! It sucks too, teacher just said "We're using PL/Sql" No one in class had used it at all...

This post has been edited by absynthe: 10 September 2009 - 07:35 PM

[Martyr2]

Ok, now show us the simplified version you are using.

[absynthe]

Is the logic wrong? Is the if statement wrong you think?

CREATE OR REPLACE TRIGGER Tax_Return_Trigger
BEFORE UPDATE OF AdjGrossIncome
ON TaxReturns
FOR EACH ROW
DECLARE
v_TaxRate1 FLOAT := 0.00;
v_TaxRate2 FLOAT := 0.20;
v_TaxRate3 FLOAT := 0.30;
v_TaxRate4 FLOAT := 0.40;
v_TaxRate5 FLOAT := 0.50;
v_AdjGrossIncome NUMBER;
v_NewRate NUMBER;
BEGIN
SELECT AdjGrossIncome INTO v_AdjGrossIncome FROM TaxReturns;

END;
SHOW ERRORS
 /

This post has been edited by absynthe: 10 September 2009 - 07:38 PM

[Martyr2]

Try now running a more simplified version....

CREATE OR REPLACE TRIGGER Tax_Return_Trigger
BEFORE UPDATE OF AdjGrossIncome
ON TaxReturns
FOR EACH ROW
DECLARE
v_AdjGrossIncome NUMBER;

BEGIN
SELECT AdjGrossIncome INTO v_AdjGrossIncome FROM TaxReturns;

END;

Does this compile without problem?

[absynthe]

Martyr2, on 10 Sep, 2009 - 08:42 PM, said:

Try now running a more simplified version....

CREATE OR REPLACE TRIGGER Tax_Return_Trigger
BEFORE UPDATE OF AdjGrossIncome
ON TaxReturns
FOR EACH ROW
DECLARE
v_AdjGrossIncome NUMBER;

BEGIN
SELECT AdjGrossIncome INTO v_AdjGrossIncome FROM TaxReturns;

END;

Does this compile without problem?

Yes, that compliled perfectly Awesome! The first trigger I've seen done without errors.

Ill add some chunks to it to see where it breaks

[Martyr2]

Ok, now step two is to add in your variables only. If it errors out it is because your variables are in the incorrect format. I suggest adding one tax rate variable first.

[absynthe]

Shit the first chunk I put in had errors? Even with FLOAT

CREATE OR REPLACE TRIGGER Tax_Return_Trigger
BEFORE UPDATE OF AdjGrossIncome
ON TaxReturns
FOR EACH ROW
DECLARE
v_AdjGrossIncome NUMBER;
v_TaxRate1 := 0.00;
v_TaxRate2 := 0.20;
v_TaxRate3 := 0.30;
v_TaxRate4 := 0.40;
v_TaxRate5 := 0.50;

BEGIN
SELECT AdjGrossIncome INTO v_AdjGrossIncome FROM TaxReturns;

END;
/

OOOPS let me try one at a time

[mostyfriedman]

absynthe, on 10 Sep, 2009 - 06:34 PM, said:

My fingers hurt now and the stupid Compile Error is still there...

I know! It sucks too, teacher just said "We're using PL/Sql" No one in class had used it at all...

aww, lemme kiss that for ya ..mwah, there better now ??

[absynthe]

What the hell? It immediately went into an error with the first Tax1 added.

From the looks of it here im right on with syntax:

http://www.java2s.co...talogPL-SQL.htm

[mostyfriedman]

am pretty sure though that you need to declare the data types you are using

[absynthe]

It doesnt show it here though:

http://www.java2s.co...rmineagrade.htm

[Martyr2]

mostyfriedman is right with the data type. Try using... v_TaxRate1 CONSTANT NUMBER(2,2) := 0.00;

Here we are defining a constant tax rate value of 0.00