php - Hvorfor virker dette ikke - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

<< < 1 2 3 4 > >>

Bruger #985 @ 30.03.04 11:10

Hey

nu har jeg kigget min koden igemmen i snart en time og kan ikke finde nogle fejl men det virker stadigvæk ikke her er koden

Kode


<?php 
	// Opret forbindelse til MySQL og vælg database
		include('mysql_funktioner.php');
		aabn_forbindelse_og_vaelgdb();
	
	// Find ip-adresse og dato, tid	
		$ip = $_SERVER["REMOTE_ADDR"];
		$dato = date("d-m-Y H:i");
	
	//finder ud af om brugenre har været her før
		$r = mysql_fetch_row(mysql_query("SELECT * FROM counter WHERE ip like '$ip'")) or die (mysql_error());
        if ($r == (false))  {
		mysql_query("INSERT INTO counter (hits,ip) VALUES ('1','$ip')") or die (mysql_error());
 		}
	
	// Insæt data i databasen
		mysql_query("UPDATE counter SET hits=hits+1 dato='$dato' WHERE ip='$ip'") or die (mysql_error());
	
	// Henter antal hits
	   $gethits = mysql_query("select hits from counter") or die (mysql_error());
		
			while($r = mysql_fetch_array($gethits)) 
			{
				$hits = $r["hits"];
			}

	// Fortæl brugere hvad nummer han/hun er
		echo (" Hits Ialt: $hits");

	// Luk forbindelsen til MySQL
		luk_forbindelse();
   	?>

Håber nogel kan hjælpe

36 svar postet i denne tråd vises herunder
2 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #4003 @ 31.03.04 13:59

You have an error in your SQL syntax near 'dato='2004-03-31 13:42:08' WHERE ip='212.10.48.43'' at line 1

Hmmm....er du sikker på dit datofelt i tabellen er af typen DATETIME!? -jeg begynder at mistænke, at det er DATE eller TIMESTAMP....tjek det lige...

Til dit script SKAL feltet 'dato' i tabellen være af typen DATETIME.

Bruger #985 @ 31.03.04 14:09

You have an error in your SQL syntax near 'dato='2004-03-31 13:42:08' WHERE ip='212.10.48.43'' at line 1

Hmmm....er du sikker på dit datofelt i tabellen er af typen DATETIME!? -jeg begynder at mistænke, at det er DATE eller TIMESTAMP....tjek det lige...

Til dit script SKAL feltet 'dato' i tabellen være af typen DATETIME.

Jeg havde glemt det så den stod som varchar men det hjælp ikke

You have an error in your SQL syntax near 'dato='2004-03-31 14:08:21' WHERE ip='212.10.48.43'' at line 1

Bruger #4003 @ 31.03.04 15:57

Ok, nu er vi der næsten. Så tror jeg kun vi mangler et komma for at være i mål.

Erstat følgende:

mysql_query("UPDATE counter SET hits=(hits + 1) dato='$dato1' WHERE ip='$ip'") or die (mysql_error());

Med nedenstående:

mysql_query("UPDATE counter SET hits=(hits + 1), dato='$dato1' WHERE ip='$ip'") or die (mysql_error());

Venlig hilsen
Jon Krondal Honum

Bruger #985 @ 31.03.04 16:04

Ok, nu er vi der næsten. Så tror jeg kun vi mangler et komma for at være i mål.

Erstat følgende:

mysql_query("UPDATE counter SET hits=(hits + 1) dato='$dato1' WHERE ip='$ip'") or die (mysql_error());

Med nedenstående:

mysql_query("UPDATE counter SET hits=(hits + 1), dato='$dato1' WHERE ip='$ip'") or die (mysql_error());

Venlig hilsen
Jon Krondal Honum

Så var vi der nu virker det men ikke helt den ligger tæller en hver gang siden man trykker på 5f kan man ikke lave sådan at den kun kan ligge et til en gang man dagen ved hver ip

Bruger #4003 @ 31.03.04 16:27

I dag gør du følgende:

1. Du tjekker om ip-adressen er kendt, og hvis den er det, så tæller du 1 op og opdaterer dato for ip-adressen i databasen.

2. Er ip-adressen ikke kendt, så tilføjer du den med '1' i tæller, og datoen i dag.

Altså tælles der altid 1 op! Hvis de kun må tælle 1 gang per dag, så mangler du et tredje tjek:

3. Hvis ip-adressen er kendt i databasen, og allerede står med en tæller i dag (dato feltet = $dato1), så skal der ikke opdateres i databasen, for så har den allerede talt 1 gang i dag.

