操作
レビュー指摘 #12108
未完了KS60350指摘一覧
開始日:
期日:
進捗率:
0%
予定工数:
実開始日:
ＤＤ名:
INCLUDE:
DSN:
バグ区分:
QA分析:
方針（第一貨物）:
実終了日:
ツールのバージョン:
AsIsSource行:
進捗計画:
設計・開発者:
画面ID:
機能区分:
機能ID:
レビュー指摘区分:
記述の仕方
検査完了日:
検査依頼日:
説明
KS60350 移行設計書（クラス設計書） レビュー指摘（誤り）¶
| 項目 | 内容 |
|---|---|
| 対象資料 | クラス設計書_KS60350_ＰＯＳメートマスター ＶＳ 管理台帳.xlsx（表紙／改版履歴／クラス概要／処理フロー／プロパティ一覧／メソッド一覧／メソッド詳細／DB更新仕様／マッピング仕様／処理フロー生成定義） |
| 突合せ基準 | KS60350P_仕様書.md（現行 PL/I ソース KS60350P.pl1 から作成）、管理台帳レイアウト KANBASED・ＰＯＳメート・ＦＡＸサービス SSPOSFAX（KS60350P.xlsx）、仮定の JCL（DAIMNT3J_jobflow_work.svg） |
| 作成日 | 2026-10-26 |
| 関連資料 | 確認事項（漏れ・要確認・軽微）は「KS60350_クラス設計書_確認事項.md」 |
凡例¶
| 区分 | 意味 |
|---|---|
| 【重大な誤り】 | 現行と処理結果が変わり、出力の内容・有無に大きく影響するもの |
| 【誤り】 | 現行と処理結果が変わるもの、または記述が明確に誤っているもの |
- 現行の根拠は、ソースを「pl1 L○○」、仕様書を「仕様書 ○章」と書きます。
- シート番号は Excel のシート順です（1＝表紙、2＝改版履歴、3＝クラス概要、4＝処理フロー、5＝プロパティ一覧、6＝メソッド一覧、7＝メソッド詳細、8＝DB更新仕様、9＝マッピング仕様、10＝処理フロー生成定義）。
現行の処理（要点）¶
| 項目 | 現行（KS60350P） | 仮定の JCL での扱い |
|---|---|---|
| 入力 | ＰＯＳメート・ＦＡＸサービス（VSAM KSDS）をキー順に順次読込（pl1 L22〜L23、L133、L193） | ＰＯＳメイトマスタを キー順（POSQVKEY、POSFAXKB 昇順） で読む |
| 照合 | 荷主コードで管理台帳をキー読込し、管理台帳に無い荷主は出力しない。ＰＯＳ側の請求締日（1件目）が、管理台帳の締日6件のうち正式仮区分＝1 のものと1つも一致しなければアンマッチ（pl1 L139〜L150） | 同じ |
| 出力 | アンマッチリスト（帳票）を読込順に2行1組で印字。毎回 SYSOUT に新しく出力（JCL STEP19 OUTFILE DD SYSOUT=I） |
DAIMNT3J_0090TD に登録（日次処理で帳票にする） |
件数¶
| シート | 重大な誤り | 誤り | 計 |
|---|---|---|---|
| プロパティ一覧 | 1 | 0 | 1 |
| DB更新仕様 | 1 | 0 | 1 |
| 合計 | 2 | 0 | 2 |
上記以外のシート（表紙／改版履歴／クラス概要／処理フロー／メソッド一覧／メソッド詳細／マッピング仕様／処理フロー生成定義）は指摘なし。締日の照合条件（正式仮区分＝1 の締日との一致、請求締日は履歴1件目）と、管理台帳に無い荷主を出力しない扱いは、現行と一致しています。
プロパティ一覧¶
| No | 区分 | セル | 指摘内容 | 対応案 |
|---|---|---|---|---|
| 5-1 | 【重大な誤り】 | AY27（ＰＯＳメイトマスタ VKPFSEQ の取得条件） | ＰＯＳメイトマスタを「取得件数上限（1000件）」と「オフセット」でページングして取得しているが、並び順（ORDER BY）の指定が無い。仮定の JCL では、現行の VSAM のキー順読込（pl1 L22〜L23）に代わり、POSQVKEY・POSFAXKB 昇順で読む。ORDER BY を付けずにページングすると、ページごとに並びが保証されないため、同じレコードを2回処理したり、一度も処理しないレコードが出たりする。その結果、アンマッチリストに同じ荷主が重複して載る、または載るべき荷主が漏れる。さらに、DAIMNT3J_0090TD には並び順を表す項目が無く、帳票の明細順（現行は荷主コード順）も保証されない。 | 取得条件に「ORDER BY 荷主コード[POSQVKEY] 昇順, 区分[POSFAXKB] 昇順」を加える。帳票を作る後続処理で並べ替えるなら、その並び順を書く。 |
DB更新仕様¶
| No | 区分 | セル | 指摘内容 | 対応案 |
|---|---|---|---|---|
| 8-1 | 【重大な誤り】 | BC4（処理：INSERT）、A3〜BO20 全体 | DAIMNT3J_0090TD への登録が INSERT だけで、登録の前にテーブルをクリアする処理がどこにも無い。現行の帳票は実行のたびに SYSOUT へ新しく出力する（JCL STEP19）ため、前回分は残らない。このままでは2回目以降の実行で前回のアンマッチが残り、日次処理で作る帳票に前回分と今回分が混ざる。 | 登録前に DAIMNT3J_0090TD の全件削除を行う処理を追加する（JOB 側で行う場合は、その旨をクラス概要に書く）。 |
ファイル
表示するデータがありません
操作