Как правильно реализовать кэширование модели для rules?

Question

[kiranananda]

Здравствуйте!

Что то всю голову сломал, как корректно сделать.

Есть у меня куча различных rules валидации и часть из них проверяет данные в одной и той же модели. И если в каждом правиле я делаю запрос модели из базы, да потом еще и делаю запрос этой модели из контроллера, то получаю, в данный момент 3 одинаковых запроса в бд. Что не есть хорошо. Как бы правильно это реализовать? Или это нормально так делать?

Comments

[Евгений Вольф]

Есть у меня куча различных rules валидации и часть из них проверяет данные в одной и той же модели.

Модель - это некоторая единица информации которая хранится в базе... как и зачем у Вас в базу попадают данные, которые потом валидировать нужно? Думается мне, правильнее было бы, для начала данные валидировать, а потом уже записывать в базу...

[kiranananda]

Опишу чуть подробнее. В модельке есть номер телефона, я его не валидирую, но когда идет запрос с номером телефона, я проверяю есть ли такой. Так же в модели есть параметры отправки смс, когда была, сколько попыток и сам код который надо подтвердить. Соответсвенно для этого тоже правила валидации. Ну и наконец контроллер...

[kiranananda]

Сейчас думаю создать некий промежуточный класс и в конструкторе у него сделать первую выборку по номеру телефона, далее уже определю его в ServiceProvider как singleton и далее буду обращаться к нему. Загвоздка сейчас в том, что этот класс посути обертка модели и мне бы хотелось как то проксировать все свойства модели, например обращаются к моему классу, а я переадресую это на $this->model. Пока не могу найти как это делается. Наследовать модель не хочется, хотя может и это и правильно не знаю точно....

[JhaoDa]

Почему это нехорошо? Запросы в БД платные?

[kiranananda]

JhaoDa, так мало того что бесплатные, ещё и сервер по нагрузке свободный :)

Answer 1

[kiranananda]

Реализовал так, поправьте если можно проще или правильнее...
Сервис провайдер

class DriverLoginServiceProvider extends ServiceProvider
{
	// protected $defer = true;
	
    public function register()
    {
    	$this->app->singleton(DriverLogin::class, function ($app) {
	    	return DriverLogin::where('phone', Request::input('phone', 'none'))->first();
		});
    }

    public function provides()
    {
        return [DriverLogin::class];
    }    
}

Контроллер

public function loginCheckPhone(CheckDriver $request, DriverLogin $driver)
    {
    	// Проверяем можно ли делать повторную отправку.
    	if ( ($checkSend = Sms::checkSend($driver->verify_code)) !== true ) {
    		return response()->json( $checkSend, 403);
    	}

    	$driver->verify_code = Sms::send($driver->phone, 'Ваш код авторизации в личном кабинете: %code');

    	$driver->save();
    }

Самое правило валидации

public function passes($attribute, $value)
    {	
       return (app()->get(DriverLogin::class)) ? true : false;
    }

Наследовение оригинальной модели. У меня 2 сервис провайдера, один при входе, другой уже когда человек авторизовался. Вот для этого наследование.

<?php

namespace App\Models;

class DriverLogin extends \Backend\Driver\Models\Driver
{

}

?>

В итоге эта схема чуть сложна для данной ситуации, в реале сделал просто так вот

<?php

namespace App\Services;

use Backend\Driver\Models\Driver;
use Request;

class DriverLogin
{
	private static $model = null;

	public static function get() 
	{
		if (DriverLogin::$model === null ) {
			$phone = Request::input('phone', '');
			DriverLogin::$model = (iconv_strlen($phone) != 12) 
				? false 
				: Driver::where('phone', $phone)->first();
		}
    	return DriverLogin::$model;
  	}
}

?>

И из нужных мест получаю резултат запроса...

Comments

[kiranananda]

vism, ну по сути он и получился. Заодно в контроллеры сразу готовую модель передаю.
А так пример пришлите как считаете правильным...

[kiranananda]

Сделал чуть проще, написал класс к которому м обращаюсь...

Answer 2

[NubasLol]

public function rules()
    {
        $model = Model::firsy();

        return [
            'phone' => new Rule($model),
            'phone1' => new Rule1($model),
            'phone2' => new Rule2($model),
        ];
    }

Comments

[kiranananda]

Спасибо!
Это уже гораздо лучше, 2 запроса не так сильно напрягают мой перфекционизм :).