プログラマ専用SNS ミクプラ

Shiftを押さないでQかWを押すとFlagは１
Shiftを押しながらQかWを押すとFlagは２
WもSも押されてないならFlagは０

としたいです。

現在、押下キーは

コード:

char Key[256]; //キーが押されているかいないかを格納する
Flag = 0;
if( Key[KEY_INPUT_Q] == 1 )
{
    Flag = 1;
    if( Key[KEY_INPUT_LSHIFT] == 1 )
    {
        Flag = 2;
    }
}
if( Key[KEY_INPUT_W] == 1 )
{
    Flag = 1;
    if( Key[KEY_INPUT_LSHIFT] == 1 )
    {
        Flag = 2;
    }
}

としていますが、同じような形が２個もあるため、もっと簡単に出来ないかと思いましたが方法が思いつきません
このあと、E,R,T・・・・と増えていった場合、見にくくなってしょうがないと思うので
いい方法があったら教えてください

先にKEY_INPUT_LSHIFTをチェックしてFlagに状態を代入せずにFlagに代入する値を作るのがひとつの方法です。
int FlagType = Key[KEY_INPUT_LSHIFT]==1 ? 2 : 1;
って感じですね。

こういうことですか？

コード:


char Key[256]; //キーが押されているかいないかを格納する
Flag = 0;
if( Key[KEY_INPUT_Q] == 1 || Key[KEY_INPUT_W] == 1 )
{
    Flag = 1;
    if( Key[KEY_INPUT_LSHIFT] == 1 )
    {
        Flag = 2;
    }
}

史上最悪のデスペナさんが書きました:WもSも押されてないならFlagは０

これは、QもWも、の間違いでしょうか。

一つ疑問なのは、どうしてQとWで違う値ではなく同じ値をFlagに設定するのでしょう。
QとWは区別しなくていいのでしょうか。
それとも、何か本当の目的は違うところにあるのでしょうか。

beatle さんが書きました:これは、QもWも、の間違いでしょうか。

その通りです

beatle さんが書きました:一つ疑問なのは、どうしてQとWで違う値ではなく同じ値をFlagに設定するのでしょう。
QとWは区別しなくていいのでしょうか。
それとも、何か本当の目的は違うところにあるのでしょうか。

簡単に言うと
Wで前進、Sで後退しますが（Qはまた別の動きですが説明しにくいのでWとSでやり直し）
Shiftを押しながらだと歩き、Shiftを押さないと走ります。
Flagはモーション用であり、１で走るモーション、２で歩くモーション、０で立っているモーションとさせたいのです

softya（ソフト屋) さんが書きました:先にKEY_INPUT_LSHIFTをチェックしてFlagに状態を代入せずにFlagに代入する値を作るのがひとつの方法です。
int FlagType = Key[KEY_INPUT_LSHIFT]==1 ? 2 : 1;
って感じですね。

何をしているのかが分からないのでちょっと調べてきます

これは条件演算子（三項演算子）ですよ。
条件式 ? 条件式が真の場合の値 : 条件式が疑の場合の値

こんなものがあるんですね。勉強になります
条件演算子って便利ですね。見た目も綺麗になりますし。

史上最悪のデスペナさんが書きました:こんなものがあるんですね。勉強になります
条件演算子って便利ですね。見た目も綺麗になりますし。

これの多用は凄く見づらくなるのでＢＡＤです。いっぱい並ぶと最悪です。
なので、あまり見かけないですよね？
ここぞと言うワンポイントで使うのが吉です。

史上最悪のデスペナさんが書きました:条件演算子って便利ですね。見た目も綺麗になりますし。

こんな見た目ではどう思います？

コード:

char Key[256]; //キーが押されているかいないかを格納する
Flag = 0;
Key[KEY_INPUT_Q] == 1 && (Flag = 1);
Key[KEY_INPUT_W] == 1 && (Flag = 1);
Key[KEY_INPUT_LSHIFT] == 1 && Flag == 1 && (Flag = 2);

softya（ソフト屋) さんが書きました:これの多用は凄く見づらくなるのでＢＡＤです。いっぱい並ぶと最悪です。
なので、あまり見かけないですよね？
ここぞと言うワンポイントで使うのが吉です。

確かに一度も見かけたことないですね。
ifが沢山あるよりは見やすいかと思ったのですが・・・・・・

ISLe さんが書きました:
史上最悪のデスペナさんが書きました:条件演算子って便利ですね。見た目も綺麗になりますし。
こんな見た目ではどう思います？
コード:
char Key[256]; //キーが押されているかいないかを格納する
Flag = 0;
Key[KEY_INPUT_Q] == 1 && (Flag = 1);
Key[KEY_INPUT_W] == 1 && (Flag = 1);
Key[KEY_INPUT_LSHIFT] == 1 && Flag == 1 && (Flag = 2);

自分はすっきりしていてすごい見やすいと思います

ちなみに、

コード:

Key[KEY_INPUT_Q] == 1 だったらFlag=1
Key[KEY_INPUT_W] == 1 だったらFlag = 1
Key[KEY_INPUT_LSHIFT] == 1 でFlag == 1だったらFlag = 2

でいいですか？

今は誤認しないかも知れませんがISLeさんのもの含めて後々誤認や書き間違いを起こしやすいです。
条件式と代入文が渾然一体だからですが、これはトリッキーなんですよね。どっかで==と=を間違えてしまった場合を考えてみましょう。
それとデバッガでトレースが困難なのも弱点ですね。

参考
「Programming Place Plus　Ｃ言語編　第２７章　簡略記法」
http://www.geocities.jp/ky_webid/Progra ... itional_op

【補足】
ついでにダークサイドな演算式編。
「闇プログラマーに喧嘩を売ってしまった昼下がり～条件分岐篇～ - Togetter」
http://togetter.com/li/140355

softya（ソフト屋) さんが書きました:今は誤認しないかも知れませんがISLeさんのもの含めて後々誤認や書き間違いを起こしやすいです。
条件式と代入文が渾然一体だからですが、これはトリッキーなんですよね。どっかで==と=を間違えてしまった場合を考えてみましょう。
それとデバッガでトレースが困難なのも弱点ですね。

普通のif文でも==と=を間違えてしまってどこが間違ってるのか分からずにってのが前まであったのを思い出しました。
あれは見つけにくいですよね・・・・・・・・

softya（ソフト屋) さんが書きました:【補足】
ついでにダークサイドな演算式編。
「闇プログラマーに喧嘩を売ってしまった昼下がり～条件分岐篇～ - Togetter」
http://togetter.com/li/140355

途中のprintfで何をしようとしているのかが分からない・・・・・後ろのコメントアウトを見る限り文字コードに合わせてバイトをずらしているのでしょうか？
printf("誤¥0正"+(a==b)*4);
だったら、
a==bなら(a==b)は1なので４バイトずらし、違うなら０なのでずらされないで「誤」になる
という解釈でいいのでしょうか

結論として、今回はしょうがないから条件演算子を使う。
しかし、使う必要のないところは極力使わない

ということでいいのですよね？

しょうが無いと言うより、その方が今回は私は分かりやすいと思ったからです。
三項演算子が大キライな人も大好きな人もいますので、人によって意見は違うと思います。
私は、使い所を間違えなければ使っても良いと思います。

ありがとうございます。
今回のところ以外に対してはいろいろ試してみて自分のコーディング方法を確立しようと思います

プログラマ専用SNS ミクプラ

押下キーによるフラグ立てのいい方法

押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法

Re: 押下キーによるフラグ立てのいい方法