客户可以选择两种类型的套餐:电话套餐或宽带和电话套餐。
当用户准备下订单时,它将显示包含包裹名称信息和成本的摘要视图。某些摘要信息不需要显示是否选择电话套餐或宽带和电话套餐。
有没有更好的方法来提高可读性和可维护性?例如
订单侧边栏.刀片.php
注意:这是代码块的小示例。在实际应用程序中,它非常大,有很多if
语句broadbandphone
或phone
@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.php
和order-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">
{{-- ... --}}