フォームの編集・追加時のVALUE値の有無でフォームの使いまわし

空のフォームに入力して追加
追加した内容を参照し編集

この2つの機能の場合にフォームを使い回せるのでしょうか？
以下のような郵便番号欄があったとします。

<input type="text" name="zip1" maxlength="3" style="ime-mode: disabled;"
                 value="<?=isset($address_rows['zip1']) ? h($address_rows['zip1']) : ''?>" />

編集時にも同じフォームを流用して
作成時はvalue値になにも入らないので、そのまま入力してCREATE
編集時にはデータベースから取得した値が入るので、編集などをしてUPDATE

上記のソースのような書き方にしてもよいのでしょうか？
外側のVALUEそのものもif文で囲んだほうがいいのでしょうか？

よろしくおねがいします

行動規範の内容に同意します

回答2件

ベストアンサー

登録時も更新時も共通なのは「バリデーションに引っかかった場合はPOSTの値」となるかと思いますので、そこをきちんと考慮すれば良いかと思います。

※丁寧に書いた場合

php
1$value = ''; //初期空（新規）
2
3//データがあるか (編集)
4if(isset($data) && count($data) > 0){
5  $value = $data['hoge'];
6}
7//POSTがあるか（新規・編集）
8$hoge = filter_input(INPUT_POST,'hoge');
9if(!is_null($hoge)){
10  $value = $hoge;
11}
12
13echo >>>FORM
14<input type="text" name="hoge" value="{$value}">
15FORM;
16

投稿2019/12/16 22:49

m.ts10806

総合スコア80875

m.ts10806

2019/12/16 22:58

「正しいか」というと、「要件通りに動くなら正しい」としか言いようがなかったりします。コードは要件通りに動くように書くものであって絶対の基準があるわけではありません。ひとつのことを表現するためにとれる手法はひとつではないので。

creative_09

2019/12/17 01:37

ありがとうございます。動作的にはどちらの方法でも、バグを引き起こす原因であったりなどはなさそうで安心しました。ですが、回答いただいた内容のように簡潔で見やすい書き方がいいです。複数項目があった場合など、調べながらしてみたいと思います。ありがとうございます

m.ts10806

2019/12/17 01:45

三項演算子やNULL合体演算子などは確かに便利で高機能ですが慣れてないと挙動が想像しづらく実装に漏れが起きやすくなります。そういうときはまずベタで書いてみると良いです。自身が挙動を把握しきれないコードを書いてもメンテナンスしづらくなりますしね。複数項目などは、まさに設定ファイルのような形でそこか、タイプや初期値など分岐させると汎用性の高いコードが書けると思います。共通化できそうなところは変数にしたりですね。

creative_09

2019/12/17 02:22

ここは三項演算子でなくても問題はないですよね？値がなければvalue=""になるだけで、エラー表示もなさそうなので。これは認識はあっていますか？おっしゃるとおり、複雑化しそうで、今回はすべて別ファイルだったフォームのCRUDを一つのフォームで使いまわしたいという練習ですので、三項演算子などは省き、少しシンプルにしていきたいと思います認識に間違いがあるようでしたらコメント頂ければ幸いです

creative_09

2019/12/17 02:23

value="<?= h($address_rows['zip1']) ?>" にするだけということです

m.ts10806

2019/12/17 02:40

$address_rowsと各キーを確実に定義するのであれば良いと思います。ただ、編集時にDBなどからとってきたデータと変数名が被ったり紛らわしくならないかどうかは検討されたほうが良いですね。

creative_09

2019/12/17 04:49

ありがとうございます。理解が深まりました。

行動規範の内容に同意します

フォームやページを新規登録と編集で使いまわすことはそれほど珍しいことではないと思います。
よくあるのは会員情報の入力画面を新規登録と編集で同じページを切り替えて使うとか。
その場合はフォーム１つだけではないので、新規登録か編集かの判定はフォームの入力有り無しではなく、会員IDが空なら新規登録、会員ID設定済みなら編集と切り替えたりします。

投稿2019/12/16 14:51

javahack

総合スコア1088

creative_09

2019/12/16 15:38

回答ありがとうございます。その切替ですが、新規登録であっても編集であっても共通のフォームにしたい場合、編集の場合は登録されているデータを読み込み、value値へ挿入しますが新規の場合は必要ない状態となりますそこで正しくは上記のように value="<?=isset($address_rows['zip1']) ? h($address_rows['zip1']) : ''?>" としてもよいのか、 if (isset($address_rows['zip1'])) ｛ echo value="・・・・・・・・・・・"; } のようにvalue=の外側でするのが正しいのかどちらなのでしょうか？という質問でした。。説明が下手で申し訳ございません

行動規範の内容に同意します

あなたの回答