編集履歴

回答編集履歴

修正

2019/05/01 08:23

投稿

スコア80850

test CHANGED Viewed

@@ -330,7 +330,7 @@
 	    echo '<p>';
-	    if(ak('required',$setting)){
+	    if(ak('required',$setting) && $setting['required']){
             echo '(必須)';

修正

2019/05/01 08:22

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -52,4 +52,492 @@
 ---
+入力画面includeを生かして適当に組みなおしたコード
+適当に役割分担。Validationのところとか入力フォーム表示とかClassでやったほうがそれっぽい感じになるので望ましい。（今回の内容だけでもオレオレフレームワーク一歩手前）
+※[返り値の型宣言](https://www.php.net/manual/ja/functions.returning-values.php#functions.returning-values.type-declaration)とか使ってるのでPHP7以上でのみ動作可能
+helper.php
+```php
+<?php
+//二重定義阻止
+if(!function_exists('h')){
+    function h($str)
+    {
+        return htmlspecialchars($str, ENT_QUOTES, 'UTF-8');
+    }
+}
+if(!function_exists('ak')){
+    function ak($key,array $array):bool
+    {
+        return array_key_exists($key,$array);
+    }
+}
+if(!function_exists('iar')){
+    function iar($key,array $array):bool
+    {
+        return in_array($key,$array);
+    }
+}
+```
+input_config.php
+```php
+<?php
+return [
+    'name'=>[
+        'label'=>'名前',
+        'type'=>'text',
+        'required'=>true,
+        'validation'=>'required'
+    ],
+    'age'=>[
+        'label'=>'年齢',
+        'type'=>'text',
+        'required'=>true,
+        'validation'=>'required|number'
+    ],
+    'color'=>[
+        'label'=>'好きな色',
+        'type'=>'checkbox',
+        'select'=>['赤','緑','青']
+    ],
+];
+```
+validation.php
+```php
+<?php
+function validation():array
+{
+    $input_config = require_once 'input_config.php';
+    $err = [];
+    foreach($input_config as $input_name=>$setting){
+        if(ak('validation',$setting)){
+            $vali = explode('|',$setting['validation']);
+            foreach($vali as $checktype){
+                $check_name = $checktype.'Check';
+                $res = false;
+                if(function_exists($check_name)){
+                    switch($checktype){
+                        default:
+                            $res = $check_name(filter_input(INPUT_POST, $input_name),$setting['label']);
+                    }
+                    if($res){
+                        $err[$input_name][] = $res;
+                    }
+                }
+            }
+        }
+    }
+    return $err;
+}
+function requiredCheck(string $value,string $name)
+{
+    if(trim($value) === ''){
+        return $name.'を入力してください';
+    }
+    return false;
+}
+function numberCheck(string $value,string $name)
+{
+    if(!empty($value) && !(ctype_digit($value))){
+        return $name.'は整数を入力してください';
+    }
+    return false;
+}
+```
+test1.php
+```php
+<?php
+require 'helper.php';
+$input_config = require 'input_config.php';
+if(!isset($err)){
+    $err = [];
+}
+?>
+<!DOCTYPE html>
+<html lang="ja">
+<head>
+<meta charset="UTF-8">
+<title>入力</title>
+<style>
+h1 {
+	color: #f00
+}
+.error {
+	color: #f00
+}
+</style>
+<script>
+function check(){
+    if(!confirm('送信してよろしいですか？')){
+        return false;
+    }
+}
+</script>
+</head>
+<body>
+	<h1>お問合せ画面</h1>
-後ほど適当に組みなおしたコードを投稿するのであとは頑張って理解してみてください。
+	<h2>お問合せ内容を入力してください</h2>
+	<form action="test2.php" method="post" onSubmit="return check()">
+	<?php
+	foreach($input_config as $input_name=>$setting){
+	    echo '<p>';
+	    if(ak('required',$setting)){
+            echo '(必須)';
+	    }
+	    echo $setting['label'];
+	    switch($setting['type']){
+	        case 'text':
+	            $input_data = filter_input(INPUT_POST,$input_name);
+	            echo '<input name="'.$input_name.'" value="'.h($input_data).'">';
+	            break;
+	        case 'checkbox':
+	            $input_data = filter_input(INPUT_POST,$input_name, FILTER_DEFAULT, [
+	               'flags' => FILTER_REQUIRE_ARRAY
+	            ]);
+	            if(ak('select',$setting) && is_array($setting['select'])){
+	                if(is_null($input_data)){
+	                    $input_data = [];
+	                }
+	                foreach($setting['select'] as $in=>$val){
+	                    echo '<input type="checkbox" name="'.$input_name.'[]" value="'.$in.'"'.(iar($in,$input_data)?' checked':'').'>'.$val.'&nbsp;';
+	                }
+	            }
+	            break;
+	    }
+	    echo '</p>';
+	    if(ak($input_name,$err)){
+	        echo '<span class="error">'.implode('<br />',$err[$input_name]).'</span>';
+	    }
+	}
+	?>
+	<input type="submit" value="送信">
+	</form>
+</body>
+</html>
+<?php
+exit();
+```
+test2.php
+```php
+<?php
+if($_SERVER['REQUEST_METHOD'] !== 'POST'){
+    require_once 'test1.php';
+}
+require 'helper.php';
+require_once 'validation.php';
+$err = validation();
+if (count($err) > 0) {
+    require_once 'test1.php';
+}
+$input_config = require 'input_config.php';
+?>
+<!DOCTYPE html>
+<html lang="ja">
+<head>
+<meta charset="UTF-8">
+<title>送信</title>
+</head>
+<body>
+	<h1 class="red">お問合せ完了画面</h1>
+	<h2>送信しました</h2>
+	<table border="1">
+	<?php
+	foreach($input_config as $input_name=>$setting){
+	    echo '<tr>';
+	    echo '<td>'.$setting['label'].'</td>';
+	    echo '<td>';
+	    switch($setting['type']){
+	        case 'text':
+	            $input_data = filter_input(INPUT_POST,$input_name);
+	            echo h($input_data);
+	            break;
+	        case 'checkbox':
+	            $input_data = filter_input(INPUT_POST,$input_name, FILTER_DEFAULT, [
+	               'flags' => FILTER_REQUIRE_ARRAY
+	            ]);
+	            if(ak('select',$setting) && is_array($setting['select'])){
+	                if(is_null($input_data)){
+	                    $input_data = [];
+	                }
+	                foreach($setting['select'] as $in=>$val){
+	                    if(iar($in,$input_data)){
+	                        echo $val.'&nbsp';
+	                    }
+	                }
+	            }
+	            break;
+	    }
+	    echo '</td>';
+	    echo '</tr>';
+	}
+	?>
+	</table>
+</body>
+</html>
+<?php
+exit();
+```
+includeでの別画面表示をいかしたためにhelperで二重定義防止をしていますが
+ビューファイルはやはり1画面1つにしたほうがいいと思う。
+入力チェック後入力画面をそのまま入力値を初期値で表示させるなら入力チェックは自画面で行ったほうが良いと思います。
+リダイレクトするとか、リダイレクトが嫌ならJavaScriptでするとか、サーバーサイドでチェックしたいならAjaxでするとか、方法は幾らでもあります。

修正

2019/05/01 05:02

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -46,6 +46,10 @@
+入力値の出力にHTMLエスケープが施されてないですね。
 ---
 後ほど適当に組みなおしたコードを投稿するのであとは頑張って理解してみてください。