これはAnd演算に変えてよいか？

質問

こんにちは。c#初心者です。今回はcで共通する部分（のはず）なのでc#でなくても回答できると思います。

　最初はこういうコードを書いていました。
（a, b, cはint、switchOnはbool値です）

int value;
if ( switchOn ) value = a + b + c;
else value = a + b;

これを

（a, b, c, maskはint。またmaskは0または111……111）
int value = a + b + (c & mask);

のようにifを使わない形にしてもいいのでしょうか？

　この部分は今後とも変更・メンテナンスの必要が特になく、他のクラスの中に埋め込まれるクラスのメソッドで、特に第三者が見る事・必要もほとんど無く（もとい、チームとかありませんから）、可読性が重要視される訳ではないのですが、速度が重視される訳でもないんですね。
　とりあえず、後者のほうを選んでも、特にデメリットはないのですが（かといってメリットもあまり無いのですが）、使えるなら予備知識になると思い、質問させていただきました。

　どなたか詳しい方、いらっしゃいましたら教えていただけませんか？

回答No.3

επιστημη（@episteme） · ベストアンサー

やってもいいけどややこしくするだけ損かと。

僕なら
int value = a + b + (swichOn ? c : 0);
かしら。

お礼

回答ありがとうございます。

　３項演算も念頭にはありましたが、投稿するのはifだけでいいかなと。
　まあ、ややこしくなるだけで、アセンブラでもしない限り無用の長物なんでしょうが、知識はあっても困らないということで。

　こんな事に付き合ってください、ありがとうございました。

回答No.4

kmee

「昨日の自分は今日とは別人」という言葉もありまして。
メンテナンス性は、なにより自分のためだったりします。

> int value = a + b + (c & mask);

このmaskを決定する際に
if( switchOn ) {
mask=~ 0 ;
}else{
mask=0;
}
などとやっているなら、valueの計算でだけifを避ける意味がないですよね。

この手法が有効なのは、 int に対してだけで、double等には使えませんよね。

アセンブリまで考えると、コンパイラの最適化の設定や性能、C#ならCLI自体の性質等、いろんな要素があって、メリットがあるかどうか単純に判断できません。

「難読」というデメリットを受け入れてまで導入するメリットは無いように思います。
// 難読そのものが目的でない限り

お礼

回答・ご指摘ありがとうございます。

> などとやっているなら、valueの計算でだけifを避ける意味がないですよね。
　他の方の回答に書かせていただいているのですが、maskはifを使わずに設定します。それと、またしても忘れていたのですが、ローカル変数でも、引数（あれっ？　引数って、ローカル変数の一種？）でもなく、privateなフィールド変数です。
　TurnSwithOn(), TurnSwithOff()メソッドなどを用いて変更します。

> 「昨日の自分は今日とは別人」
　ちょっと違いますが、「一週間後の自分のコードは他人のコード」という話を聞いた事があります。さすがに、そこまで多量のコードは書いていないので一週間は自分にとって大げさですが、一ヶ月触っていなかったコードを触ったときのギャップとショックと読めなさには驚きました（ショックと驚きって重複しているような）。
　それからは基本的にオープンソースとして他人に見せるつもりで書いています。あと、「コードがドキュメントだ」という他称人間コンパイラさんの言葉に感銘を受けたので、（他人の見そうなところは）コメントなしでも十分に読めるようにはしています。
　といいつつ、時々、（他人が見ない部分、それもあんまりパフォーマンスに影響しそうに無い部分で）極限チューニングに拘ってみたくなったりで、葛藤に苦しんでいます。

　とりあえず、こんなつまらない事に付き合ってくださいありがとうございました。

回答No.2

TreatMeGently

回答者１さんの仰る通り３年後に見て理解し難い、追跡し難いコードは避けるべきだと思います。
　私なら
　value = a+b;
　if( switchOn ) value += c;
　ですね。
　boolean　の　switchOnの違いでcを加算するかしないかが決まるという事が一目瞭然です。

お礼

回答・ご指摘ありがとうございます。

　確かにそうなのですが、とりあえず、c#にはプロパティがあるので、

private const int FullMask = （２進数で111……111）;

public bool SwitchIsOn { get { return mask == FullMask; } }

public bool SwitchIsOff { get { return mask == 0; } }

にするか、または、

public bool GetSwitchIsOn { return mask == FullMask; }

public bool GetSwitchIsOff { return mask == 0; }

　こんな感じにすれば、maskが0のとき、スイッチオンで、maskが111……111のときにスイッチオフだと一応分かるので、理解にそんなには苦労しないと思います（c#っていうと、どんどん極限パフォーマンスから遠ざかるなぁ）。

　まあ、今後アセンブラみないなことはしないと思うのですが予備知識ということで。
　こんなつまらない事に付き合ってください、ありがとうございました。

回答No.1

wormhole

悪くということはないですけどマスクだと対象となる値の一部を取り出すみたいな気がするのであまり使わなそうな。
どちらかといえば三項演算子使うんじゃないでしょうか(if使うのと変わりないけど)
それにmaskの値は結局switchOnで切り替えたりしませんか？

お礼

回答ありがとうございます。

　まあ、普通そういう回答になりますよね。言い忘れていたのですが、アセンブラとかの場合につかえるかな～と思い、質問してみました。
　あと、maskの値は、今実際にswitchOnの変数を使う方法でやっているように、SwitchOn()/SwitchOff()見たいなメソッドで変更するつもりだったので、switchOnという変数は現れません。
　あと、maskを使うにしても、もう少しいいネーミングを検討する予定です（思い立ったがなんとやらで、考える前に投稿したせいで命名してなかったもんで）。例えば、switchMaskみたいに（そのまんまですが）少しは分かりやすいものにはする予定です（使うタイミングなんてあるかどうか分かりませんが）。