我有一些PHP代码,除了一些小的变量命名差异外,几乎都是重复的。如何将其转换为可重用的函数,我可以通过参数传递?
这是我使用两次的代码。第二个是相同的,除了“会员”的所有引用都改为“社交”。
<?php
$affiliate = wp_list_bookmarks( array( 'categorize' => 0, 'category' => '7', 'title_li' => '', 'orderby' => 'rating', 'show_images' => 0, 'echo' => 0 ) );
preg_match_all( '/<li>.*?<\/li>/', $affiliate, $affiliate_matches );
foreach ( $affiliate_matches[0] as $affiliate_match ) {
preg_match( '/title=".*?"/', $affiliate_match, $affiliate_title );
echo str_replace(
$affiliate_title[0],
$affiliate_title[0] . ' ' . strtolower( str_replace( array( 'title="', ' ' ), array( 'class="', '-' ), $affiliate_title[0] ) ),
$affiliate_match
) . "\n";
}
?>
另一个是:
<?php
$social = wp_list_bookmarks( array( 'categorize' => 0, 'category' => '2', 'title_li' => '', 'orderby' => 'rating', 'show_images' => 0, 'echo' => 0 ) );
preg_match_all( '/<li>.*?<\/li>/', $social, $social_matches );
foreach ( $social_matches[0] as $social_match ) {
preg_match( '/title=".*?"/', $social_match, $social_title );
echo str_replace(
$social_title[0],
$social_title[0] . ' ' . strtolower( str_replace( array( 'title="', ' ' ), array( 'class="', '-' ), $social_title[0] ) ),
$social_match
) . "\n";
}
?>
我在想也许我可以调用这个函数
<?php links( array( 'affiliate', 7 ) ); ?>
或
<?php links( array( 'social', 2 ) ); ?>
将它们组合成可重复使用的函数可以节省处理时间/资源还是无关紧要?
答案 0 :(得分:0)
它不会节省任何计算机时间,它节省的是你的时间,不需要维护代码两次。 (但请注意,将其转换为函数并不会让您花费更多精力而不仅仅是两次。)
此外,我认为没有理由将“社交”一词传递给该功能 - 它实际上从未在任何地方使用过。
答案 1 :(得分:0)
唯一真正改变的是类别ID,所以你只需要将它传递给函数。
function links($categoryId) {
$affiliate = wp_list_bookmarks( array( 'categorize' => 0, 'category' => $categoryId, 'title_li' => '', 'orderby' => 'rating', 'show_images' => 0, 'echo' => 0 ) );
preg_match_all( '/<li>.*?<\/li>/', $affiliate, $affiliate_matches );
foreach ( $affiliate_matches[0] as $affiliate_match ) {
preg_match( '/title=".*?"/', $affiliate_match, $affiliate_title );
echo str_replace(
$affiliate_title[0],
$affiliate_title[0] . ' ' . strtolower( str_replace( array( 'title="', ' ' ), array( 'class="', '-' ), $affiliate_title[0] ) ),
$affiliate_match
) . "\n";
}
}