Hilsen Jon

Bruger #985 @ 31.03.04 20:24

I dag gør du følgende:

1. Du tjekker om ip-adressen er kendt, og hvis den er det, så tæller du 1 op og opdaterer dato for ip-adressen i databasen.

2. Er ip-adressen ikke kendt, så tilføjer du den med '1' i tæller, og datoen i dag.

Altså tælles der altid 1 op! Hvis de kun må tælle 1 gang per dag, så mangler du et tredje tjek:

3. Hvis ip-adressen er kendt i databasen, og allerede står med en tæller i dag (dato feltet = $dato1), så skal der ikke opdateres i databasen, for så har den allerede talt 1 gang i dag.

Hilsen Jon

Jeg tror ikke vi forstår hinanden, jeg godt vil havde er at første gang bruger kommer ind på siden ligges der et til men kommer han så fed andre gang i løben af dagen skal der ikke ske noget

Bruger #4003 @ 01.04.04 09:37

Jeg tror ikke vi forstår hinanden, jeg godt vil havde er at første gang bruger kommer ind på siden ligges der et til men kommer han så fed andre gang i løben af dagen skal der ikke ske noget

Lige nøjagtig....men kommer brugeren igen dagen efter, så skal den tælle en gang igen ikke sandt? -i så fald, så er det lige netop det jeg har beskrevet!

Hilsen Jon

Bruger #985 @ 01.04.04 13:26

Jeg tror ikke vi forstår hinanden, jeg godt vil havde er at første gang bruger kommer ind på siden ligges der et til men kommer han så fed andre gang i løben af dagen skal der ikke ske noget

Lige nøjagtig....men kommer brugeren igen dagen efter, så skal den tælle en gang igen ikke sandt? -i så fald, så er det lige netop det jeg har beskrevet!

Hilsen Jon

Ja okay det er sådan sorry men hvordan laver jeg lige for den skal jo tjekke dato og ip på sammen tid

jeg skal nok ligge nogle flere point i det

Bruger #985 @ 01.04.04 15:22

Ja okay det er sådan sorry men hvordan laver jeg lige for den skal jo tjekke dato og ip på sammen tid

jeg skal nok ligge nogle flere point i det

Jeg har godt nok ikke noget PHP/MySQL miljø hvor jeg sidder lige nu, så jeg har ikke kunne teste, men prøv nedenstående tilrettede script:

Kode


// Tilslut til DB 
include('../inc/funktion.inc.php'); 
aabn_forbindelse_og_vaelgdb(); 

// Find ip-adresse og dato, tid 
$ip = $_SERVER["REMOTE_ADDR"]; 
$dato1 = date("Y-m-d H:i:s");
$dato2 = date("Y-m-d");

//finder ud af om brugeren har været her før 

$r = mysql_fetch_row(mysql_query("SELECT hits,ip,dato,DATE_FORMAT(dato,'%Y-%m-%d') AS kun_dato FROM counter WHERE ip like '$ip'")) or die (mysql_error()); 

if ($r == (false)) { 
	// Første besøg fra ip-adresse. Tilføj i databasen og sæt tæller til 1
	mysql_query("INSERT INTO counter (hits,ip,dato) VALUES ('1','$ip','$dato1')") or die (mysql_error()); 
} 

if (!$r['kun_dato'] == $dato2) {
	// Har ikke talt i dag. Tæl 1 op i databasen
	mysql_query("UPDATE counter SET hits=(hits + 1), dato='$dato1' WHERE ip='$ip'") or die (mysql_error()); 
}

// Henter antal hits 
$gethits = mysql_query("select hits from counter") or die (mysql_error()); 

while($r = mysql_fetch_array($gethits)) 
{ 
$hits = $r["hits"]; 
} 

// Fortæl brugere hvad nummer han/hun er 
echo (" Hits Ialt: $hits"); 

// Luk forbindelsen til MySQL 
luk_forbindelse();

Hilsen Jon

Det hjælp ikke noget den tæller endnu selv om jeg har været der en gang

Bruger #4003 @ 01.04.04 15:51

Det hjælp ikke noget den tæller endnu selv om jeg har været der en gang

Prøv lige at tilføj nedenstående i linjerne før "if (!$r['kun_dato'] == $dato2)" :

echo('kun_dato: ' . $r['kun_dato'] . '<br>');
echo('dato2: ' . $dato2 . '<br>');

Bare lige for at finde ud af, hvad det er for to værdier vi sammenligner

Hilsen Jon

<< < 1 2 3 4 > >>

Hvorfor virker dette ikke

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler