login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Inhits

Offline Os8 - 31/03/2005 21:34 (laatste wijziging 31/03/2005 21:35)
Avatar van Os8Gouden medaille

PHP interesse
Hallo,
Ik heb een query gemaakt een een neiuwe row in men sql voor hits te tellen, nu heb ik een row die men hits reset :?:
Ik heb zo wat ********* en ben men oude query kwijt die de row update.

  1. mysql_query("UPDATE RedirectURLHosting SET hits ='+1' WHERE hits");


Dat is de query dat ik nu gebruikt.
Hij moet per hit +1 optellen.

17 antwoorden

Gesponsorde links
Offline xSc - 31/03/2005 21:37
Avatar van xSc Onbekend SET hits ='+1'

Dat zal:

SET hits = hits + 1 WHERE id = " . $_GET['id']

zijn.

- Cijfers buiten quotes
- Geef een juiste WHERE mee (zie bovenstaand voorbeeld).
Offline Gerard - 31/03/2005 21:37
Avatar van Gerard Ouwe rakker Maak je WHERE eens af?
Offline Os8 - 31/03/2005 21:59 (laatste wijziging 31/03/2005 22:01)
Avatar van Os8 Gouden medaille

PHP interesse
Hoe bedoel je?

Citaat:
maak je WHERE eens af?


Moet het dan zo??

  1. mysql_query("UPDATE RedirectURLHosting SET hits = hits + 1 WHERE id = " . $_GET['id']");


Hier zit een fout in want kheb error.
Offline Klopper - 31/03/2005 22:11
Avatar van Klopper HTML interesse Welke error? Als je dat verteld kunnen we je misschien verder helpen 
Offline Os8 - 31/03/2005 22:18
Avatar van Os8 Gouden medaille

PHP interesse
Nu heb ik geen error maar hij telt geen hit bij.

  1. <?
  2. mysql_query("UPDATE RedirectURLHosting SET hits=hits+1 WHERE id='$id'");
  3. ?>

Dat is men query nu en ik hij registreert geen hit 
Offline Klopper - 31/03/2005 22:28
Avatar van Klopper HTML interesse Ik zou het zelf op een andere manier hebben gedaan eigenlijk...
  1. <?php
  2. $qry_sel = "SELECT * FROM RedirectURLHosting WHERE id = $_GET['id']";
  3. $res_sel = mysql_query($qry_sel);
  4. while($obj = mysql_fetch_object($res_sel){
  5. $pluseen = $obj->id+1;
  6. $qry_upd = "UPDATE RedirectURLHosting SET hits = $pluseen";
  7. mysql_query($qry_upd);
  8. }
  9. ?>
Offline Maarten - 31/03/2005 22:29
Avatar van Maarten Erelid @Klopper:

het is sowieso al omslachtig, maar waarom dan nog eens een while() gebruiken?
Offline Klopper - 31/03/2005 22:34
Avatar van Klopper HTML interesse Dit is gewoon mijn manier van werken hè 
Als iedereen dezelfde manier van scripten zou hebben, dan zouden er al veel meer OS/2's zijn (als voorbeeld)
Offline Os8 - 31/03/2005 22:37 (laatste wijziging 31/03/2005 22:41)
Avatar van Os8 Gouden medaille

PHP interesse
Potvolkoffie waarom wil die nu nie uit mijn tabel halen?

  1. mysql_query("UPDATE RedirectURLHosting SET hits=hits+1 WHERE ID='".$row->$ID."'");


Zo zou het moeten maar het heeft geen effect!!

Crew edit door Murfy: Niet vloeken.. heb het ff veranderd 
Offline Maarten - 31/03/2005 22:42
Avatar van Maarten Erelid $row->$ID?

Moet dat niet

$row->ID

zijn?
Offline Os8 - 31/03/2005 22:43 (laatste wijziging 31/03/2005 22:48)
Avatar van Os8 Gouden medaille

PHP interesse
Hij wil nog steeds niet registreren.
Volgens mij lig da begot niemeer aan die query
Kzou het niemeer weten 


Citaat:
Structuur:

Kheb tabel 'RedirectURLHosting' met daarin een rij: 'hits'
wanneer iemand een verbinding maakt met een url word er een hit opgeslaan: de query is gemaakt in hits.php en die is geinlude in een frame in de titelbalk zodat die geregistreert zou worden.

Nu zou deze ook moeten geprint worden (das geen probleem)

Het probleem zit hem dus in het registreren of het UPDATE
Offline Maarten - 31/03/2005 23:04
Avatar van Maarten Erelid zet er eens "or die(mysql_error());" achter...
Offline Ontani - 01/04/2005 00:43
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
ge zijt toch wel zeker van da ge wel uw waarde hits uitleest e 
Offline delta_004 - 01/04/2005 07:26
Avatar van delta_004 Onbekend
  1. <?php
  2. //connect
  3. If(IsSet($_GET['id']))
  4. {
  5. mysql_query("UPDATE tabel SET hits = hits + 1 WHERE id='".$_GET['id']."'");
  6.  
  7. }
  8. else
  9. {
  10. //hier als je geen id hebt ingevuld}
  11. ?>
Offline xSc - 01/04/2005 09:26 (laatste wijziging 01/04/2005 09:52)
Avatar van xSc Onbekend
  1. <?php
  2. //connect
  3. if (isSet($_GET['id']))
  4. {
  5.  
  6. $sHits = "UPDATE tabel SET hits = hits + 1 WHERE id = " . addslashes($_GET['id']);
  7.  
  8. if (!mysql_query($sHits))
  9. {
  10.  
  11. trigger_error('Er is een fout opgetreden tijdens het uitvoeren van ' . $sHits . '. MySQL retourneerde: ' . mysql_errno() . ': ' . mysql_error());
  12.  
  13. }
  14.  
  15. }
  16. else
  17. {
  18.  
  19. //hier als je geen id hebt ingevuld
  20.  
  21. }
  22. ?>


* Nummers buiten quotes!
* Pas op voor MySQL injection (vandaar die addslashes())!
* Zorg voor goede foutafhandeling!
Offline delta_004 - 01/04/2005 09:31 (laatste wijziging 01/04/2005 10:51)
Avatar van delta_004 Onbekend maar hij werkt!!!

Edit door Henri

Welke oplossing bedoel je? In jouw oplossing zit een aantal fouten. Zie hiervoor mijn bovenstaande reactie.

Huh hij werkt toch

Edit door Henri

Uitermate slecht argument! 'Het werkt toch' verpest veel. Je moet ook aan beveiliging en efficiency denken!
Offline Frisbee - 01/04/2005 16:40
Avatar van Frisbee HTML beginner je kan beter ook nog ff checken of $_GET['id'] wel een getal is

zo bijvoorbeeld:

if (isset($_GET['id'])&&is_numeric($_GET['id']))
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2025 Sitemasters.be - Regels - Laadtijd: 0.291s