出来るのだＡＳＰＱ＆Ａ掲示板（過去ＬＯＧ）

出来るのだＡＳＰＱ＆Ａ掲示板（過去ＬＯＧ） 　訪問数 52046 昨日 889 今日 776
【PR】パソコン入門からIT専門書まで幅広く取り揃えています。セブン-イレブン受取り手数料無料のセブンアンドワイ。

Topに戻る

掲示板に戻る

検索

削除

管理者

[5580] Re:ソースファイルの書き方が下手くそですが・・・

コメントする

投稿者：Lightningさん 2006/01/12 9:39:48

> お世話になっています。私のプログラムは非常に読みづらいと先輩に言われました。実は、初めてaspでプログラムというのを組んだのですが、具体的にどの変が読みづらいでしょうか？何かプログラム一部を簡略化するのか、はたまた改行などをして読みやすくするのか、変数の宣言の仕方が下手だとか、技術面もろもろといろいろ中傷願います。
>

> > お世話になっています　Ligthningさん
> > if request.form("year") = "" then
> > iyear = year(now)
> > else
> > iyear = request.form("year")
> > end if
> > if isnumeric(iyear) then
> > if 0 > iyear or 9999 < iyear then
> > iyear = year(now)
> > end if
> > else
> > iyear = year(now)
> > end if
> > ここの文なのですが、入力するところが、空欄ならば現在の年を表す、もし何か数値が入ってればその数値をiyearに返す。さらにその数値は
> > 整数であって、0～9999の間の数値ということで解釈していいんでしょうか？
解釈はそれであってます。
入力数値の監視機構が存在しないとバッファオーバーフロー攻撃が可能になったりと大変なので入力値の監視は必ずしてください。

読みにくい部分、ですが、何度か組めばローマ字は駄目だとわかると思います。
母音が被るのでよく間違えるし、今回の例だと「saisyuubi」なんかは「saisyubi」と書き、実行時に何が違うのかわからない落とし穴にはまる可能性があります。

あとは、タグの書き方とか気になったかも。
閉じてないとか多々あったし。
あと、入れ子構造とかが多発しているのも気になった。
仕方ないとはいえ、なるべく避けたほうが良いです。

あと、カウンタに使う変数は正規教育を受けた場合、i,jあたりかn,mあたりに落ち着きます。
これは結構気になる人がいたりするので注意です。

あとはー
エディタによりますが、コメントの場合は色を変える設定をしておけば、ifごとにどういう分岐かのコメントをつけておくと目立ったりとか。
掲示板書き込み時に崩壊したのだと思いますが、意識してインデント（行頭にスペースをつけることで入れ子などの構造を見えるようにする手法）をするとかしてみてください。

とかかなあ・・・
私もあんまりきれーなソースじゃないからなあ・・・
#vbcrlf（改行コード）とvbtab（タブ）は普通はいらないですが、table構造やformを書くときは意識して入れるとHTMLソースを確認するときに便利ですよ～

削除する