0

私は SQL/T-SQL の独学を始めた初心者です。私は SQL 2014 Express を使用しており、T-SQL スクリプトを作成してすべてを実行しようとしています。私はこれを、彼女が書いている実写ゲームのデータベースで友人を助けるために行っています。次のユーザー定義関数をより合理化する/より正確な方法で書くという点で、いくつかの助けを借りることができます。

アイテムの経済的価値を決定するために永続的な計算が必要なため、関数自体が必要であると私は信じていました。これは、Items テーブルが更新されるたびにすべての値を更新する何らかの更新スクリプトを実行するよりも優れた方法のように思えました。これがすべて初歩的で、学習しようとしているように見える場合は申し訳ありません。

以下はコードです。私はそれを機能させ、開発データベースに入れました。それは想定どおりに機能します。しかし、それをきれいにしたいのですが、それを行う方法がわかりません。

    CREATE FUNCTION dbo.ValueCalc (@itemid int)
    RETURNS INT
    AS
    BEGIN
    declare @rm1 int, @rm1amount int, @rm1value int,
    @rm2 int, @rm2amount int, @rm2value int,
    @rm3 int, @rm3amount int, @rm3value int,
    @rm4 int, @rm4amount int, @rm4value int,
    @cm1 int, @cm1amount int, @cm1value int,
    @cm2 int, @cm2amount int, @cm2value int,
    @cm3 int, @cm3amount int, @cm3value int,
    @cm4 int, @cm4amount int, @cm4value int,
    @productionvalue int;

    select @rm1 = MatReqs.RM1 FROM MatReqs WHERE MatReqs.ItemId = @itemid; 
    select @rm1amount = MatReqs.RM1Amount FROM MatReqs WHERE MatReqs.ItemId = @itemid; 
    select @rm1value = RawMats.BaseValue FROM RawMats WHERE RawMats.RawMatId = @rm1; 
    if (@rm1 IS NULL) set @rm1=0;
    if (@rm1amount IS NULL) set @rm1amount=0;
    if (@rm1value IS NULL) set @rm1value=0;

    {Repeat the above 3 more times, for rm2, rm3, and rm4}

    select @cm1 = MatReqs.CM1 FROM MatReqs WHERE MatReqs.ItemId = @itemid;
    select @cm1amount = MatReqs.CM1Amount FROM MatReqs WHERE MatReqs.ItemId = @itemid;
    select @cm1value = Items.ProdValue FROM Items WHERE Items.ItemId = @cm1;
    if (@cm1 IS NULL) set @cm1=0;
    if (@cm1amount IS NULL) set @cm1amount=0;
    if (@cm1value IS NULL) set @cm1value=0;

    {same here, just removed the repetitions}      

    set @productionvalue = (@rm1amount * @rm1value) 
    + (@rm2amount * @rm2value) 
    + (@rm3amount*@rm3value) 
    + (@rm4amount*@rm4value)
    + (@cm1amount*@cm1value)
    + (@cm2amount*@cm2value)
    + (@cm3amount*@cm3value)
    + (@cm4amount*@cm4value);

    set @productionvalue = @productionvalue + (@productionvalue * .15);

    return @productionvalue;
    END
    GO
4

1 に答える 1

0

サンプル データと目的の結果と関数全体があれば、より効率的なソリューションを作成できます。SQL Server はセット ベースのコード用に最適化されており、実際には使用しないからです。単純にクリーンアップする限り、これを行うこともできますが、どのサンプル データと目的の結果を投稿して、代わりに適切なセットベースのソリューションを取得することをお勧めします.

CREATE FUNCTION dbo.ValueCalc (@itemid INT)
RETURNS INT
AS
BEGIN
DECLARE @rm1 INT, @rm1amount INT, @rm1value INT,
        @rm2 INT, @rm2amount INT, @rm2value INT,
        @rm3 INT, @rm3amount INT, @rm3value INT,
        @rm4 INT, @rm4amount INT, @rm4value INT,
        @cm1 INT, @cm1amount INT, @cm1value INT,
        @cm2 INT, @cm2amount INT, @cm2value INT,
        @cm3 INT, @cm3amount INT, @cm3value INT,
        @cm4 INT, @cm4amount INT, @cm4value INT,
        @productionvalue INT;

--They both come from the same table with the where clause so just combine them
SELECT  @rm1 = ISNULL(MatReqs.RM1,0), 
        @rm1amount = ISNULL(MatReqs.RM1Amount,0)    
FROM MatReqs 
WHERE MatReqs.ItemId = @itemid; 

SELECT @rm1value = ISNULL(RawMats.BaseValue,0)  F
FROM RawMats 
WHERE RawMats.RawMatId = @rm1; 

--The ISNULL() function will take care of this
    --if (@rm1 IS NULL) set @rm1=0;
    --if (@rm1amount IS NULL) set @rm1amount=0;
    --if (@rm1value IS NULL) set @rm1value=0;


--Combine the first two statements again
SELECT  @cm1 = ISNULL(MatReqs.CM1,0),
        @cm1amount = ISNULL(MatReqs.CM1Amount,0)
FROM MatReqs 
WHERE MatReqs.ItemId = @itemid;

SELECT @cm1value = ISNULL(Items.ProdValue,0)
FROM Items 
WHERE Items.ItemId = @cm1;

--Again just use ISNULL
    --if (@cm1 IS NULL) set @cm1=0;
    --if (@cm1amount IS NULL) set @cm1amount=0;
    --if (@cm1value IS NULL) set @cm1value=0;
于 2015-03-26T05:38:20.873 に答える