[Syfer]

<?php
	echo "<table border=\"1\">";
					$query = mysql_query("SELECT * FROM news ORDER BY news_id") or die(mysql_error());
                    $total_results= mysql_num_rows($query);
	                if ($total_results >0){
					echo "<tr><td>News ID</td><td>News Title</td><td>News Content</td><td>News Picture</td><td>News Date</td></tr>";	
					for($i = 0; $i < mysql_num_rows($query); $i++)				
						$a = mysql_result($query, $i, 'news_id');
						$b= mysql_result($query,$i,'news_title');
						$c= mysql_result($query,$i,'news_content');
						$d = mysql_result($query, $i, 'news_pic');
						$e = mysql_result($query, $i, 'news_date');
                         echo '<tr><td>'.$a.'</td><td>'.$b.'</td><td>'.$c.'</td><td>'.$d.'</td><td>'.$e.'</td><td><a href="editnews.php?action=edit&id='.$a.'">edit</a></td><td><a href="editnews.php?action=delete&id='.$a.'">delete</a></td>';	
						 
				}else{
						echo "<div style='position:absolute;left:420px;font-size:50px;'>No News created!</div>";
					}	
	echo "</table>";				

it shows this kind of error

arning: mysql_result() [function.mysql-result]: Unable to jump to row 2 on MySQL result index 6 in C:\xampp\htdocs\caps\editnews.php on line 82

Warning: mysql_result() [function.mysql-result]: Unable to jump to row 2 on MySQL result index 6 in C:\xampp\htdocs\caps\editnews.php on line 83

Warning: mysql_result() [function.mysql-result]: Unable to jump to row 2 on MySQL result index 6 in C:\xampp\htdocs\caps\editnews.php on line 84

Warning: mysql_result() [function.mysql-result]: Unable to jump to row 2 on MySQL result index 6 in C:\xampp\htdocs\caps\editnews.php on line 85

i paste the lines where i get errors.

p.s

or error from the misplacing of my if statement and its brackets

This post has been edited by Syfer: 08 November 2011 - 06:14 AM

[dD3]

just a quick look.. there are no opening nor closing bracket on this for statement:

 for($i = 0; $i < mysql_num_rows($query); $i++)               

[Atli]

It's usually better to use mysql_fetch_assoc rather than mysql_result.

while($row = mysql_fetch_assoc($query)) {
    $a = $row['news_id'];
    // etc...
}

That way you aren't manually moving around inside the result set, you're just reading it in row by row.

[satis]

You're missing a { on line 7 above, where you're opening your for loop. That may be the cause

for($i = 0; $i < mysql_num_rows($query); $i++) {               

I may also suggest you use PDO or MySQLi instead of the old MySQL functions... they're bad.

Also, lines 8-12 are superfluous... you can just spit out the results directly in your string on line 13. No need to put them in a holder variable. And lastly, using variables like $a, $b, $c etc etc is bad practice. If you ever write code you have to come back to 6 months later, you'll rue using non-descriptive variable names. Even if this is just for learning, it's a good idea to get in practice.

*edit* woo, jeez, all kinds of responses beat mine in. Need to remember to post faster.

This post has been edited by satis: 08 November 2011 - 06:19 AM

[Syfer]

dD3, on 08 November 2011 - 06:16 AM, said:

just a quick look.. there are no opening nor closing bracket on this for statement:

 for($i = 0; $i < mysql_num_rows($query); $i++)               

Atli, on 08 November 2011 - 06:16 AM, said:

It's usually better to use mysql_fetch_assoc rather than mysql_result.

while($row = mysql_fetch_assoc($query)) {
    $a = $row['news_id'];
    // etc...
}

That way you aren't manually moving around inside the result set, you're just reading it in row by row.

tried the both of this
and got the some error

Parse error: syntax error, unexpected T_ELSE in C:\xampp\htdocs\caps\editnews.php on line 88

[satis]

That's actually a different error, bud. Check line 88... you're missing a closing brace. }

[Syfer]

oh yes , thanks