Как оптимизировать код?

Question

[ortsuev33]

<span>
                                                @if(count($LastArticle->comments))
                                                <a href="{{route('post',['articles'=>$LastArticle->id])}}#comments" title="">
                                                 @endif   
                                                        <i class="fa fa-comments-o"></i> 
                                                        {{count($LastArticle->comments)}} 
                                                        {{Lang::choice('ru.comments',count($LastArticle->comments))}}
                                                  @if(count($LastArticle->comments))
                                                </a>
                                                @endif
                                            </span>

У меня два раза используется одна и та же проверка , какие варианты оптимизации можно применить ?

На ум приходят поставить только условие с двумя вариациями кода ?Есть что нибудь еще?(
тоесть if и ifelse)
надеюсь нормально объяснил

Comments

[alex-1917]

у тебя ЧЕТЫРЕ раза вызывается подсчет)))

и кстати у вас точно этот код рабочий?))) если счетчик НОЛЬ, то Lang::choice что покажет?

Answer 1

[alex-1917]

Если уж и говорить про читаемость, то вот так читаемость будет на самом высшем уровне, к.м.к., ну и само собой всего ОДНА проверка, которая повзолит тебе сэкономить 16 наносекунд процессорноговремени и 24 байта оперативной памяти! ТС, не благодари:

<span>
@$temp = count($LastArticle->comments)
@if($temp)
   <a href="{{route('post',['articles'=>$LastArticle->id])}}#comments" title="">
       <i class="fa fa-comments-o"></i> 
       {{$temp}} 
       {{Lang::choice('ru.comments',$temp)}}
   </a>
@else
   <i class="fa fa-comments-o"></i> 
   {{$temp}} 
   {{Lang::choice('ru.comments',$temp)}}
@endif
 </span>

Кстати, код все равно остается неидеальным, так как скорее всего есть какие-то стили для тега a.
Ну и еще - a внутри span - поток мата от W3C гарантирован.

Comments

[ortsuev33]

Ну и еще - a внутри span - поток мата от W3C гарантирован, --- не совсем понял тут

[alex-1917]

ortsuev33, тут - https://validator.w3.org/

Answer 2

[Daria Motorina]

Экономия на одной из двух проверок - это экономия на спичках, оптимизации которой снижают читаемость вьюхи.
Лучше сохраните в переменную count($LastArticle->comments) и передавайте ее из контроллера во вью, чтобы она гарантированно вычислялась один раз, а не три.

Comments

[ortsuev33]

Там просто массив фореч все дела во view