[HaBo]

Catarrer · 05.09.08, 19:33

Moin Hackerboad-Gemeinde,
ich hab mal wieder n Problem.
Und zwar will ich für die Hörer meines Radios eine Chart-Seite (top 100 oda so) machen, wo sie selber vorschläge einbringen können.

Das ganze funktioniert so:
ich gebe in ein Formular den Interpreten und den Titel ein. Dieser wird dann mit einer Datenbank abgeglichen. Wenn dieser schon vorhanden ist, dann wird in der spalte "vote" der Datenbank bei diesem EIntrag die Zahl um eins erhöht. Is ja ne ganz simple rechenaufgabe... ($var++). Is dieser Titel noch nich vorhanden, wird ein neuer eintrag erstellt und die Zahl bei Vote wird auf 1 gesetzt (=1 voter).

wenn ich mir die Charts anschaue, kann ich auch per link wählen ob mir der gefällt.. dementsprechen wird dann auch $var++ oder $var-- eingesetzt und dann mit "UPDATE ... WHERE ..." der eintrag bearbeitet.

Des mit dem Link funktioniert auch super, aber des mit dem eingeben irgend wie nich... er erkennt zwar, dass ein eintrag vorhanden is, und erstellt keinen neuen, aber er setzt vote nicht +1 (also $var++).

hier mal der betroffene Ausschnitt:

PHP-Code:

  [...]



if ($_REQUEST['action']== "edit") {
  if(isset($_POST['action'])) {
  $titel = $_POST['interpret']." - ".$_POST['titel'];
    $charts=$DB->query("SELECT * FROM ".$db_tab['charts']." WHERE titel='".$titel."' LIMIT 1");
    }
  
  if(isset($_POST['send']))  {
    $charts=$DB->query("SELECT * FROM ".$db_tab['charts']." WHERE titel='".$titel."' LIMIT 1");
    $vote=$charts['vote'];
    $if = mysql_num_rows($charts);
    $vote2 = $vote++;
    if ($if > 0) {
      $DB->query("UPDATE ".$db_tab['charts']." SET vote='".$vote2."' WHERE titel='".$titel."'");
    }
    else {
      $DB->query("INSERT INTO ".$db_tab['charts']." (titel, vote) VALUES ('".$titel."','1')");
     }
   }
   
 eval ("\$site_body.= \"".getTemplate("charts_edit","")."\";");
}
elseif ($_REQUEST['action']=="show") {
 $charts = $DB->query("SELECT * FROM ".$db_tab['charts']." ORDER BY 'vote' DESC");  

  for($i = 1, $Export = ""; $i < mysql_num_rows($charts)+1;$i++)
    {
     
   
    
      $platz = $i;
      $titel = $charts['titel'];
    
      eval ("\$charts_row.= \"".getTemplate("charts_row","")."\";");
    
    }
  eval ("\$site_body.= \"".getTemplate("charts1","")."\";");
  
}

[...]

vielleicht findet ihr den fehler.

Gruß Adi

easteregg · 05.09.08, 19:41

ohne jetzt auf deinen speziellen fehler einzugehen(habsch grad irgenwdie nich den nerv zu!), würd ich dir noch raten, statt auslesen des counts und anschliesend wieder einfügen das ganze so realisieren:

Code:

UPDATE Table SET col = col + 1 WHERE irgendwas = dies und jenes!

damit sparst du die nen query

Anzeige

- Anzeige -

Catarrer · 05.09.08, 19:51

den query brauch ich ja sowieso, um zu schauen, ob der eintrag schon vorhanden is (mach ich ja mit mysql_num_rows() )

werd ich ma versuchen... danke für deine schnelle antwort

EDIT: aba is richtig ^^ ein query is überflüssig is mir garned aufgefallen

easteregg · 05.09.08, 19:52

wieso, brauchste nicht, entweder der update query bringt bei mysql_affected_rows 1 oder null zurück.
wenn das null is, machste nen insert.

da haste die ganze problematik auf 2 querys reduziert!

beavisbee · 05.09.08, 20:00

und wenn du einen SELECT-Query machst, wo du nur 1 Feld brauchst, dann ist es wesentlich Datenbank-schonender, wenn du

Code:

SELECT `vote` FROM `".$db_tab['charts']."` WHERE `foo`='bar'

nutzt, anstatt deinem

Code:

SELECT * FROM ...

Catarrer · 05.09.08, 20:05

hmm die idee is gut... da ich noch relativ neu in php bin, hab ich mich mit mysql_affected_rows auseinander gesetzt...

@beavisbee: es werden eh nur 2 daten zurückgegeben und der * war hald für mich weniger schreibarbeit ^^ *lach*

aba wenn mehr zurückgegeben wird und ich so ne abfrage mach, is es wirklich schonender, da hast du recht

da verwend ich den * auch ned, auser ich brauch ihn wirklich *g*

greets adi

skymuss · 06.09.08, 12:09

------------------------------

Anzeige

- Anzeige -

