■地獄の社内ＳＥ

社内ＳＥになった。

仕事を辞めて主夫業に勤しんでいたら、知り合いから声がかかった。

１人で社内システムを作ってきたおじいさんがあと数年で定年になるから、

引き継げないかとのこと。

メインのシステムはベンダーに委託してて、そのおじいさんが作っているのは、

メインシステムのデータを加工して２次利用しているものがほとんどとのことだった。

社内ＳＥはなんとなく楽そうなイメージがあったので、就職した。

言語はエクセルＶＢＡとＶＢ．ＮＥＴ　１.0。

中身を見るとどちらもかなりやばい。

ＶＢＡ編

・ウォッチウインドウを知らないのか、変数はすべてセルに入れてる。

　変数名はすべてRANGE("A1").valueみたいな感じで全く意味が分からない。

・処理遷移がおかしい。

　セルに１を入れる。そのセルのchangeイベントで処理が動くとか。

　SHIFT+F2が無力化されてる。

・なるべくワークシート関数で処理してる

　データベースからとってきたデータを丸ごとワークシートにコピーして

　if,vlookup,match関数を駆使して帳票にしたり、ＣＳＶにしてる。

　データ１００件制限があったり、１関数を直すときは１００行コピーしないといけない。

　画面中に埋め尽くされたワークシート関数をみて途方に暮れる。

・format関数を知らない。

　８桁の日付をとりたいときyear、month、day関数がワークシートにあり、

　その下の行で月の二けた判定、日の二けた判定のif関数で頭の０をつけ

　３行目でconcatenateしている

・タイマー起動

　毎朝１００本ぐらいのマクロが動いてる。

　タイマー起動なので、毎日セットしないといけない。（タスクスケジューラーを知らない）

　がんがんエラーが発生するので、マクロ設定をエラー処理対象外のエラーで中断にしないと動かない。

・遅い

　textboxのchangeイベントでＤＢからのデータ取得処理を入れているので、データが多くなると１文字打つごとに数分待つ状態。

　exitイベントを知らないらしい

　ＤＢの更新処理でもテーブル全件とってきて、ループしながらキーが一致するのを探して更新。

そんなつっこみどころしかないＥＸＣＥＬマクロが２００本以上ある。

ＶＢ編

・．ＮＥＴ１．０

　windows7や８に無理やり.netframework1.0を入れて動かしてる。

　顧客のＰＣにも入れてる

・オブジェクト名は代えない

　変えられることを知らないのかもしれない

　textbox100とか存在してる。

　ＥＸＣＥＬ同様変数は隠しtextboxに入れてる。

・設定ファイルがおかしい

　１．０なのでconfigがないのはしょうがないが、設定ファイルは固定パスのテキスト

　行数で管理

・ＷＡＩＴがいっぱいある

　試しに取ったら動かない

・ＤＢを最後まで回すとき

　例外が発生するまでまわす。

・変数や関数のスコープの管理

　ない。基本グローバル。

・クラス

　ない。

・ネスト

　ない。

おじいさんが２０年にわたって深夜残業や休日出勤を厭わず作ってきた、地獄の社内システムの担当になったらしい。

ツイートシェア

Permalink | トラックバック(1) | 00:56