複数の条件に基づいてフィールドを更新する

Question

ユーザーのxp値が彼のレベルの最大xpより大きいかどうかを確認して、経験値バーが値を適切に更新しようとしています。私はこのコード

$sql1 = "UPDATE progression SET xp=(xp + 2) WHERE username='$username'"; 

によって価値を高めています。そして、私はXPの値が今の最大のXPよりも高い場合は、データベースにチェックしたいと思いますし、それがXPを取るある場合 - 最大XPを。これはプレーヤーが98％xpなどで10％xpを得た場合、次のレベルで8％に更新する必要があります（次のレベルでは最大xpの増加がない場合）。私はこのようなものが欲しいですが、これはうまくいかないようです。

$sql2 = "UPDATE if xp > maxxp SET xp=(xp - maxxp) WHERE username='$username'"; 

Answer 1

あなたのクエリが間違った構文を持っている場合は、余分を必要としないので：

$sql2 = "UPDATE if xp > maxxp SET xp=(xp - maxxp) WHERE username='$username'"; 

が

$sql2 = "UPDATE progression SET xp=(xp - maxxp) WHERE username='$username' AND xp > maxxp"; 

する必要がありますが、私はあなただけで更新する必要があることをデビッドに同意します経験値の数を計算し、その場でレベル計算を行います。

このクエリでは、更新するテーブルの名前がありません。あなたはまだ進歩を更新していると思います。

さらに、準備されたステートメントを見てみてください。これはSQLインジェクションを防ぎます。

Answer 2

SQLは、IF文を使用する他の言語と同様の文をCASEで使用します。だから私はあなたが代わりに〜と何かを探していると信じています。

CASE WHEN (xp > maxxp) THEN SET xp=(xp - maxxp) WHERE username='$username' 

SQL内のCASE文の詳細は、linkを参照してください。しかし、他の人が言及したように、この問題を解決するために、whereステートメントに別の条件を簡単に追加することができます。

AND;これはセキュリティの問題であるため、変数をSQL文に直接渡すべきではありません。代わりに、パラメータをクエリにバインドする必要があります。あなたはこのために2つのクエリを必要としない

Answer 3

、あなたは直接modulusを使用して計算することができます。

$sql1 = "UPDATE progression SET xp = (xp + 2) % maxxp WHERE username = :username"; 

はまた、あなたがあなたのクエリで変数を注入ではなく、準備されたステートメントを使用するべきではないことに注意してください。

Answer 4

IFを使用する場合は、このようなものを試すことができます。

UPDATE progression 
SET  xp = IF(xp > maxxp, xp - maxxp, xp) 
WHERE username='$username' 

しかし、おそらく、WHEREの方が良い場合があります。