05.09.08, 19:33	#1 (permalink)
Catarrer Registriert seit: 03.08.08 Likes: 0	rechnen mit php und sql Anzeige Moin Hackerboad-Gemeinde, ich hab mal wieder n Problem. Und zwar will ich für die Hörer meines Radios eine Chart-Seite (top 100 oda so) machen, wo sie selber vorschläge einbringen können. Das ganze funktioniert so: ich gebe in ein Formular den Interpreten und den Titel ein. Dieser wird dann mit einer Datenbank abgeglichen. Wenn dieser schon vorhanden ist, dann wird in der spalte "vote" der Datenbank bei diesem EIntrag die Zahl um eins erhöht. Is ja ne ganz simple rechenaufgabe... ($var++). Is dieser Titel noch nich vorhanden, wird ein neuer eintrag erstellt und die Zahl bei Vote wird auf 1 gesetzt (=1 voter). wenn ich mir die Charts anschaue, kann ich auch per link wählen ob mir der gefällt.. dementsprechen wird dann auch $var++ oder $var-- eingesetzt und dann mit "UPDATE ... WHERE ..." der eintrag bearbeitet. Des mit dem Link funktioniert auch super, aber des mit dem eingeben irgend wie nich... er erkennt zwar, dass ein eintrag vorhanden is, und erstellt keinen neuen, aber er setzt vote nicht +1 (also $var++). hier mal der betroffene Ausschnitt: PHP-Code: [...] if ($_REQUEST['action']== "edit") { if(isset($_POST['action'])) { $titel = $_POST['interpret']." - ".$_POST['titel']; $charts=$DB->query("SELECT * FROM ".$db_tab['charts']." WHERE titel='".$titel."' LIMIT 1"); } if(isset($_POST['send'])) { $charts=$DB->query("SELECT * FROM ".$db_tab['charts']." WHERE titel='".$titel."' LIMIT 1"); $vote=$charts['vote']; $if = mysql_num_rows($charts); $vote2 = $vote++; if ($if > 0) { $DB->query("UPDATE ".$db_tab['charts']." SET vote='".$vote2."' WHERE titel='".$titel."'"); } else { $DB->query("INSERT INTO ".$db_tab['charts']." (titel, vote) VALUES ('".$titel."','1')"); } } eval ("\$site_body.= \"".getTemplate("charts_edit","")."\";"); } elseif ($_REQUEST['action']=="show") { $charts = $DB->query("SELECT * FROM ".$db_tab['charts']." ORDER BY 'vote' DESC"); for($i = 1, $Export = ""; $i < mysql_num_rows($charts)+1;$i++) { $platz = $i; $titel = $charts['titel']; eval ("\$charts_row.= \"".getTemplate("charts_row","")."\";"); } eval ("\$site_body.= \"".getTemplate("charts1","")."\";"); } [...] vielleicht findet ihr den fehler. Gruß Adi

05.09.08, 19:41	#2 (permalink)
easteregg Member of Honour Registriert seit: 14.09.07 Likes: 62	ohne jetzt auf deinen speziellen fehler einzugehen(habsch grad irgenwdie nich den nerv zu!), würd ich dir noch raten, statt auslesen des counts und anschliesend wieder einfügen das ganze so realisieren: Code: UPDATE Table SET col = col + 1 WHERE irgendwas = dies und jenes! damit sparst du die nen query __________________ » Flattr mich! - Wenn dir mein Beitrag geholfen hat! « <\| 2 AMD Opterons 2384@ 8x3,2ghz \| Tyan S2915 \| 10GB \| 2x 8800GT \| 8400GS \| Dell 3008WFP + 2x2007FP \|>

05.09.08, 19:52	#4 (permalink)
easteregg Member of Honour Registriert seit: 14.09.07 Likes: 62	wieso, brauchste nicht, entweder der update query bringt bei mysql_affected_rows 1 oder null zurück. wenn das null is, machste nen insert. da haste die ganze problematik auf 2 querys reduziert! __________________ » Flattr mich! - Wenn dir mein Beitrag geholfen hat! « <\| 2 AMD Opterons 2384@ 8x3,2ghz \| Tyan S2915 \| 10GB \| 2x 8800GT \| 8400GS \| Dell 3008WFP + 2x2007FP \|>

05.09.08, 20:00	#5 (permalink)
beavisbee Member of Honour Registriert seit: 22.02.07 Likes: 77	und wenn du einen SELECT-Query machst, wo du nur 1 Feld brauchst, dann ist es wesentlich Datenbank-schonender, wenn du Code: SELECT `vote` FROM `".$db_tab['charts']."` WHERE `foo`='bar' nutzt, anstatt deinem Code: SELECT * FROM ...

06.09.08, 12:09	#7 (permalink)
skymuss Registriert seit: 12.06.08 Likes: 0	------------------------------ Geändert von skymuss (04.06.10 um 09:57 Uhr)

05.09.08, 19:51	#3 (permalink)
Catarrer Themenstarter Registriert seit: 03.08.08 Likes: 0	den query brauch ich ja sowieso, um zu schauen, ob der eintrag schon vorhanden is (mach ich ja mit mysql_num_rows() ) werd ich ma versuchen... danke für deine schnelle antwort EDIT: aba is richtig ^^ ein query is überflüssig is mir garned aufgefallen

05.09.08, 20:05	#6 (permalink)
Catarrer Themenstarter Registriert seit: 03.08.08 Likes: 0	hmm die idee is gut... da ich noch relativ neu in php bin, hab ich mich mit mysql_affected_rows auseinander gesetzt... @beavisbee: es werden eh nur 2 daten zurückgegeben und der * war hald für mich weniger schreibarbeit ^^ lach aba wenn mehr zurückgegeben wird und ich so ne abfrage mach, is es wirklich schonender, da hast du recht da verwend ich den * auch ned, auser ich brauch ihn wirklich g greets adi

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Mit wieviel kann ich für den PC rechnen?	nonpretium	Kaufberatung	6	28.02.07 12:14
Probleme beim rechnen im Trapez	DolphVS	Off topic-Zone	7	21.01.07 15:43
Rechnen mit Formulartdaten ?	Franzl	(Web-) Design und webbasierte Sprachen	2	23.01.06 07:05
[gelöst] Rechnen mit PHP - kein Ergebnis	ioda	Code Kitchen	7	04.11.05 21:46
vernetztes Rechnen (alá SETI)	Scrat	Windows	2	12.07.04 11:13

[HaBo]

rechnen mit php und sql