操作
レビュー指摘 #12413
未完了利用運送_直集・直配_兼用_指、送No_センター照会 メソッド 現行を再現できていない 新の処理の流れにそぐわない
開始日:
期日:
進捗率:
0%
予定工数:
実開始日:
DD名:
INCLUDE:
DSN:
バグ区分:
QA分析:
方針(第一貨物):
実終了日:
ツールのバージョン:
AsIsSource行:
進捗計画:
設計・開発者:
画面ID:
機能区分:
機能ID:
レビュー指摘区分:
処理の方法
検査完了日:
検査依頼日:
説明
利用運送_直集・直配_兼用_指、送No_センター照会 メソッドに 問題点あり
■1.メソッド側でのエラーが握りつぶされている
メソッド側でエラーがあった場合、エラー区分 にセットしているが、
呼び出し側で何も見ていない。


エラーがあった場合に処理を続行してしまうようになっており、問題。
現行はCICSからの戻り値で
データ区分でエラーのセット(OKFLG='1')や
メッセージのセットを行っている。

メソッドの戻り値でもプロパティ経由でも良いので、エラーであれば、
dataProcessMode2 側で 処理エラーとする。 の指示が必要。
■2.不要なCOMM_AREA_DETAIL 及び 構造体
現行のSVとCICSのIncludeを模倣して COMM_AREA_DETAILと構造体を用意したものと思われるが、
modeを跨がない為、COMM_AREA_DETAILは不要。
構造体に入れるような内容もほぼ無いため、項目毎にプロパティ等を使うことで代替可能。



※構造体は複雑な処理の為、一旦複数の値を格納しないといけない、
処理のタイミング的に現行のIncludeの変数と同じように枠があった方が都合がいい、
別のmodeにInclude変数の中身丸ごと引き継ぐ必要がある
と言った場合は、是非とも構造体を作っていただきたいです。
■3.そもそもメソッド化が必要か
複数回呼ばれない & 内容もそこまで無い 為、
dataProcessMode2内に書いてしまってもよいぐらいの内容。
プロパティではなく変数で済む用になる為、
今回はメソッド化せずに進めても良い。
ファイル
表示するデータがありません
操作