Witam mam taki problem. Napisałem skrypt glosowania, podczas oddawania głosu żaden błąd się nie wyswitla, lecz liczba głosów w mysql się nie zmienia. Czy mógłby ktoś pomoc?

Kod:

[HTML] pobierz, plaintext 
Głosowanie na najlepszego DJ'a
<br><br>
<form action="/php/pooldj.php" method="get">
<input type="radio" value="DJ Cherry" name="pool">DJ Cheery<br>
<input type="radio" value="DJ Pastyl" name="pool">DJ Pastyl<br>
<input type="radio" value="DJ Mario" name="pool">DJ Mario<br>
<input type="radio" value="Hard Player" name="pool">Hard Player<br>
<input type="radio" value="Perzyk" name="pool">Perzyk<br>
<input type="radio" value="DJ Tasior" name="pool">DJ Tasior<br>
<input type="submit" value="Wyślij">
</form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
 
if (isSet($_GET['pool'])){
$vote = $_GET['pool'];
}
else {
$vote = '';
}
 
if($vote = ''){
echo ("Proszę zaznaczyć 1 z DJ'ów");
return true;
}
 
$link = mysql_connect("host", "login", "pass") or
die(mysql_error());
 
mysql_select_db(bassplay_forum)or
die(mysql_error());
 
$sql = "UPDATE djpool SET
glosy = glosy + 1;
WHERE dj = '" . $_GET['pool'];
 
 
?>
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '" . $_GET['pool']."'";
 
mysql_query($sql);
 
 
?>
 
[PHP] pobierz, plaintext 

podstawy podstaw ...

edit: tam nie zauważyłem, był średnik. Teraz powinno hulać i grzmieć

Zmieniłem wszystko, lecz dalej nie działa. Wartość liczbowa w glosy pozostaje jako 0.

[PHP] pobierz, plaintext 
<?php
 
 
if (isSet($_GET['pool'])){
$vote = $_GET['pool'];
}
else {
$vote = '';
}
 
if($vote = ''){
echo ("Proszę zaznaczyć 1 z DJ'ów");
return true;
}
 
$link = mysql_connect("host", "login", "pass") or
die(mysql_error());
 
mysql_select_db(bassplay_forum)or
die(mysql_error());
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '" . $_GET['pool']."'";
 
mysql_query($sql);
 
 
?>
 
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 
echo $vote."<br />";
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '$vote'";
 
mysql_query($sql) or die('Błąd: ').mysql_error();
 
 
[PHP] pobierz, plaintext 

co wyświetla?

Niestety dalej jest pusta strona.

[PHP] pobierz, plaintext 
<?php
 
 
if (isSet($_GET['pool'])){
$vote = $_GET['pool'];
}
else {
$vote = '';
}
 
if($vote = ''){
echo ("Proszę zaznaczyć 1 z DJ'ów");
return true;
}
 
$link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or
die(mysql_error());
 
mysql_select_db(bassplay_forum)or
die(mysql_error());
 
echo $vote."<br />";
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '$vote'";
 
 
mysql_query($sql) or die('Błąd: ').mysql_error();
 
 
?>
 
 
 
 
[PHP] pobierz, plaintext 

o man... wiem chyba.

to :

[PHP] pobierz, plaintext 
if($vote = ''){
echo ("Proszę zaznaczyć 1 z DJ'ów");
return true;
}
[PHP] pobierz, plaintext 

na to:

[PHP] pobierz, plaintext 
if($vote == ''){
echo ("Proszę zaznaczyć 1 z DJ'ów");
return true;
}
[PHP] pobierz, plaintext 

tu ma być porównanie a nie przypisanie.

a może masz włączone ukrywanie błędów.
daj na początku po <?php

[PHP] pobierz, plaintext 
error_reporting(E_ALL | E_STRICT);
ini_set('display_errors', true);
[PHP] pobierz, plaintext 

i zobacz czy pokaże błędy

Niestety glosu nie dodalo, lecz podczas gdy wyslalem juz formularz na bialej stronie pojawil sie nick DJ'a

[PHP] pobierz, plaintext 
<?php
 
 
if (isSet($_GET['pool'])){
$vote = $_GET['pool'];
}
else {
$vote = '';
}
 
    if($vote == ''){
    echo ("Proszę zaznaczyć 1 z DJ'ów");
    return true;
    }
 
$link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or
die(mysql_error());
 
mysql_select_db(bassplay_forum)or
die(mysql_error());
 
