+1 point

Ved godt hvor fejlen lå nu, havde skrevet i ?&id= der havde jeg skrevet "Morten" men skulle selvfølgelig have skrevet 1 . Min ftp svarer heller ikke lige pt, så tester det lige om lidt.

mysql_query("UPDATE `brugere` SET point = point + 1 WHERE id = '$_GET[id]'") or die(mysql_error());

Jeg ville anbefale mit eget eksempel da jeg syntes at det giver en ren kode uden alt for meget irreterende lang kode.

Kode 

$total= "$point"+1;

mysql_query("UPDATE `brugere` SET point = '$total' WHERE id = '$_GET[id]'") or die(mysql_error());

Men det er dig der bestemmer

irriterende lang kode? Jeg tror de fleste kan blive enige om at det er langt pænest og mest effektivt at lade SQL'en ordne den slags

mysql_query("UPDATE `brugere` SET point = point + 1 WHERE id = '$_GET[id]'") or die(mysql_error());

Jeg ville anbefale mit eget eksempel da jeg syntes at det giver en ren kode uden alt for meget irreterende lang kode.

Kode 

$total= "$point"+1;

mysql_query("UPDATE `brugere` SET point = '$total' WHERE id = '$_GET[id]'") or die(mysql_error());

Men det er dig der bestemmer

irriterende lang kode? Jeg tror de fleste kan blive enige om at det er langt pænest og mest effektivt at lade SQL'en ordne den slags

Bestemt. Det er helt standard database optimering at man minimerer antallet af kald mellem database og klient. Ofte kan man også optimere mere komplekse scenarier ved hjælp af stored procedures.

Det er mere komplekst, men giver et mere optimalt resultat.

mysql_query("UPDATE `brugere` SET point = point + 1 WHERE id = '$_GET[id]'") or die(mysql_error());

Jeg ville anbefale mit eget eksempel da jeg syntes at det giver en ren kode uden alt for meget irreterende lang kode.

Kode 

$total= "$point"+1;

mysql_query("UPDATE `brugere` SET point = '$total' WHERE id = '$_GET[id]'") or die(mysql_error());

Men det er dig der bestemmer

Dit eksempel kræver dog, at han kender antallet af point på forhånd, dvs. han allerede har lavet en forespørgsel på `brugere` tabellen ala:

select point from `brugere` where id = '".$_GET['id']."'

, hvorefter han så skal opdatere tabellen med $point+1 (Dvs. 2 forespørgsler i alt).

Robert Larsens løsning kræver derimod ikke den første forespørgsel og bruger altså kun 1 forespørgsel. Om denne løsning er irriterende er så en smagssag, jeg foretrækker den

Edit: Jeg overser også altid pagingen :/

Tak til alle besvarrelser har gjort sådan her:

 <?php
 session_start();
 include("config.php");
 $ip = $_SERVER["HTTP_X_FORWARDED_FOR"];
 if($_GET[id] != "") {
 $lawl = mysql_query("SELECT * FROM besog ORDER BY id DESC") or die(mysql_error());
 $row = @mysql_fetch_array($lawl);
 
 if($ip == $row[ip]){
     echo "<META HTTP-EQUIV=\"REFRESH\" CONTENT=\"0; URL=index.php\">";
 } else {
 mysql_query("INSERT INTO besog(bruger, ip, dato)values ('$_GET[id]', '$ip', now())") OR die (mysql_error());
 mysql_query("UPDATE `brugere` SET point = point + 1 WHERE brugernavn = '$_GET[id]'") or die(mysql_error()); 
 print "<META HTTP-EQUIV=\"REFRESH\" CONTENT=\"0; URL=index.php\">";
 }
 }
 ?>