zunbe

2005/12/08

初心者にありがちな誤ったプログラム例をひとつ。

Visual Basic で書かれているプログラムで、以下の様なコードを最近、よく目にする。

--
Call MsgBox("プロンプト", vbCritical + vbOKOnly, "タイトル")
--

このコードには、根本的な間違いがある。
この部分である。

　vbCritical + vbOKOnly

「vbCritical」と「vbOKOnly」に、値がどのように定義されていて、どのような結果を求める必要があるかを考えれば、「+」で演算を行ってはならない事は明白である。
ここで行おうとしている演算は、「vbCritical」と「vbOKOnly」が持つ値のビットパターンを演算し、立っているビットをすべて残す事が目的なのであるから、行うべき演算は「+」ではない。
当然、この様に記述するべきである。

　vbCritical Or vbOKOnly

よほど変な組み合わせを書かなければ、「+」で記述しても誤動作する事は無い。
しかし、そんな事は問題ではない。
関数や処理が求めている演算方法を正しく理解し、適切な演算子を使用することは、プログラマとして当たり前にできなければならない事だと思う。
しかし、最近は、「動くからいいじゃん」的な考えのプログラマが多いような気がしてならない。

若いプログラマは、もっともっと勉強してほしいと思う。

このエピソードはいかがでしたか？

投稿者 zunbe : 2005/12/08 03:18:02