【C言語】三項演算子は悪者なのか？

 C言語で条件に応じた代入をするときはif文を普通に使っていると思います。ただ、テキストなどには「三項演算子」という方法も書かれていると思います。

三項演算子を使うとコードは非常にスッキリとします。しかし、if文と比べると何をやっているのか良くわからないという問題、つまり可読性の問題があります。

じゃー三項演算子は封印？

それでもいいんじゃないですか？と言っても良いかもです。とにかく可読性のことを考えるとif文が分かりやすいわけだから。

しかし、if文って長いです。elseを使って場合によって２つの値を入れる場合は

1.     if(z<0){a=msgNeg;} else {a=msgPos;}

となります。さらに場所を節約するために一行にしているので、可読性は下がっていると考えます。であれば、同じ表現を

1. b=z<0?msgNeg:msgPos;

のように三項演算子で書けば良いのでは無いでしょうか？そして、何をやっているかコメントで残しておけば十分理解できると考えます。

簡単な結果だけに適用する

冷静に考えると、三項演算子で見にくい場所は、判断結果の収納場所です。判断を書く場所はif文のカッコの中と同じです。

なので、結果によって単純な定数を代入する程度の処理だけに使って下さい。もちろん、三項演算子を使ってこのような処理をしています。というコメントは必須です。

以下はIF分と三項演算子を使った比較サンプルです。

1. #include<stdio.h>
3. const char *msgNeg = "負の値";
4. const char *msgPos = "正の値";
6. main(){
8.     int z;
9.     const char *a,*b;
10.     
11.     printf("z=?");
12.     scanf("%d",&z);
13.     
14.     //ｚが正か負の値かでメッセージを表示する。
15.     
16.     //zが正か負かでメッセージを変更する。bにメッセージの文字列先頭が入る。
17.     if(z<0){a=msgNeg;} else {a=msgPos;}
19.     //三項演算子を使った方法。コードはスッキリするが初見ではわかりにくい
20.     b=z<0?msgNeg:msgPos; //zが正か負かでメッセージを変更する。bにメッセージの文字列先頭が入る。　三項演算子を使用。
21.     
22.     printf("a=%s\nb=%s\n",a,b);
23.     
25. }

このような単純な処理であれば、しっかりとコメントをした上で三項演算子を使いコードのスペースを削減するのが良いと考えます。

馴染みのない複雑なアルゴリズムを記述する場合は、丁寧にIF分を使うことをおすすめしますが、単純で誰でも分かるロジックであればコード単体での可読性より、文字の少なさを優先したほうが良いと考えます。

そして、簡単なコードがコンパクトになることにより、より高水準のやりたいことに集中出来ると考えます。

AmazonでC言語の本を調べる