echo $vote."<br />";
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '$vote'";
 
 
mysql_query($sql) or die('Błąd: ').mysql_error();
 
 
?>
 
 
 
 
[PHP] pobierz, plaintext 

teraz:

[PHP] pobierz, plaintext 
<?php
 
 
if (isSet($_GET['pool'])){
$vote = $_GET['pool'];
}
else {
$vote = '';
}
 
    if($vote == ''){
    echo ("Proszę zaznaczyć 1 z DJ'ów");
    return true;
    }
 
$link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or
die(mysql_error());
 
mysql_select_db('bassplay_forum')or
die(mysql_error());
 
echo $vote."<br />";
 
$sql = "UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '$vote'";
 
 
mysql_query($sql) or die('Błąd: ').mysql_error();
 
 
?>
 
[PHP] pobierz, plaintext 

w ogóle masz to trochę dennie zaprojektowane ...

Jest dennie zaprojektowane, bo jestem początkujący
Zła wiadomość, dalej nie działa ;/

a zobacz ten kodzik:

[PHP] pobierz, plaintext 
<?php
 
if (isset($_GET['pool']))
{
    $link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or die(mysql_error());
    mysql_select_db('bassplay_forum') or die(mysql_error());
 
    $dj = mysql_real_escape_string($_GET['pool'], $link);
    $query = sprintf('UPDATE djpool SET glosy = glosy + 1 WHERE dj = "%s"', $dj);
    mysql_query($query, $link) or die(mysql_error());
}
else
{
    echo ("Proszę zaznaczyć 1 z DJ'ów");
}
 
?>
[PHP] pobierz, plaintext 

poczytaj sobie o mysql injection

dobra, spróbujemy inaczej. Sprawdź co pokaże ten kawałek kodu:

[PHP] pobierz, plaintext 
<?php
 
mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or
die(mysql_error());
 
mysql_select_db("bassplay_forum")or die(mysql_error());
 
echo $vote."<br />";
 
