常連

注目の的

社会貢献

質問力

自己紹介

私は現在、Laravel5.1を使用しています。

LaravelはほとんどAPIとして利用しています。  
現在、FormRequestクラスをControllerのメソッド単位でクラスを作成しているのですが、バリデーションルールが被るような場合もあること、そしてAPIの数が多い等あり、出来るだけ少ないファイル数でバリデーションをしたいと考えています。

このような場合、どのようなアプローチを行うと良いのでしょうか？
(Controllerにそのまま持たせる？、ルールだけ別のクラスにまとめる？・・・？)

よろしくお願いいたします。

Laravel5のFormRequestクラスで複数のRequestを処理したい

###前提・実現したいこと
フォームから受け取った配列のバリデーションをしたい。

###仕様
・2つのセレクトボックスをもつフォームを@forループで複数行作成。
   渡ってくるリクエストパラメータは下記のようなイメージ
	==============================
		array(2) {
		  ["name_a"]=>
		  array(5) {
		    [0] => string(0) ""
		    [1] => string(1) "1"
		    [2] => string(1) "2"
		    [3] => string(1) "3"
		    [4] => string(0) ""
		  }
		  ["name_b"]=>
		  array(5) {
		    [0] => string(0) ""
		    [1] => string(1) "1"
		    [2] => string(0) ""
		    [3] => string(0) ""
		    [4] => string(0) ""
		  }
	==============================

・カスタムバリデーション内で、選択されたセレクトボックスが同一かどうか確認。
	→同一の場合バリデーションエラーとしたい

・セレクトボックスで選択されたものに対し全てバリデーションチェックし、結果をviewに返す。

###ソースコード
**リクエストされるView**
```php
// test.blade.php
// ～中略～
	@if ($errors->any())
	    <div class="alert alert-danger">
	        <ul>
	            @foreach($errors->all() as $error)
	                <li>{{ $error }}</li>
	            @endforeach
	        </ul>
	    </div>
	@endif
	<br>
   {!! Form::open() !!}
        <div class="form-inline">
            @for ($i = 0; $i < 5; $i++)
                <?php $n = $i+1 ?>
				{{ "#$n ： " }}
                <div class="form-group">
                    {!! Form::select("name_a[$i]", $namelist, null, ['class' => 'form-control', 'placeholder' => "名称A"]) !!}
                </div>
                &nbsp;―&nbsp;
                <div class="form-group>
                    {!! Form::select("name_b[$i]", $namelist, null, ['class' => 'form-control', 'placeholder' => "名称B"]) !!}
                </div>
                <br>
                <br>
            @endfor
        </div>
        <div class="form-group">
            {!! Form::submit(' 登 録 ', ['class' => 'btn btn-primary']) !!}
        </div>
    {!! Form::close() !!}
// ～中略～
```

**カスタムバリデーション**
```php
<?php
namespace app\Services;

class CustomValidator extends \Illuminate\Validation\Validator
{
    public function validateBoo()
    {

        if($this->data['name_a'] == $this->data['name_b']){
            return false;
        }
        return true;

    }

}
```

**コントローラー**
```php
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests;
use App\Http\Controllers\Controller;

class GameResultController extends Controller
{
// ～中略～

    public function store(Request $request)
    {
        $formlist = $request->all();
    	
        foreach ($request->get('name_a') as $key => $value) {
            $i= $key+1;

            if (("" !== $formlist ["name_a"][$key]) && ("" !== is_null($formlist ["name_b"][$key]))
            {

                $rules['name_a.'.$key] = 'boo';
                $messages['name_a.'.$key.'.boo'] = '#'.$i.' ：同一の名称が選択されています。';

            }
        }
        
        /*** バリデーション実行 ***/
        $this->validate($request,$rules,$messages);
        
        // ～中略～
        
    }
}

```

###発生している問題・エラーメッセージ
連続するバリデーション内でエラーと判定されなかったものが存在する場合に、
エラーとなるべき項目が無視されて後続が処理される。

・以下のように連続してバリデーションエラーとなる場合は、
 正常にviewにエラーメッセージが表示されます。
  (配列0～3までがエラーとなり、4つのエラーメッセージが表示されます)
```
array(2) {
  ["name_a"]=>
  array(5) {
    [0] => string(1) "1"
    [1] => string(1) "2"
    [2] => string(1) "3"
    [3] => string(1) "4"
    [4] => string(0) ""
  }
  ["name_b"]=>
  array(5) {
    [0] => string(1) "1"
    [1] => string(1) "2"
    [2] => string(1) "3"
    [3] => string(1) "4"
    [4] => string(0) ""
  }
```
・以下のような場合は、途中のエラーとなるべき箇所が無視されて後続を処理してしまいます。
```
array(2) {
  ["name_a"]=>
  array(5) {
    [0] => string(1) "1"
    [1] => string(1) "2"
    [2] => string(1) "3"
    [3] => string(1) "4"
    [4] => string(0) ""
  }
  ["name_b"]=>
  array(5) {
    [0] => string(1) "1"
    [1] => string(1) "2"
    [2] => string(1) "2"
    [3] => string(1) "4"
    [4] => string(0) ""
  }
```
そもそも、やり方自体が違う等あるかもしれません。
どなたか、お力添え頂ければ幸いです。

###補足情報(言語/FW/ツール等のバージョンなど)
・XAMPP
・Laravel 5.2.7
・PHP 7.0.1

Laravel5 配列を利用した連続バリデーションに関して

なんらかの処理をするためのファイルや、includeして使うファイルなど、ユーザーから直接URLを叩くなどをしてアクセスされることを期待しないファイルはどうしたらよいでしょうか。

推測されにくいファイル名にする方法を考えましたが、同じ開発グループ内でわかりづらくなってしまうのでボツとしました。


追記
---
レンタルサーバーを利用しています。
とりあえずサーバーの情報を載せておきますがこんなかんじでいいでしょうか… 
![イメージ説明](631c5e67134082773a477e0fa518e655.png)

ユーザーに直接アクセスされないようにしたい

エンジニアならこれは読んでおけ みたいな本を教えてください。
ほかにも読んですごく良かった本などでも知りたいです。

アルゴリズムの本や正規表現、TCP/IPなどなど
お願いします

読んでおいたほうがいい本

webサービスのユーザー登録画面を作成中ですが、エラーが出てしまいます。どのように修正すれば良いかアドバイスを頂けませんでしょうか？

以下登録画面のHTML&PHPです。
```index.php
コード
```
<!DOCTYPE html>
<html>
<head>
  <meta charset="UTF-8" />
  <title>新規会員登録</title>
</head>
<body>
  <p>必要事項をご記入ください</p>
  <form action="" method="post" enctype="multipart/form-data">
 
<dl>
  <dt>ユーザー名<font color="red"> 必須</font></dt>
  <dd>
    <input type="text" name="name" size="35" maxlength="255" value="<?php echo htmlspecialchars($_POST['name'], ENT_QUOTES, 'UTF-8'); ?>">
    <?php if($Error['name'] == 'blank'): ?>
    <p><font color="red">* ユーザー名を入力してください</font></p>
    <?php endif; ?>
  </dd>
  <dt>メールアドレス<font color="red"> 必須</font></dt>
  <dd>
    <input type="text" name="mail" size="35" maxlength="255" 
        value="<?php echo htmlspecialchars($_POST['mail'], ENT_QUOTES, 'UTF-8'); ?>">
    <?php if($error['mail'] == 'blank'): ?>
    <p><font color="red">* メールアドレスを入力してください</font></p>
    <?php endif; ?>
    <?php if($error['mail'] == 'duplicate'): ?>
    <p><font color="red">* 指定されたメールアドレスは既に登録されています</font></p><?php endif; ?>
  </dd>
  <dt>パスワード<font color="red"> 必須</font></dt>
  <dd>
    <input type="password" name="pass" size="10" maxlength="20" 
        value="<?php echo htmlspecialchars($_POST['pass'], ENT_QUOTES, 'UTF-8'); ?>">
    <?php if($error['pass'] == 'blank'): ?>
    <p><font color="red">* パスワードを入力してください</font></p>
    <?php endif; ?>
    <?php if($error['pass'] == 'length'): ?>
    <p><font color="red">* パスワードは４文字以上で入力してください</font></p>
    <?php endif; ?>
  </dd>
</dl>
  <div><input type="submit" value="入力内容を確認"></div>
  </form>
  
  <?php
session_start();

if(!empty($_POST)){
  //エラー項目の確認
  if($_POST['name'] == ''){
    $error['name'] = 'blank';
  }
  if($_POST['mail'] == ''){
    $error['mail'] = 'blank';
  }
  if(strlen($_POST['pass']) < 4){
    $error['pass'] = 'length';
  }
  if($_POST['pass'] == ''){
    $error['pass'] = 'blank';
  }
 
  if(empty($error)){
    $_SESSION['ドットインストール'] = $_POST;
    header('Location: check.php');
    exit();
  }
}
// 書き直し
if ($_REQUEST['action'] == 'rewrite'){
  $_POST = $_SESSION['ドットインストール'];
  $error['rewrite'] = true;
}
?>

---------------------------------------------------------------------------
また、下記がエラー画面です。
（このファイルはindex.phpというファイル名です。）

![![イメージ説明](7e301dab104527017e36201e93270d1e.png)](d3379a43d423ff4f2d2c1a3bc3d1f22a.png)