Home/ Questions/Q 5991001
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-22T23:19:29+00:00

I am working on a ‘grading’ system, and am trying to make sure that

  • 0

I am working on a ‘grading’ system, and am trying to make sure that a person is not able to submit a grade twice by using a stored procedure that will check if a person has graded a particular item before allowing a new grade to be saved. The odd thing is, I am passing a user ID and object ID, but when my stored procedure selects the COUNT(*), I get the wrong number.

Here is my stored procedure:

CREATE PROCEDURE `ADD_GRADE`(IN objectID int, IN grader int)
BEGIN
DECLARE gradeCount INT;

SELECT COUNT(*)
FROM GRADES
WHERE Obj_ID = objectID AND Grader = grader
INTO gradeCount;

IF gradeCount <= 0 THEN INSERT INTO Grades (Obj_ID, Grader) VALUES (objectID, grader);
END IF;

The IF statement is working, but for some reason my gradeCount appears to be ignoring the Grader ID and just checking based upon the Obj_ID. I’ve added selects to be sure my parameters are staying at the correct value and they are. If I copy just the select and do it elsewhere manually, I get the correct number.

Though new to MySql, I’m not new to SQL itself. Am I just going crazy? Or am I doing something wrong?

Thanks

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    Even if this code worked (I don’t know why it does not) it is not the proper way to make sure something is only entered once.

    The proper way is to apply a unique constraint on the objectID and grader columns. Then try inserting the row. If the row inserts then the values are unique. If you get a unique violation then the values have already been entered.

    If you do not have unique constraints available you should lock the table to make sure no other updates are happening between your commands.

    • 0