mysql_query("UPDATE djpool SET
glosy = glosy + 1 WHERE dj = '$vote'") or die('Błąd: ').mysql_error();
 
 
?>
[PHP] pobierz, plaintext 

w innym wypadku nie mam pojęcia co może być błędem.

swoją drogą jeśli coś jest źle to mysql_error powinien pokazać błąd
ciekawe dlaczego nie pokazuje

obydwa kody nie dzialaja, nie to ze error ale wartosc pozostaje wciaz ta sama

jeśli nie ma errora to ja już zgłupiałem. W końcu jeśli by coś było źle to powinno Ci to wyrzucić.

a teraz:

[PHP] pobierz, plaintext 
<?php
 
if (isset($_GET['pool']))
{
    $link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or die(mysql_error());
    mysql_select_db(bassplay_forum) or die(mysql_error());
 
    $dj = mysql_real_escape_string($_GET['pool'], $link);
    $query = sprintf('SELECT glosy FROM djpool WHERE dj = "%s"', $dj);
    $result = mysql_query($query, $link);
    $row = mysql_fetch_array($result);
 
    $glosy = $row['glosy'] + 1;
 
    $query = sprintf('UPDATE djpool SET glosy = %d WHERE dj = "%s"', $glosy, $dj)
    mysql_query($query, $link) or die(mysql_error());
    mysql_free_result($result);
    mysql_close($link);
}
else
{
    echo ("Proszę zaznaczyć 1 z DJ'ów");
}
 
?>
[PHP] pobierz, plaintext 

@edit
a powiedz czy nazwe bazy masz zdefiniowaną jako stałą czy nie?

Parse error: syntax error, unexpected T_STRING in /home/bassplay/public_html/php/pooldj.php on line 16

Odpowiedz twojego kodu

@edit1
Juz prawdopodobnie wiem co jest nie tak.
Podczas gdy wy dawaliscie mi kody glosowalem na DJ Cheery. Gdy sprobowal zaglosowac na kogos innego, wtedy zadzialalo. Mam jeszcze 1 pytanie tak zeby nie zakladac 2 tematu. Co zrobic aby osoba nie mogla glosowac 2x z 1 komputera?

@edit2
nie

@edit3
<input type="radio" value="DJ Cherry" name="pool">DJ Cheery<br> <-blad jest w tym miejscu

set_cooki http://php.net/manual/en/function.setcookie.php

powinno pomoc ewentualnie spisawac ip do bazy ale to bez sensu jak ktoś chce to i tak oszuka ten cały system

$_COOKIE i zapisanie IP do bazy

[PHP] pobierz, plaintext 
<?php
 
if (isset($_GET['pool']))
{
    $link = mysql_connect("localhost", "bassplay_forum", "ceowwyso12") or die(mysql_error());
    mysql_select_db('bassplay_forum') or die(mysql_error());
 
    $dj = mysql_real_escape_string($_GET['pool'], $link);
    $query = sprintf('SELECT glosy FROM djpool WHERE dj = "%s"', $dj);
    $result = mysql_query($query, $link) or die(mysql_error());
    $row = mysql_fetch_array($result);
 
    $glosy = $row['glosy'] + 1;
 
    $query = sprintf('UPDATE djpool SET glosy = %d WHERE dj = "%s"', $glosy, $dj);
    mysql_query($query, $link) or die(mysql_error());
    mysql_free_result($result);
    mysql_close($link);
}
else
{
    echo ("Proszę zaznaczyć 1 z DJ'ów");
}
 
?>
[PHP] pobierz, plaintext 

ten u mnie działa

@edit
pliczek cookie też można usunąć
ja jak robię antyflood to zapisuję ip w bazie + wysylam cookie i sprawdzam sobie

a czy ktos ewentualnie moglby mi wytlumaczyc jak to zrobic?
bo rozumiem ze to bedzie jakos pobranie ip z tablicy a nastepnie sprawdzenie tego konstrukcja if?

osobiście zrobił bym to na ciasteczkach bo prawda jest taka ze dzieci neo nie obrażając nikogo już nauczyły się restartować modem żeby oglądać filmy

po poprawnym zagloowaniu dodaj linijke setcookie("głosowanie", 1, 60*60*24); <== po jednym dniu ciasteczko się usunie

cooki odczytujesz za pomoca $_COOKI['nazwa'];

[PHP] pobierz, plaintext 
if($_COOKI['nazwa']==1){
 
## juz glosował
}
else{
 
##moze glosowac
 
}
[PHP] pobierz, plaintext 

oczywiście przefiltruj cooki na początku jakimś mysql_escape albo coś

hmm, ale jezeli po 1 dniu iasteczko sie usunie, to na nastepny dzien znow ten ktos bedzie mogl glosowac. czy da rade ustawic takowe ciasteczko na miesiac?

co do "jakimś mysql_escape albo coś " moge poprosic o opis jak to zrobic? na prawde jestem bardzo poczatkujacy w tych sprawach

Da sie.. ale ciastko usunac to 2 klikniecia

hmmm, to jakie rozwiazanie by bylo najlepsze?

ciastko + ip + User Agent.. po wiecej nt user agent zapraszam do jakze cudownego narzedzia jakim jest Google..

opis masz tutaj
jednak ja to bym przefiltrował jakimś stripslashes
lub po prostu sprawdzał czy dana wartość cookie jest taka sama jaka powinna być wysłana

P.S jeśli jesteś naprawdę początkujący to zapoznaj się z manualem

To biorę się do roboty, ale zanim, czy mógłby mi ktoś wytłumaczyć jak ustawiać czas istnienia COOKIE? rozumiem ze to jest jak 60(sec)*60(min)*24(godz)? to jakby to wygladalo aby ustawic na 1 miesiac?

@edit
znalazlem juz tego User Agenta, czy to wyglada jak $_SERVER['HTTP_USER_AGENT']?
jezeli juz ten ciag uzyskalem, to powinienem go dodac do bazy danych i sprawdzac przez if?

czas życia cookie podaje się w sekundach możesz wpisać po prostu ile sekund ma żyć cookie i tyle a on pomnożył czas
na miesiąc ustawisz tak: 60*60*24*30

Tak, sprawdzaj czy sa rowne.. jesli tak to wywalasz echo i koniec skryptu :]
@Up myslalem, ze mu nie podasz chcialem zeby raz chociaz zajrzal do manuala, tam jest wlasnie ten wzor.

Dziekuje za pomoc Daje wszystkim pomogl

$glosy = $row['glosy'] + 1;

$query = sprintf('UPDATE djpool SET glosy = %d WHERE dj = "%s"', $glosy, $dj);

a nie lepiej

$query = sprintf('UPDATE djpool SET glosy = glosy+1 WHERE dj = "%s"', $glosy, $dj);




60*60*24*30 jesli miesiac ma 30 dni