[LØST] Hjelp med news.php, mysql query

[tAno16]

Hei.

 

Har et nyhetssystem som jobber opp mot mysql. Men hva er det som er feil her som gjør at det ikke blir skrevet ut noe title, date eller content?

 

<?php
$query = "SELECT *," .
"DATE_FORMAT(postdate, '%Y-%m-%d') as date " .
"FROM news ORDER BY id DESC LIMIT $news_limit"; // 1.
$result = mysql_query($query);

while($r=mysql_fetch_array($result)) // 2.
{
echo "<br><table width='100%'><tr bgcolor='$title_cell_color'><td>
<img src='$bullet'><b>$title</b> posted on $date</td></tr>
<tr bgcolor='$news_cell_color'><td>$content</td></tr>
</table><br>";

}

?>

 

Takk, på forhånd.

Endret 24. august 2007 av tAno16

[Crowly]

Sql settningen din er ikke riktig

 

$query = "SELECT *," .

"DATE_FORMAT(postdate, '%Y-%m-%d') as date " .

"FROM news ORDER BY id DESC LIMIT $news_limit"; // 1.

 

Fjern * og spesifiser de feltene du trenger, ikke bruk * hvis ikke du trenger absolutt alle feltene i tabellen.

 

Og du trenger ikke å avslutte hver enkelt linje med ". holder lenge med ett linjeskift. Når du setter variabler inn i en tekststreng, så bruk {} rundt variablen, det er sikrere. Eller konkatiner variablene inn.

 

Du bruker ikke verdiene du henter i ut fra basen riktig.

Bare en liten pirk mysql_fetch_array bruker mere minne enn mysql_fetch_row eller mysql_fetch_assoc, sine mysql_fetch_array inneholder informasjonen to ganger (en gang som assoc og en gang som row)

 

PHP

<?php

$query = "SELECT title_cell_color, bullet, title, news_cell_color, content,

DATE_FORMAT(postdate, '%Y-%m-%d') as date 

FROM news 

ORDER BY id DESC 

LIMIT {$news_limit}"; // 1.

 

//for testing, kan kommenteres ut

echo '<div>'.$query.'</div>';

 

$result = mysql_query($query);

 

while($r=mysql_fetch_assoc($result)) // 2.

{

echo "<br /><table width='100%'><tr style='background-color: ".$r['title_cell_color'].";'><td>

<img src='".$r['bullet']."'><span style='font-weight: bold;'>".$r['title']."</span> posted on ".$r['date']."</td></tr>

<tr style='background-color: ".$r['news_cell_color'].";'><td>".$r['content']."</td></tr>

</table><br />";

}

?>

 

Oppdaterte html koden litt

Endret 24. august 2007 av crowly

[tAno16]

kjempe.

 

Tusen takk crowly. Skal teste dette med en gang

 

FUNKER. Du er en gud.

 

Forresten, bare manglet et komma i slutten av

 

$query = "SELECT title_cell_color, bullet, title, news_cell_color, content

 

så slik:

 

$query = "SELECT title_cell_color, bullet, title, news_cell_color, content,

 

whee

 

Gjør som signaturen din sier

Endret 24. august 2007 av tAno16