简化blade和laravel中的@foreach/if

Question

我对laravel（特别是L5)相当陌生，我正在制作我自己版本的todo应用程序，而不是按照那里的某个教程去做。到目前为止，我已经学到了很多东西，但是我现在在刀片模板中展示的这段代码让我觉得它们可能是一种更简单的方法。

我的 fn是

public function index()
{
    $todos = Todo::get();
    return view('todos', compact('todos'));
}

扩展了一个 模型，使数据处理变得非常容易！

我的路线是:

Route::bind('todos', function($slug)
{
    return App\Todo::whereSlug($slug)->first();
});

所以我的页面只是显示了一个无序的“TODOS”列表。我想要两份单独的名单。一个用于完成的待办事项，一个用于未完成的待办事项。我的刀片模板到目前为止看起来是这样的，看起来有点凌乱。同时，我也循环了两次结果，这是我认为我可以改进的地方。

<h3>Incomplete</h3>
<ul>
    @foreach ($todos as $todo)
        @if ($todo->completed == 'No')
            <li>
                <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
            </li>
        @endif
    @endforeach
</ul>

<h3>Complete</h3>
<ul>
    @foreach ($todos as $todo)
        @if ($todo->completed == 'Yes')
            <li>
                <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
            </li>
        @endif
    @endforeach
</ul>

有什么建议可以简化刀片模板吗？

Answer 1

只是有点简化但是。。。

您可以在控制器中尝试:

public function index()
{
    $completed = Todo::where('completed','Yes')->get();
    $incompleted = Todo::where('completed','No')->get();
    return view('todos', compact('completed', 'incompleted'));
}

在您的模板中:

<h3>Incomplete</h3>
<ul>
    @foreach ($incompleted as $todo)
            <li>
                <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
            </li>
    @endforeach
</ul>

<h3>Complete</h3>
<ul>
    @foreach ($completed as $todo)
            <li>
                <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
            </li>
    @endforeach
</ul>

另一种使用子模板的方法如下所示:

    //_list_todos.blade.php

    @foreach ($todos as $todo)
            <li>
                <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
            </li>
    @endforeach

您的主模板如下所示:

<h3>Incomplete</h3>
<ul>
     @include('_list_todos',['todos'=>$incompleted] )
</ul>

<h3>Complete</h3>
<ul>
     @include('_list_todos',['todos'=>$completed] )
</ul>

使用类似于最后一个子模板的优点是可以重用代码，并简化主模板。

Answer 2

把你的代码弄干。您可以通过将实际项目标记向上移动到部分模板来简化它，因为它在完整列表和不完整列表中都是重复的:

<h3>Incomplete</h3>
<ul>
    @foreach ($todos as $todo)
        @if ($todo->completed == 'No')
            @include('partials.items.todo')
        @endif
    @endforeach
</ul>

<h3>Complete</h3>
<ul>
    @foreach ($todos as $todo)
        @if ($todo->completed == 'Yes')
            @include('partials.items.todo')
        @endif
    @endforeach
</ul>

和 如下所示:

<li>
    <a href="{{ route('todos.show', [$todo->slug]) }}">{{ $todo->title }}</a>
</li>

我也会重新考虑你的循环。您可以在控制器中拆分它们，而不是在同一个列表上循环两次:

public function index()
{
    $todos = Todo::where('user_id', '=', Auth::id())->get();

    $complete = $todos->filter(function ($item) {
        return $item->completed = 'Yes';
    });

    $incomplete = $todos->filter(function ($item) {
        return $item->completed = 'No';
    });

    return view('todos', compact('complete', 'incomplete'));
}

查看您的 模型，Id还使数据库中的 列成为布尔字段，而不是包含yes或no字符串的列。然后可以将该列值强制转换为正确的布尔值（因为MySQL没有本机布尔字段类型）:

class Todo extends Model
{
    protected $casts = [
        'completed' => 'boolean',
    ];

    public function isComplete()
    {
        return $this->completed;
    }
}

然后重新考虑控制器操作以使用以下方式:

public function index()
{
    $todos = Todo::where('user_id', '=', Auth::id())->get();

    $complete = $todos->filter(function ($item) {
        return $item->isComplete() === true;
    });

    $incomplete = $todos->filter(function ($item) {
        return $item->isComplete() === false;
    });

    return view('todos', compact('complete', 'incomplete'));
}

您甚至可以将这些集合筛选器移动到自定义的 类:

use Illuminate\Database\Eloquent\Collection as EloquentCollection;

class TodoCollection extends EloquentCollection
{
    public function complete()
    {
        return $this->filter(function ($item) {
            return $item->isComplete() === true;
        });
    }

    public function incomplete()
    {
        return $this->filter(function ($item) {
            return $item->isComplete() === false;
        });
    }
}

对于冗长的回复，很抱歉，但是应该给您提供一些关于如何重新考虑您的代码的食物。