سوال در مورد پیاده سازی global middleware

سلام و احترام

چیزی که الان به ذهنم میرسه برای داشتن global middleware ها اینکه شما یه فایل داشته باشید برای مثال به اسم GlobalMiddleware.php که محتوا داخل کلا یه آرایه بر میگردونه

<?php
return [
	'global' => [
		MiddlewareOne::class,
		MiddlewareTwo::class,
	]
]

به این شکل داشته باشید و include کنید و بعد یه foreach بزنید توی همون کلاس route و همشو اجرا کنید

middleware هاتون رو هم پوشه بندی کنید تا زیاد شلوغ نشه و بشه راخت middleware پیداشون کرد

سلام امیر جان 

مرسی بابت پاسخگوییت،من خیلی ابهامات در مورد بهترین نحوه پیاده سازی global middleware ها دارم..

ممنون میشم اگر امکانش هست گزینه به گزینه پاسخ بدی.

۱- الان این روشی که گفتید بهترین روش برای تعریف middleware ها است؟

۲- روشی که بالا من گفتم بهتره یا روش شما یا روش استاد آوند در ویدیو تمرین؟ (حس میکنم روش خودم این مشکل رو داره که اگر یک زمان نخوایم یکی از global middleware ها فعال باشه باید فایل اون رو حذف کنیم یا پوشه رو عوض کنیم تا در دستور glob لیست نشه! این رو میشه یک ایراد دونست؟)

۳- برای اجرای middleware ها به روش شما باید در router.php و در constructor به این شکل مقدار دهی کنیم:

$this->global_middleware = require BASEPATH . '/App/Middleware/GlobalMiddleware/GlobalMiddleware.php';

۴- توی روش خودتون به جای این کار بیایم نام کامل کلاس هامون رو در یک متغیر global در فایل init.php درج کنیم بهتر نیست؟ 

5- یا به جای این دو روش یه property توی constructor تعریف کنیم و همونجا نام کلاس ها رو مقدار دهی کنیم!

6- وقتی از MiddlewareOne::class استفاده میکنیم باز باید بالاش namespase رو use کنیم که میشه همون نام کامل کلاس، خب بهتر نیست کلا از اول همون نام کامل رو بنویسیم؟

سلام و احترام

۱ - من روش استاد رو ندیدم و اگه استاد روشی گفتند حتما از اون استفاده کنید، روشی که من گفتم شبیه به لاراول بود که تمامی middleware ها رو توی یه آرایه بزاریم

۲ - روش خودتون همون glob میتونست مشکل ساز باشه بعدا که زیاد جالب نیست استفاده ازش

۳- بله میتونید به همون شکل محتوای فایل رو داشته باشید

۴ و ۵ - ۶ - کلاس های middleware ها رو با namespace کاملشون داخل فایل قرار بدید به شکل زیر:

$middleware = [
        // \App\Http\Middleware\TrustHosts::class,
        \App\Http\Middleware\TrustProxies::class,
        \Fruitcake\Cors\HandleCors::class,
        \App\Http\Middleware\PreventRequestsDuringMaintenance::class,
        \Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
        \App\Http\Middleware\TrimStrings::class,
        \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
    ];

پیشنهاد بعدی اینکه این متغیر middleware رو ببرید داخل یک کلاس و به عنوان پراپرتی داشته باشید تا دردسر include کردن به صورت آدرسی هم نداشته باشید و راحت از کلاس یه new بگیرید و به property درسترسی داشته 

.باشید و همه کلاس های لیست  شده داخلشو استفاده کنید

مثال:

<?php

namespace App\Http;

use Illuminate\Foundation\Http\Kernel as HttpKernel;
use App\Http\Middleware\CheckUsrHadTheSecondRegistrationStep;

class Middlewares 
{
    /**
     * The application's global HTTP middleware stack.
     *
     * These middleware are run during every request to your application.
     *
     * @var array
     */
    protected $globalMiddleware = [
        // \App\Http\Middleware\TrustHosts::class,
        \App\Http\Middleware\TrustProxies::class,
        \Fruitcake\Cors\HandleCors::class,
        \App\Http\Middleware\PreventRequestsDuringMaintenance::class,
        \Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
        \App\Http\Middleware\TrimStrings::class,
        \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
    ];


	public function getGlobalMiddlewares()

	{

		return $this->globalMiddleware ;

	}
}

سلام

ممنون بابت پاسخگوییتون

۳ تا سوال

۱- الان که نام کامل رو آوردیم چرا هنوز از ::class استفاده کردیم؟

۲- اگر توی constructor فایل router.php بخوایم از این کلاس new کنیم مشکلی ندارد و اصول solid نقض نمیشه؟

۳- بهتر نیست این کلاس و پراپرتی هاش را به صورت abstract تعریف می‌کردیم؟

سلام و احترام

۱ - با گذاشتن ::class ما به آدرس کامل کلاس دسترسی داریم

۲ - میتونید یک آبجکت ساخته شده ازش رو پاس بدید به کلاس route و دیگه ازش شی داخل construt نسازید 

۳ - فکر نکنم چون قرار نیست جایی ازش ارث برده بشه قراره فقط این یه وظیفه رو داشته باشه که همین global middleware هارو بر گردونه