请高手指点这个存储过程的事务是否正确！
CREATE PROCEDURE MakeMedicineNo

@ShipId varchar(30),
@ProductId varchar(30),
@AreaNo varchar(5),
@Quantity int


AS

BEGIN TRAN MakeMedicineNo

DECLARE @maxID INT,@maxSerial varchar(6) ,@i int,@CardNo varchar(20),@IceNo varchar(20),@MedicineNo varchar(24),@Six varchar(6)
SET @i=0
SELECT @maxID=MAX(Serial) FROM WinShipNo WHERE ProductId=@ProductId
IF @maxID IS NULL
BEGIN
SET @maxID=0
END

CREATE TABLE #(Serial int, SerialNo varchar(30))
WHILE @i <@Quantity
BEGIN

--SELECT @maxSerial=REPLICATE( '0',6-LEN(@maxID)) + cast(@maxID as varchar),@maxID=@maxID+1
SET @maxID=@maxID+1
INSERT #(Serial,SerialNo) SELECT @maxID, REPLICATE( '0',6-LEN(@maxID))+CAST(@maxID AS varchar)
SET @i =@i +1
END

SET @MedicineNo=REPLICATE( '&',12-LEN(@ProductId))+@ProductId
SET @Six=+'000000'
SET @IceNo=@AreaNo+'0000000000001'
SET @CardNo=@AreaNo+'0000000008888'

INSERT WinShipNo(ShipId,IceNo,ProductId,MedicineNo,Serial,InTime,CardNo,[Sign])
SELECT @ShipId,@IceNo,@ProductId,@MedicineNo+SerialNo+@Six,Serial,GETDATE(),@CardNo,0
FROM #


/*
SELECT @maxID=MAX(FlowNum) FROM WinShipFlowIfUser WHERE ProductId='A' AND IfUse=0
SELECT @maxSerial=REPLICATE( '0',6-LEN(RTRIM(@maxID))) + cast(@maxID as varchar)
SET @maxID=@maxID+1
INSERT WinShipFlow (ProductId,FlowNo) SELECT 'A',@maxSerial
INSERT WinShipFlowIfUser (ProductId,FlowNum) SELECT 'A',@maxID
UPDATE WinShipFlowIfUser set IfUse=1 where ProductId='A' and FlowNum=(@maxid-1)
*/

COMMIT TRAN MakeMedicineNo--事务提交

IF (@@error <> 0)--事务回滚

BEGIN
ROLLBACK TRANSACTION MakeMedicineNo
RETURN 0
END
RETURN 1

GO


[解决办法]
可能会出问题，2次INSERT，第2次@@ERROR值已经改变了，可以用变量保存下，最后判断2个值，如果都正确，才提交，否则回滚
[解决办法]
你可以在上面出错的地方就回滚事务，然后就退出存储过程。不需要在最后再来判断这个。