LC_Page_Shopping_Payment::action() mode=confirm の deliv_id 補完に疑問
Open
discussion
question
- Dominant language
- PHP
- Stars
- 92
- Forks
- 97
- Avg merge
- 4d 2h
- Merged PRs (30d)
- 9
Description
目視によるソースコードレビュー。動作未確認。
```php
$deliv_id = $objFormParam->getValue('deliv_id');
if (strval($deliv_id) !== strval(intval($deliv_id))) {
$deliv_id = $this->arrDeliv[0]['deliv_id'];
$objFormParam->setValue('deliv_id', $deliv_id);
}
```
IF 処理は、単一選択肢の場合の省略か、不正入力時の代替値投入だろうか。
前者なら判定条件が不適切な気がする。後者なら入力値の変換とエラーチェックの過程で検出し、それでもすり抜けを生じうるのであれば、例外処理とすべき気がする。
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.