Correcting teacher:PHPz
Correction status:qualified
Teacher's comments:
变量有作用域,外部与函数内互不可见,除非成为全局变量,两种方式,可以通过global 声明为全局变量,或变为$GLOBALS超全局变量的参数
$one = 200;
$two = 300;
const NATION ='中国';
function test()
{
// 1.
// global $one, $two;
// 2. $GLOBALS超全局变量
echo NATION;
echo "运算的结果是".($GLOBALS['one'] + $GLOBALS['two']);
}
include __DIR__.DIRECTORY_SEPARATOR.'config'.DIRECTORY_SEPARATOR.'common.php';
include TEMP_PATH_PUBLIC . DS . 'header.php';
$news = require DATA_PATH. DS . 'news.php';
$items = require DATA_PATH . DS . 'item.php';
?>
<!-- 主体 -->
<main>
<!-- 新闻列表 -->
<div class="news">
<h3>新闻列表</h3>
<div class="list">
<?php for($i=0; $i<10; $i++): ?>
<a href="<?= $news[$i]['url'] ?>">
<?=
strlen($news[$i]['title'])>48? mb_substr($news[$i]['title'], 0,16).'...':$news[$i]['title'];
?>
</a>
<?php endfor ?>
</div>
</div>
<!-- 产品列表 -->
<div class="items">
<h3>产品列表</h3>
<div class="list">
<?php foreach($items as $k=>$v): extract($v)?>
<div class="item" id="item">
<img src="<?=$img?>" alt="<?= $title?>" />
<a href="<?=$url?>"><?= strlen($title)>45? mb_substr($title, 0,15).'...':$title; ?></a>
</div>
<?php endforeach ?>
</div>
</div>
</main>
<!-- 页脚 -->
<?php
include TEMP_PATH_PUBLIC . DS . 'footer.php';
?>
<!-- 新闻列表 -->
<div class="news">
<h3>新闻列表</h3>
<div class="list">
<?php foreach($news as $k=>$v): extract($v)?>
<a href="<?= $news[$i]['url'] ?>"><?=
strlen($title)>48? mb_substr($title, 0,16).'...':$title;
?></a>
<?php endforeach ?>
</div>
</div>
优化包括以下内容:
1、定义常量,简化代码;
2、超过16个汉字的标题取前16个字并加点;
3、首页采用for循环取前10条记录,新闻页用foreach循环显示左右记录。
我觉得DS常量可以加到路径里面;网页中首页词条始终是红的,其他词条激活了也不能变色,这些都是需要后续完善的。