Jak sugeruje rozszerzenie pliku, .phtmlplik umożliwia mieszanie kodu PHP z HTML. Jednakże fakt, że może nie powinien być postrzegany jako licencja do szaleją.
Dlaczego wciąż widzimy tak wiele plików .phtml wypełnionych dużą ilością PHP? A jakie jest dobre podejście do zmniejszenia ilości PHP w .phtmlpliku?
Rzeczywiście, im mniej PHP w .phtmltym lepiej, ponieważ:
Plik podstawowy Magento /app/design/frontend/base/default/template/catalog/product/price.phtml
jest bolesnym przykładem. Ten kod „prezentacji” HTML wyświetla cenę. Ma 471 linii! Głównie z powodu logiki PHP.
Aby twój .phtmlszczuplejszy i czystszy:
unikaj niepotrzebnych sekwencji <?php … ?>, łącz je w pojedyncze kawałki<?php … ?>
wpychaj jak najwięcej PHP do bloku, a nie .phtml
aby pomóc z powyższym, w bloku użyj assign(‘myvar’,
[expression])do utworzenia zmiennych $, do których można odwoływać się bez
$this->...w pliku .phtml, dzięki czemu możesz mieć naprawdę zwięzły<?php echo $myvar; ?>
chciałbym, aby Magento przyjęło Twig w przyszłości, aby uzyskać jeszcze czystszy wygląd
Zastosujmy powyższy fragment kodu z oryginalnego kodu z powyższego przykładu: /app/design/frontend/base/default/template/catalog/product/price.phtml
<?php if ($this->getDisplayMinimalPrice() && $_minimalPriceValue && $_minimalPriceValue < $_product->getFinalPrice()): ?>
<?php $_minimalPriceDisplayValue = $_minimalPrice; ?>
<?php if ($_weeeTaxAmount && $_weeeHelper->typeOfDisplay($_product, array(0, 1, 4))): ?>
<?php $_minimalPriceDisplayValue = $_minimalPrice+$_weeeTaxAmount; ?>
<?php endif; ?>
….
<?php echo $_coreHelper->currencyByStore($_minimalPriceDisplayValue, $_storeId, true, false) ?>Pierwszy krok: usuń powtórzenie, <?php … ?>aby dojść do czegoś takiego:
if ($this->getDisplayMinimalPrice() && $_minimalPriceValue && $_minimalPriceValue < $_product->getFinalPrice()) {
$_minimalPriceDisplayValue = $_minimalPrice;
if ($_weeeTaxAmount && $_weeeHelper->typeOfDisplay($_product, array(0, 1, 4))) {
$_minimalPriceDisplayValue = $_minimalPrice+$_weeeTaxAmount;
}
…
echo $_coreHelper->currencyByStore($_minimalPriceDisplayValue, $_storeId, true, false)
?>Powyżej umieszcza wszystkie PHP w jednej kropli kodu.
2 + 3. Ewoluując w coś lepszego, przenieś ten kod do jego bloku:
protected function _prepareLayout() {
$this->assign(‘minPrice’, $this->calculateMinPrice(…));
}
protected function calculateMinPrice(…) {
if ($this->getDisplayMinimalPrice() && $_minimalPriceValue && $_minimalPriceValue < $_product->getFinalPrice()) {
// etc...
}
}Zwróć uwagę na użycie do _prepareLayout()tego assign()funkcji i.
Teraz tę zawiłą sekcję pliku .phtml można sprowadzić do tej prostej linii:
<?php echo $minPrice; ?>Myślę, że wszyscy możemy z tym żyć!
Niezły opis, @fris, zgadzam się w prawie wszystkich punktach.
Główną zaletą jest przeniesienie całej logiki do klasy bloków i uczynienie szablonu możliwie „głupim”.
Właściwie wolę wywołania metod w szablonie niż zmienne, które zostały „przypisane”, ponieważ nie chcę stracić funkcji uzupełniania kodu IDE i funkcji nawigacyjnych. „przypisywanie” wygląda bardziej zwięźle w szablonie, ale jest o wiele za dużo magii na mój gust, co czyni go jeszcze gorszym niż pobieracze i ustawiacze magii.