水曜日に発覚した「同日に同一銘柄を複数回買った時に監視してくれない」バグの
修正をはじめとした各問題に対応するため、ソースコードを大幅に修正しました。

上記問題は、監視銘柄が売却され実際のDBからは監視銘柄が削除されているのに
メモリー空間に保持しているDataTableにはまだレコードが残っていて、あたかも
「まだ監視している」状態に見えたから次に同じ銘柄を購入しても既に監視中だと
みなされて新しく監視を始めてくれないというのが原因でした。
これは私のVB2005に対するスキルが低いから起こったバグで、恥ずかしい事です。

この問題を解決する為に、定期的にDataTableを作り直す(TableAdapterからFillする)と

でもテストをするとガンガン落ちまくり(滝汗)。
しかも「ユーザーがハンドル出来ない例外」になり、いわゆるWindowsアプリが
「ご迷惑をお掛けいたします」などとダイアログを出して突然落ちる状態なので
何もログが残らず最悪な状態です。。。
今までこんな事が無かったので焦ったのですが、どうやら監視銘柄のDataTableに対して
非同期処理が追加したり変更したりしてる中で別の処理がDataTableをクリアしたり
読み直したりする事によって起こる「マルチスレッドのハンドリングミス」に
よるものみたいです。
私はJavaでもマルチスレッド処理は苦手な分野なので焦りました。。。
とりあえず割り込まれたら整合性がおかしくなる所を「SyncLockブロック」で囲み
同期化させました。Javaでいうところのsynchronizedブロックです。

「マズい。。。」と落胆しましたが、もしかしたらDataTableのクリアやFillを
行わなくてもDataTableのAcceptChangeメソッドによって綺麗にしてくれるか、
あるいは削除されたレコードにはちゃんとRowStateに削除された印が付いているので
読み飛ばせば良いのでは無いかと思いました。
但しこれについては現在マケスピにログイン出来ない状態なので今晩試そうと思います。

それ以外に「分かっていたけど運用でカバーしていた」不具合・・・と言うか未実装の
ロジックがありました。

まずは「同一銘柄の買い増し対応」。既に売り注文のための監視を行っている銘柄を
買い増した時、株数が合わなくなって理想的な動きをしてくれないと予想してました。
例えば10株購入してその監視を行っている時、さらに5株買い増した場合。
トレーリングストップにより10株が売れてから5株の監視を始めていたと思います。
しかも5株を購入後の値動きは繁栄されず、10株が売れた時点からのトレーリングストップで
売られます。それではいけないと思い、買いましたら監視銘柄の株数も増やすように
ロジックを加えました。これで今まで敢えて避けていた同一銘柄の買い増し」が
出来るようになりました。

次に「差金取引の問題」。
持ち越した銘柄を当日売って、売った資金でその銘柄を買い戻した場合、当日中は
売る事が出来ません。でも今のKATSなら必死で売り注文を出して私の携帯にむやみに
メールを送りつけるでしょう。売れないのに(苦笑)。
それが予想出来たので運用でカバーしていました。
でも今回、保有銘柄一覧を取得した時にその銘柄に「注文可能か」を表す区分があって
それを判断して注文不可の保有銘柄なら監視しないというロジックを追加しました。
本来売る事が出来ない差金取引をする事自体が問題かも知れませんが、もしそうなっても
KATSがおかしな動作をしないようにしました。

あと修正したのは、

・設定に関する一式を外部ファイルに移動
・メッセージ文言などをリソースファイルに移動
・メソッドを然るべきクラスへ移動(リファクタリング)
・信頼された証明書の登録

などです。思いっきり手を入れました。
メンテ性向上、一元管理などを理由にしたものが多いです。
今後KATSのソースコードを他の方に見せる機会も多くなると思っているので
少しでも汎用化させて私のローカル設定を外出しするようにしています。

かなり大掛かりに修正したので今晩は念入りにテストしないと来週が恐いです。
明日は卓球の試合ですが、明日の晩までもつれこんだら厄介なので今晩完成させないと。

お気に入りの記事を「いいね！」で応援しよう