您有很多PHP标记垃圾邮件,例如:
?><?php
这很糟糕,令人困惑,使您的代码难以阅读,并且浪费您的时间来键入。阅读代码的困难导致了您的问题,包括缺少缩进。这些都很重要,任何好的编辑器都会毫不费力地为您完成这些任务。
如果我们删除PHP标记垃圾邮件并正确缩进,我们会得到以下结果:
<?php
{
if ( is_front_page() ) {
get_template_part(\'home-page-content\');
} else if ( is_page(\'archives\') ) {
get_template_part(\'archives\');
} else
get_template_part(\'secondary-page-content\');
}
?>
首先,整件事周围都有一套大括号,没有任何作用(除了可能混淆)。移除这些,为我们提供:
<?php
if ( is_front_page() ) {
get_template_part(\'home-page-content\');
} else if ( is_page(\'archives\') ) {
get_template_part(\'archives\');
} else
get_template_part(\'secondary-page-content\');
?>
最后,我们有编码标准,所以让我们在最后的else语句周围加上括号:
<?php
if ( is_front_page() ) {
get_template_part(\'home-page-content\');
} else if ( is_page(\'archives\') ) {
get_template_part(\'archives\');
} else {
get_template_part(\'secondary-page-content\');
}
?>
我们可以更进一步,这些都是
get_template_part
, 如果我们只调用该函数一次呢?
<?php
$template = \'secondary-page-content\';
if ( is_front_page() ) {
$template = \'home-page-content\';
} else if ( is_page(\'archives\') ) {
$template = \'archives\';
}
get_template_part( $template );
?>
然而,所有这些都意味着您已经用太多的责任重载了一个模板。我建议您查看模板层次结构,这样我们就可以使用
frontpage.php
模板删除一半if语句。