日々の破片

_ !!という構文

次のようなコードがあって、一体なんだ？ という話。

で、!!という演算子があるかと思ってGoogleで検索したら、いきなり「!! に一致する情報は見つかりませんでした。」と言われる、即座に。

おれの予想。

きっと、obj#foobar?の実装が腐っていて、苦肉の策ではなかろうか？

たとえば、次のようになっている。Javaプログラマとかだとやってもそれほどおかしかない。

def foobar?(o)
  o == true
end

もし、引数がTrueClassのインスタンスかどうかを判定する目的のメソッドであれば、もちろん正しい実装ではある。

でも、真かどうかを判定しているなら、だめだね。

正しく実装すると

def foobar?(o)
  o != nil || o != false
end

でも、ちょっと冗長だ。単純に、

def foobar?(o)
  o
end

でいいじゃん。でも、いずれにしても、== trueとか書いてあるから、!!（否定によってfalseとなり、それを否定することでtrueとなる）としてTrueClassのインスタンスに変換してから引数としてるんじゃないかな。

いずれにしてもだけど、trueという特殊変数は戻り値、メソッドの引数とかにしか利用しないほうが良い。条件式に記述していたら、たいていは潜在的なバグだと言っても良さそうだ。もちろん == falseというのも、nilを考慮していないからだめだけど。

で、そういう怪しい可能性があるメソッドの引数なので、!!二重否定してTrueClassのインスタンス（あるいはFalseClassのインスタンス）に変換してんじゃないかなぁ、

似たようなのに、COMのC++コンポーネントがVBから正しく呼べないTRUE問題ってのもあったな。

HRESULT STDCALL FooBar(LONG baz)
{
  if (baz == TRUE)
    ...
  return S_OK;
}

となっているのを、

Set obj = CreateObject("FooBar")
obj.FooBar True

とか呼び出すケース。VBのBooleanはVARIANT_BOOL（short）で、TrueはVARIANT_TRUE（-1）だということを、C++小僧が知らないという話。

結局、ベストプラクティスとして、偽（0）かどうかで判定するというのが正解だろう。

HRESULT STDCALL FooBar(LONG baz)
{
  if (baz)
    ...
  return S_OK;
}

しかも、この方が==演算子とか書かなくて済むだけ簡潔だ。

_ おれは世界一不幸な少年かも知れないが

動くな、死ね、甦れ!【字幕版】 [VHS](パーベル・ナザーロフ)

かって、VHSでは出ていたのか……