前々から思っていたことですが、たまたま以下の記事を見つけたのでこの際書くことに。

[.NET] Re: 例外について (biac の それさえもおそらくは幸せな日々@nifty 2009/8/10)

これ、全面的に同意。今やってる仕事である人がやった部分には以下のようなのが至る所に。

public bool f1(MyEnum anEnum)
{
  try
  {
    if(anEnum == MyEnum.Item1) return true else return false;
  }
  catch(Exception)
  {
    /* なにもないよ～ */
  }

  return false;
}

関数1行目がtry。勘弁してよ関数全部例外無視ってorz if文で値比較するだけのどこに例外が出ること恐れてるのよ、むしろ出たときの方が知りたいぞ。

別の先輩の話。Visual BasicにはOption Explicit という機能がありまして、これを指定すると全変数をあらかじめ宣言しないとコンパイル時にエラーを出すというもの。一見面倒くさくなるだけの機能に思えますが、変数名の打ち間違えを未然に防いでくれる ので、この辺で悩んだ事のある人には便利な機能のはず。Visual Studioでは新しいVBファイルを作るたびに自動的にこの行を入れてくれる設定があります。ところが彼は「Option Explicitを削除するとコンパイル時のエラーが減るんですよ」と、耳寄り情報のように教えてくれたそうで。

製品としてリリースした後ならともかく、開発時やら説明すれば理解してくれる限定ユーザー向けならば、エラーは恐れずどんどん表示してください。放置すると後で分けの分からない現象に悩んだり、たまたま動いているプログラムで満足しているといいプログラマーにはなれないと思います。お願いします、本当に… (直すこっちが迷惑だorz)

[2010/8/31 追記] 前者「関数1行目がtry」な方が書いたコードを久しぶりに見直したら、以下の部分が orz 意味が無いにも程がある。関数の書き始めはtryが癖になっているらしい。

public void foo()
{
  try
  {
    処理
  }
  catch (Exception e)
  {
    throw e;
  }
}

ぴったり当てはまる例が思いつかないので従業員管理の場合。一人の情報を受け渡すために、以下の構造体を定義したとします。

struct Employee
{
  int SerialNumber;  // 従業員番号
  char* OfficeRoomNumber;  // 会社での部屋番号
  char* Name;  // 氏名
  char* Address;  // 住所
  int Age;  // 年齢
}

会社では一人にひとつの部屋を割り当てるとした場合、SerialNumberとOfficeRoomNumberの両方が従業員に対して一意の情報となれます。ここで従業員を登録するために

void SetEmployee(Employee* anEmployee);

を定義。関数を呼ばれたときには引数の内容をデータベースに記録するというもの。ここまでは良かった。次にデータベースから削除するために、この関数に引数を追加、

void SetEmployee(Employee* anEmployee, int add);

「add==0ならanEmployee->OfficeRoomNumberと合致するものを削除」と仕様書に明記されました。私なら新関数RemoveEmployeeとか定義しそうですが、まぁ決めた人の都合です。

まだここまではOK。ところが後になって、引数anEmployee->SerialNumberだけ正しい情報を入れてadd=0で関数を呼ぶ例が出てきました。どっちも一意な情報なのでデータの削除にはこれだけあれば十分と思ったのでしょうが、呼ばれる方はそういう意図は知りません。当然誤動作します。

余計な情報まで含む構造体を引数として使うと定義が曖昧になり、本来必要な情報を入れずに関数が呼ばれる可能性があります。一人でプログラムを組む場合は事情が理解できているので問題とすら思わないかもしれませんが、チームでプログラムを作るときは面倒でも引数は必要十分にしたほうが良さそうです。

結局関数にさらに引数を追加、

void SetEmployee(int SerialNumber, Employee* anEmployee, int add);

SerialNumberが必要な情報だよと関数の定義レベルで明示するように私のほうで変更。案の定「構造体に入ってる情報をなぜ引数にも入れているのか」と質問が来ました。

ちなみに「関数を定義した人」=「仕様書を書いた人」=「SerialNumberだけ正しい情報を入れた人」=「質問してきた人」だったりします。まったく…

以上、今日のお仕事でした。

[2010/6/15 追記] 結局SetEmployee(Employee* newEmployee), RemoveEmployee(int SerialNumber)の2関数体制に変更

最初に話を聞いたのはGW前、具体的な内容は伏せるとして要望された内容は

とあるスポーツに関連するのですが、1/1000秒で時間を計るなんてF1等レーシングカーくらいしか聞いたことないなぁ。使用するスイッチもチャタリングの入りそうなものになりそう。依頼主もその辺の事情は理解してくれていてあまり正確にmsecにこだわらなくて良いとは言ってくれてますが、一応形的にはmsecにしたいところ。またパソコンは持ち運びが容易でないと何かと不便なのでノートPCも考慮に入れる必要があり、拡張ボードもかなり制限される。

ちなみにWindowsが提供するmsec単位の時間関数を使うという答えは残念ながら×。時間関数の精度はともかくとして、Windowsの仕組み上スイッチがOn/Offされた瞬間(もしくはそこから毎回安定した遅延で)時間関数を呼び出すことはかなり難しいから。HDDアクセスが入ったり、CPUの負荷状態が変わってスレッド切り替えに変化が生じると、msec平気でばらつきそうです。

以下検討の過程

カメレオンUSB+ロジアナキット

依頼主が考えていたのはカメレオンUSB +ロジアナキットでサンプリングレートをkHzオーダーにして計るというもの。USBでデータ転送するのでノートPCでもOK。 私も最初この方法で調べてたのですが、ひとつ疑問が。「ロジアナキットって、何分も連続してデータ取れるの?」なんか構造を見る限り、ロジアナキットのボード上にあるメモリに書き込んでいって、たまったらUSBで転送。転送中はデータ取得できないんじゃないかと。機能にしては安いとはいえ1万円以上するこの回路、買ってやっぱりだめでしたではお金がもったいない。

サウンドカード

連続データ取得が期待できる。WindowsはそれこそHDDの許す限り録音をやり続けるし、サンプリングレートも44.1kHzあれば十分、ノートPCにだって標準で付いている。世の中にはサウンドカードを利用したオシロスコープ もある。スイッチのOn/Offを録音信号レベルに置き換えて入力すれば、Windows側はDirectSoundでデータを取り込んでいけばいい。が、これもサウンドカードは低周波をまともな波形で取れない という欠点が発覚。チャタリングの素性次第では何とかなるかもしれませんが。

再びカメレオンUSB

今度はカメレオンUSB単体。高速で連続データ取得 する実例もあるので、できないわけじゃない。ただその場合カメレオンUSBで使われているアルテラ用のプログラムを書く必要があり、そんなの使ったことないorz どうせ勉強しなければいけないなら、PICでもAVRでも変わりないなと。

というわけで、現在ストロベリーリナックスのAVR AT90USB162を使ったボード を検討しています。こいつの利点としては

• 開発環境がCで無料で入手可能
• CDC(USBをCOMポートに見せかける機能)のサンプルコード が入手可能。COMポートなら.NET FrameworkとのつなぎにUnmanaged C++を使わなくても良さそう。
• このボード単体でUSBからプログラムの書き込み可能。ブートローダ機能で何度でも書き込み可能状態に戻せる。

さぁて、うまくできるかなぁと。

[2010/6/25 追記] 続き