marinyan 2012-11-26 03:15:39

[その他] 実運用中のスクリプトでそれはやめてください。 このエントリーをはてなブックマークに追加

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

コメントアウトしたらもちろんsyntax errorだらけで動きませんでした…

on error resume next

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

コメント(6)

#1 InternalServerE 2012-11-26 16:06:14  

実装方法として、On Error Resume Next自体は運用中のコードにも普通に使われうるわけですが、それをNGだと言っているのかしら。
syntax errorだらけなことが、どうして「もちろん」で繋げられてるのか、イマイチ分からない。

#2 yosilove 2012-11-29 16:07:11  

構文エラーをこれでごまかしているのか・・・これは危険。

#3 keiichiroh 2012-12-01 01:30:38  

いや、構文エラーは文法解析時にエラーとなるため、投稿者が分かってない可能性がある。 例えば CreateObject などはエラーを起こしてからチェックするしかないから on error resume next は必須テクニックだったりする。

#4 keiichiroh 2012-12-01 02:26:14  

WshShell の Exec メソッドを使う場合も、on error resume next の中で Err をチェック。なんて定石は分かって書いてるのだろうか?

#5 okadabasso 2012-12-01 10:40:32  

VBSだと On Error Goto が使えない、Try Catchもないゆえに Resume Next して逐一Errオブジェクトをチェックするしかない。これがVB6やVBAならOn Error Gotoを使うべき?環境がわからないのでなんとも。構文エラーならそもそもまともな結果が得られてないはず。

#6 keiichiroh 2012-12-04 16:24:26  

VBAとかVBSでひと目でコード品質を見分けるならば、まずはOption Explicit の有無かな。無ければ高確率でうんこ。

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

Twitter認証

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

[その他] HOW TO ABC..

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

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »

[Java] 連番

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

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C] ( 0][0 )/

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

とある組み込み系のコード。元気に動いては...

#define NUM_PORT 8
#define NUM_CHANNEL ...

鑑賞する »