[hulp gevraagd]submit met radio
Hahaha.
Maar die $_GET, waar krijgt die dan zijn waarde?
@pascal: het is maar 1 formuliertje maar zoals hierboven is uitgelegd krijgt je het ID via de GET. dat heb ik gedaan omdat je als je hebt gevote dat je dan gelijk naar de volgende pagina word doorverwezen.
(ik wil ooit ook nog checks voor ip's erop zetten zodat ze niet kunnen smokkelen met hun votes, maar dat gaat nu nog mijn pet te boven:P)
(bedankt dat jullie mij raar vinden dat is namelijk mijn tweede naam:P)
dit is nu mijn code (die is werkend) misschien kunnen jullie hem nog een keer bekijken en verbeterpunten aangeven.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
<?php
$con = mysql_connect('192.168.1.155','ontwikkeling','kipei');
mysql_select_db('simon',$con);
(is_numeric($_GET['id']) ? $n_id=$_GET['id'] : die($_GET['id']." bevat geen numerieke waarde"));
if (ereg("^[0-9]+$", $_GET['id'])) {
$sql = 'SELECT *
FROM votes
WHERE id = "'.$_GET['id'].'"';
if (!$res = mysql_query($sql)) {
trigger_error (mysql_error ());
} else {
while($row = mysql_fetch_array($res)){
echo '<img src="'.$row['path'].'" />';
echo '<br/><br/>';
echo 'numbers voted for hot :'.$row['hot'].'<br/>';
$not = $row['totaal'] - $row['hot'];
echo 'numbers voted for not : '.$not.'<br/>';
echo 'total votes: '.$row['totaal'].'<br/>';
$gem = ($row['hot'] / $row['totaal']) * 100;
$afgerond = sprintf('%01.2f', $gem);
echo 'avarage score: '.$afgerond. '<br/><br/>';
}
$query = "SELECT COUNT(1) FROM votes";
if (!$result = mysql_query($query)) {
trigger_error (mysql_error ());
} else {
$row = mysql_fetch_row($result);
mysql_query('UPDATE votes SET hot WHERE id = '.$_GET['id'].'');
echo '<form action="#" method="post">';
echo '<input type="hidden" name="saveid" value="'.$_GET['id'].'" />';
echo 'HOT: <input type="radio" onclick="form.submit()" name="hot" value="1" />';
echo 'NOT: <input type="radio" onclick="form.submit()" name="not" value="0" />';
echo '</form>';
}
if(isset($_POST['hot'])){
if(is_numeric($_POST['saveid'])){
$sql ='UPDATE votes
SET hot = hot + '.$_POST['hot'].'
WHERE id='.$_POST['saveid'].'';
if($rst = mysql_query($sql)){
}else{
}
$sql ='UPDATE votes
SET totaal = totaal + 1
WHERE id='.$_POST['saveid'].'';
if($rst = mysql_query($sql)){
echo "<p>You Voted \"HOT\"</p>";
}else{
}
}
}
if(isset($_POST['not'])){
if(is_numeric($_POST['saveid'])){
$sql ="UPDATE votes
SET totaal = totaal + 1
WHERE id='".$_POST['saveid']."'";
if($rst = mysql_query($sql)){
echo "<p>You Voted \"NOT\"</p>";
}else{
}
}
}
}
}else{
echo 'the ID must be an integer';
}
?>
$con = mysql_connect('192.168.1.155','ontwikkeling','kipei');
mysql_select_db('simon',$con);
(is_numeric($_GET['id']) ? $n_id=$_GET['id'] : die($_GET['id']." bevat geen numerieke waarde"));
if (ereg("^[0-9]+$", $_GET['id'])) {
$sql = 'SELECT *
FROM votes
WHERE id = "'.$_GET['id'].'"';
if (!$res = mysql_query($sql)) {
trigger_error (mysql_error ());
} else {
while($row = mysql_fetch_array($res)){
echo '<img src="'.$row['path'].'" />';
echo '<br/><br/>';
echo 'numbers voted for hot :'.$row['hot'].'<br/>';
$not = $row['totaal'] - $row['hot'];
echo 'numbers voted for not : '.$not.'<br/>';
echo 'total votes: '.$row['totaal'].'<br/>';
$gem = ($row['hot'] / $row['totaal']) * 100;
$afgerond = sprintf('%01.2f', $gem);
echo 'avarage score: '.$afgerond. '<br/><br/>';
}
$query = "SELECT COUNT(1) FROM votes";
if (!$result = mysql_query($query)) {
trigger_error (mysql_error ());
} else {
$row = mysql_fetch_row($result);
mysql_query('UPDATE votes SET hot WHERE id = '.$_GET['id'].'');
echo '<form action="#" method="post">';
echo '<input type="hidden" name="saveid" value="'.$_GET['id'].'" />';
echo 'HOT: <input type="radio" onclick="form.submit()" name="hot" value="1" />';
echo 'NOT: <input type="radio" onclick="form.submit()" name="not" value="0" />';
echo '</form>';
}
if(isset($_POST['hot'])){
if(is_numeric($_POST['saveid'])){
$sql ='UPDATE votes
SET hot = hot + '.$_POST['hot'].'
WHERE id='.$_POST['saveid'].'';
if($rst = mysql_query($sql)){
}else{
}
$sql ='UPDATE votes
SET totaal = totaal + 1
WHERE id='.$_POST['saveid'].'';
if($rst = mysql_query($sql)){
echo "<p>You Voted \"HOT\"</p>";
}else{
}
}
}
if(isset($_POST['not'])){
if(is_numeric($_POST['saveid'])){
$sql ="UPDATE votes
SET totaal = totaal + 1
WHERE id='".$_POST['saveid']."'";
if($rst = mysql_query($sql)){
echo "<p>You Voted \"NOT\"</p>";
}else{
}
}
}
}
}else{
echo 'the ID must be an integer';
}
?>
Gewijzigd op 01/01/1970 01:00:00 door simon
simon schreef op 06.11.2007 11:59:
@santhe: de $_GET['id'] krijgt zijn waarde van de url. dus www.blabla.com/?id=2
En waar doe je dat?
of zit ik nu in de verkeerde richting te denken?
Verbeterings punten al je echo's in aan een variabel stoppen en op het laatste moment 1 echo doen :) probeer delen in functies te schrijven ipv alles linieer word de code een stuk makelijk te onderhouden!
Zou het zoiets kunnen worden, moet je natuurlijk wel al die functies zelf schrijven:
[linebreak]Code (php) 1234567891011121314151617 <?phpif(isGeldigNum($_GET[id])){ if(isAlgepost($_POST[waarde])) { verwerkFormulier(); } else { toonFormulier($geefWaardeMeeAlsParam); }}else{toonError();}?> [linebreak]
kijken of ik de code dan mooier en korter kan maken :P