Here's another Q....

[carp]

Parse error: parse error, unexpected T_INC in /home/djstop/public_html/test/phatfx.php on line 322

my code:

<?

        myconnect();


     $query = "SELECT * FROM `songs` ORDER BY `id` DESC";
     $result = mysql_query($query);
     $rows = @mysql_num_rows($result);

     $foundentries = false;

     while ($i < $rows) {
       $foundentries = true;

  $id = @mysql_result($result, $i, 'id');
  $title = @mysql_result($result, $i, 'title');
?>
<option value="<?= $id; ?>"><?= $title; ?></option>
<?

     i++;
}
    myclose;
?>

[Gimlin]

<option value="<?= $id; ?>"><?= $title; ?></option>

whats with the "=$id" should just be $id, same for title

[DJ P@CkMaN]

the <?= $id; ?> is just a short way of typing <?php echo $id; ?>

[The Hobo]

Originally posted by DJ P@CkMaN
the <?= $id; ?> is just a short way of typing <?php echo $id; ?>

and also a very bad programming practice.

[The Hobo]

Why don't you just do this?:

<?php

  myconnect();


  $query = "SELECT * FROM `songs` ORDER BY `id` DESC";
  $result = mysql_query($query);
  $rows = @mysql_num_rows($result);

  $foundentries = false;

  while ($i < $rows) {
      $foundentries = true;

      $id = @mysql_result($result, $i, 'id');
      $title = @mysql_result($result, $i, 'title');

      echo "<option value=\"$id\">$title</option>";

      i++;
  }
  myclose;
?>

[phpman]

take the stupid ` off the column names and table

and you should never use @ to surpress the errors until you know it will work without getting an error. see you have an error and you use the @ so how do you know where the error is being generated from? nothing towards you hobo I am just stating this in general.


Carp: what line is 322? and try not to use mysql_result as it is a lot slower than mysql_fetch_array().

[phpman]

<?php

  myconnect();

  $query = "SELECT * FROM songs ORDER BY id DESC";
  $result = mysql_query($query);    
echo "<form>";      
  while ($row = mysql_fetch_array($result)) {
      $foundentries = true;

echo "<option value=\"".$row["id"]."\">$row["title"]</option>";

  }
echo "</form>";
  myclose();
?>

[Gimlin]

Originally posted by DJ P@CkMaN
the <?= $id; ?> is just a short way of typing <?php echo $id; ?>

Learn something new everyday

[cpradio]

Originally posted by phpman

<?php

  myconnect();

  $query = "SELECT * FROM songs ORDER BY id DESC";
  $result = mysql_query($query);    
echo "<form>";      
  while ($row = mysql_fetch_array($result)) {
      $foundentries = true;

echo "<option value=\"".$row["id"]."\">$row["title"]</option>";

  }
echo "</form>";
  myclose();
?>

<?php

  myconnect();

  $query = "SELECT * FROM songs ORDER BY id DESC";
  $result = mysql_query($query);    
  echo "<form>";      
  while ($row = mysql_fetch_array($result)) {
      $foundentries = true;

      echo "<option value=\"".$row["id"]."\">".$row["title"]."</option>";

  }
  echo "</form>";
  myclose();
?>

[The Hobo]

Originally posted by phpman
nothing towards you hobo I am just stating this in general.

I didn't put them in there, I just copied his code. In fact, I didn't even realize they were in there.

[carp]

I fixed my error, All I had to do was put a $ before i++. I would have said this before but I couldn't connect to the server.

[phpman]

but our way you don't need to add the i variable and is more proficient.