crackMonsieur crackMonsieur 2013-02-07 22:15:21

[VB.net] 教育的指導 このエントリーをはてなブックマークに追加

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

このプロジェクトではレビューが行き届いていたおかげか、コードの品質が一定に保たれていました。

Public Function Keisan() As Object

    Dim i As Short ' ループ変数
    Dim j As Short ' ループ変数
    Dim iKeisanTmp As Short
    ' その他延々と変数宣言が続く


    ' 中略


    ' 2007/8/32 VB太郎 修正 : 変数はちゃんと上で宣言するように!
    'Dim iKeisanTmp As Integer
    ' 2007/8/32 VB太郎 修正ここまで



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

コメント(3)

#1 test79824007 test79824007 2013-02-08 23:34:55  

「中略」の中に処理があるということですかな? VB太郎の修正はレビューのコメントのみということでよろしいですかな?

#2 c_9_l c_9_l 2013-08-08 12:31:36  

iKeisanTmpが2つあるように見えるのは気のせいだろうか。 中略は中身全部変数宣言じゃないかなぁ…

#3 harunoshirayuki harunoshirayuki 2013-12-04 00:56:02  

一番気になったのが日付なのだけど…32日なんてある月有ったっけ…まあそれ以前に変数のスコープ的に使用する直前で宣言したほうが安全な気が…

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

Twitter認証

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

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

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

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

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

鑑賞する »

[C] すげーちゃんと動く!

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

結合試験2日前に脱退したうんkプログラマ...

return 0;

鑑賞する »

[PHP] 空文字

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

空文字

define("KARAMOJI", "");
$hospital = KAR...

鑑賞する »