Yamane_D 2012-08-14 04:26:54

[VB.net] "TRUE"に完全一致しなければ処理しません! このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

以前コードレビューを頼まれた中に入っていたコード(レプリカw)。

なんといっても文字列Booleanで関数利用者が仕様を知っているかどうかを厳密にチェックする、厳密規定派プログラミングスタイルが硬派です。

そして、戻り値も徹底した文字列主義で利用者は最後まで安心できません。徹頭徹尾、俺の流儀に従ってもらうという姿勢が強く表れています。

    Public Function ModeKirikae(ByVal flag As String) As String

        If flag = "TRUE" Then
            'コントロールの状態変更処理(省略)
            Return "TRUE"
        Else
            Return "FALSE"
        End If

    End Function

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(3)

#1 math_neko 2012-08-14 18:32:16  

素直に Boolean 使えやwww

#2 Miraranran 2012-08-15 03:20:09  

Return As String なあたり、こいつ確信犯です。

#3 damedom 2012-08-15 03:41:29  

「大切なフラグをたった1ビットだけに任せられるか!」という姿勢が硬派で好感が持てます。

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] フルHD推奨。

このエントリーをはてなブックマークに追加

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[Java] スベテカンスウ

このエントリーをはてなブックマークに追加

何でもやってくれる関数って、便利ですよね...

class Processor
{
	public bool process...

鑑賞する »

[C#] コーディング時最適化

このエントリーをはてなブックマークに追加

最適化の効いた良いコードだと思います。 ...

static void Main() {
    Console.WriteL...

鑑賞する »