改进刀片模板的更好方法


A better way to improve blade template?

客户可以选择两种类型的套餐:电话套餐或宽带和电话套餐。

当用户准备下订单时,它将显示包含包裹名称信息和成本的摘要视图。某些摘要信息不需要显示是否选择电话套餐或宽带和电话套餐。

有没有更好的方法来提高可读性和可维护性?例如

订单侧边栏.刀片.php

注意:这是代码块的小示例。在实际应用程序中,它非常大,有很多if语句broadbandphonephone

@if ($summary['service'] == "phone" ||  $summary['service'] == "broadbandphone")
    <div class="x5">
        <h4>Phone Line x <span class="summary-lines">{{$summary['lines']}}</span></h4>

        <ul class="clearfix">
            @if ($summary['service'] == "phone")
                <li>
                    <p class="x5-details">
                        @if ($summary['line_type'] == "newline")
                            New Line(s)
                        @endif
                        @if ($summary['line_type'] == "switch")
                            Switch line(s)
                        @endif
                    </p>
                </li>
            @endif
            @if ($summary['service'] == "phone" && $summary['lines'] > 1)
                <li>
                    <p class="x5-details">{{$summary['linesWithMulitpleNumbers']}}</p>
                </li>
            @endif
            <li>
                <p class="x5-details">{{$summary['package']->name}}</p>
                <p class="x5-price">£{{$summary['monthlyLinesCost']}}</p>
            </li>
        </ul>
    </div>
@endif

或者我应该为摘要视图做两个单独的文件,如order-sidebar-phone.blade.phporder-sidebar-broadbandphone.blade.php

所以它会是这样的:

   @if ($summary['service'] == "phone")
        @include('sections.order-sidebar-phone')
   @end
   @if ($summary['service'] == "broadbandphone")
        @include('sections.order-sidebar-broadbandphone')
   @end

你的代码对我来说看起来不错。但是,这里有一些提示可能有助于减少 if 或使您的代码更具可读性。

速记三元如果

有时,速记看起来更清晰,更具可读性:

@if($summary['service'] == "phone")
    Foo
@else
    Bar
@endif

可以写成:

{{ ($summary['service'] == "phone" ? "Foo" : "Bar") }}

它对于根据条件而变化的小段文本特别有用。

凹痕

我甚至不需要这么说。正确缩进您的代码(不是说您没有在您的问题中...你的 if 语句不会那么令人困惑;)

部分

虽然可以选择按照您的建议将其完全拆分,但这可能会导致大量重复代码......

但是你也可以拆分你的文件,这样它们就不会那么大,也不会包含那么多if语句(即使总数仍然相同,它的结构更好,更清晰)

例如,代替:

{{-- ... --}
</div> 
    
@if ($summary['newSetup'] == false)
    <div class="installation">
        <h4>Phone Line x <span class="summary-lines">{{$summary['lines']}}</span></h4>
        <ul class="clearfix">
            <li>
                <p class="installation-details">Installation</p>
                <p class="installation-price">£{{$summary['installationCharge']}}</p>
            </li>
        </ul>
    </div>
@endif
<div class="off-costs bg-gray">
     <div class="off-costs-header clearfix">
{{-- ... --}}

installation部分放入其自己的文件中并包含它:

{{-- ... --}
</div> 
    
@if ($summary['newSetup'] == false)
    @include('installation')
@endif
<div class="off-costs bg-gray">
     <div class="off-costs-header clearfix">
{{-- ... --}}