Home/ Questions/Q 7181211
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-28T17:30:35+00:00

I made a very simple view counter. Ultimately I will convert to a cronjob

  • 0

I made a very simple view counter. Ultimately I will convert to a cronjob view counter/updater, but for now I will just use a mysql update query every time the page refreshes. I was wondering if there was any way to turn these 2 statements into one, trying to cut down on processing.

PHP:

$query=mysql_query("SELECT * FROM questions WHERE id='$q'");  //string is escaped (not shown)
while ($row=mysql_fetch_assoc($query)){
    $views=$row['views']+1;
}
mysql_query("UPDATE questions SET views='$views' WHERE id='$q'");

Also, it would be really helpful if you could point out potential security issues if you notice any. thanks

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    Why not just:

    mysql_query("UPDATE questions SET views=views+1 WHERE id='$q'");

    This will increase your views by 1.

    • 0