1

我最近在我的网站上注意到一个新闻页面正在重复来自数据库中所有“五月”的新闻文章（您会在这里看到：www.darlingtontowntwinning.co.uk/news_&_events）

我知道编码很混乱并且可能已经过时，但是，该网站是为我们构建的，而我目前还没有技能（还 - 我正在学习！）来更改整个网站。

有没有办法阻止这种情况发生 - 因为我相信我已经限制了要显示的每条记录之一：

<div id="right" class="news">
<h3>Archive</h3>
<? $news=$session->getNews("","","",1);?>
<? while($article=mysql_fetch_array($news)){?>
<? 
$date = $article['thedate'];
$year = date('Y', $date);
$month = date('F', $date);
?>
<h4><?=$month." - ".$year;?></h4>
<nav class="small">
<? $innernews=$session->getNews("",$month,$year);?>
<? while($innerarticle=mysql_fetch_array($innernews)){?>
<a href="/news/<?=$innerarticle['ftitle']?>" <? if($title==$innerarticle['ftitle']){?> class="active"<? }?>><?=$innerarticle['title']?></a>
<? }?>
</nav>
<? }?>
</div>

获取新闻功能是：

function getNews($title,$month,$year,$group){
global $database;
return $database->getNews($title,$month,$year,$group);}

$database->getNews 函数是：

//get news
function getNews($title,$month,$year,$group){
   if($title){
       $q=$this->query("SELECT * FROM ".TBL_NEWS." WHERE ftitle = '$title'" );
       return mysql_fetch_array($q);
   }else if($year && $month){
     $q=mysql_query("SELECT * FROM ".TBL_NEWS." WHERE (FROM_UNIXTIME(thedate, '%Y') = '$year') AND (FROM_UNIXTIME(thedate, '%M') = '$month') ORDER BY thedate DESC");
     return $q;
     }else if($group){
         $q=$this->query("SELECT * FROM ".TBL_NEWS." GROUP BY (FROM_UNIXTIME(thedate, '%Y')),(FROM_UNIXTIME(thedate, '%M')) ORDER BY thedate DESC" );
       return $q;
     }else{
       $q=$this->query("SELECT * FROM ".TBL_NEWS." ORDER BY thedate DESC" );
       return $q;
   }

}

4

2 回答 2

0

该代码似乎正在运行

//get news from group 1

$news=$session->getNews("","","",1);

// for each article work out the date

while($article=mysql_fetch_array($news))
    $date = $article['thedate'];
    $year = date('Y', $date);
    $month = date('F', $date);
...

// then select everything again after working out the date (odd way of doing it)

$innernews=$session->getNews("",$month,$year);

// and output each

所以，因为五月有两个事件，所以它输出了两次标题。让我过一遍...

  • 获取按日期分组的新闻（据称）
    • 六月，仅 1 篇文章
    • 1：
      • 输出标题
      • 选择文章
    • 输出文章
    • 五月，2 篇文章
    • 1：
      • 输出标题
      • 选择文章
      • 输出文章
    • 2：
      • 输出标题
      • 选择文章
      • 输出文章

此 group bySELECT * FROM ".TBL_NEWS." GROUP BY (FROM_UNIXTIME(thedate, '%Y')),(FROM_UNIXTIME(thedate, '%M')) ORDER BY thedate DESC的行为不符合预期，并返回 5 月的两个结果

试试这个代码

<div id="right" class="news">
<h3>Archive</h3>
<? 
$news=$session->getNews();
$bydate=array(); // articles by date
while($article=mysql_fetch_array($news)){
    $k=date('YF', $date);
    if (!isset($bydate[$k])) $bydate[$k]=array(); // create sub array
    $bydate[$k][]=$article; // push article to this sub array
}
foreach ($bydate as $date->$articles){ // run through top array
    ?><h4><?= substr($date,4) . " - " . substr($date,0,4); ?></h4><nav class="small"><? 
    foreach ($articles as $innerarticle){ // now each article within this date
        ?><a href="/news/<?=$innerarticle['ftitle']?>" <? if($title==$innerarticle['ftitle']){?> class="active"<? }?>><?=$innerarticle['title']?></a><?
    }
    ?></nav><?
}
?></div>

请改变

function getNews($title,$month,$year,$group){

至

function getNews($title=NULL, $month=NULL, $year=NULL, $group=NULL){
于 2013-01-03T04:22:44.210 回答
-2

韦尔普。有你的问题。

你的函数说，if($title)。由于 $title 作为函数中的必需参数出现，我认为 PHP 将其注册为，是的，这是一个已设置的变量。所以，基本上会发生什么，然后，你得到一个 mysql_fetch_array 结果，然后你第二次运行 mysql_fetch_array。

尝试：

//in your function getNews()
if($title){
   $q=$this->query("SELECT * FROM ".TBL_NEWS." WHERE ftitle = '$title'" );
   return $q;
}
//rest of function down here

那可以工作。我看到的问题是，这样做，您将在任何其他调用该功能的地方引起问题。所以要小心！上面的修复是确保你让代码进入更好状态的修复。如果你想破解，试试这个：

<? $innernews=$session->getNews("",$month,$year);?>
<? foreach($innernews as $innerarticle) {?>
<a href="/news/<?=$innerarticle['ftitle']?>" <? if($title==$innerarticle['ftitle']){?> class="active"<? }?>><?=$innerarticle['title']?></a>
<? }?>

foreach 循环应该给你你想要的！

于 2013-01-03T03:44:42.743 回答