php code check voor criminal site

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Top Low-Code Developer Gezocht!

Bedrijfsomschrijving Unieke Kansen, Uitstekende Arbeidsvoorwaarden & Inspirerend Team Wij zijn een toonaangevende, internationale organisatie die de toekomst van technologie vormgeeft door het creëren van innovatieve en baanbrekende oplossingen. Ons succes is gebaseerd op een hecht en gepassioneerd team van professionals die altijd streven naar het overtreffen van verwachtingen. Als jij deel wilt uitmaken van een dynamische, vooruitstrevende en inspirerende werkomgeving, dan is dit de perfecte kans voor jou! Functieomschrijving Als Low-Code Developer ben je een cruciaal onderdeel van ons team. Je werkt samen met collega's uit verschillende disciplines om geavanceerde applicaties te ontwikkelen en te optimaliseren met behulp van Low-code

Bekijk vacature »

Thomas de vries

thomas de vries

17/05/2012 09:04:42
Quote Anchor link
Goedemiddag beste leden,
Ik ben bezig om een criminal self te scripten om zo veel mogelijk bugs weg te werken
Nu heb ik een crimepage gescript en nu is mijn vraag aan jullie
of dit netjes is en of er dingen beter kunnen.

Code (php)
PHP script in nieuw venster Selecteer het PHP script
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
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
<?php
if($_SESSION['login'] == FALSE)
{

    header('location:/home');
}

else
{
    if(isset($_GET['p'])=='crime')
    {

        echo '<h1>Misdaden</h1>';
        
        $sql_crime_time = "SELECT
        login,
        UNIX_TIMESTAMP(crimetime) AS crimetime
        FROM tijden
        WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
        if(!$res_crime_time = $mysqli->query($sql_crime_time))
        {

            sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
            notify('Probleem met de query:'.$mysqli->error.' in inc.crime.php op lijn 3 op datum '.date("Y-m-d").'');
        }

        else
        {
            //Kijken of de gebruiker binnen de gezette tijd($settings) weer een misdaad wil doen//
            $time = $res_crime_time->fetch_object();
            if($time->crimetime + $crime1time > time())
            {

                $countdown = $time->crimetime+$crime1time-time();
                fout('Je moet nog <font id="count_timer"></font> wachten voor je weer een misdaad kan plegen.
                <script language="javascript">countdown(\''
.$countdown.'\',\'count_timer\',\'/crimes\');</script>');
            }

            else
            {
                if($_SERVER['REQUEST_METHOD'] == "POST")
                {

                    $sql = "SELECT
                    crimenr,
                    crime_min_money,
                    crime_max_money
                    FROM crimepage"
;
                    if(!$res = $mysqli->query($sql))
                    {

                        sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                        notify('Probleem met de query:'.$mysqli->error.' in inc.crime.php op lijn 3 op datum '.date("Y-m-d").'');
                    }

                    else
                    {
                        while($crime = $res->fetch_object())
                        {

                            if($_POST['crime'][$crime->crimenr])
                            {

                                $sql = "SELECT
                                login,
                                aantal_m,
                                crime"
.$crime->crimenr."
                                FROM crime
                                WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                if(!$res = $mysqli->query($sql))
                                {

                                    sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                    notify('Probleem met de query:'.$mysqli->error.' in inc.crime.php op lijn 52 op datum '.date("Y-m-d").'');  
                                }

                                else
                                {
                                    $crime1 = $res->fetch_assoc();
                                    $sql_kans = "SELECT
                                    crimenr,
                                    crimekans
                                    FROM crimekans
                                    WHERE crimenr = '"
.$mysqli->real_escape_string($crime->crimenr)."'";
                                    if(!$res_kans = $mysqli->query($sql_kans))
                                    {

                                        sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                        notify('Probleem met de query:'.$mysqli->error.' in inc.crime.php op lijn 64 op datum '.date("Y-m-d").'');
                                    }

                                    else
                                    {
                                        //Crime opties aanmaken//
                                        $crime_kans = $res_kans->fetch_object();
                                        $kans = mt_rand(0,100);
                                        $vordering = mt_rand(1,5);
                                        $kans2 = $crime1['crime'.$crime->crimenr] / $crime_kans->crimekans;
                                        if($kans2 > 100)
                                        {

                                            $kans2 = 99;
                                        }

                                        $cash = mt_rand($crime->crime_min_money,$crime->crime_max_money);
                                        if($kans < $kans2)
                                        {

                                            $sql_won = "UPDATE crime SET
                                            crime"
.$crime->crimenr." = crime".$crime->crimenr." + '".$vordering."',
                                            aantal_m = aantal_m + '1'
                                            WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                            if(!$res_won = $mysqli->query($sql_won))
                                            {

                                                sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                                notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 90 op datum '.date("Y-m-d").'');
                                            }

                                            else
                                            {
                                                $sql_tijd = "UPDATE tijden SET crimetime = NOW() WHERE login = '".$data->login."'";
                                                if(!$res_tijd = $mysqli->query($sql_tijd))
                                                {

                                                    echo $mysqli->error;
                                                }

                                                $sql_u_won = "UPDATE users SET
                                                cash = cash + '"
.$cash."',
                                                ranking = ranking + '"
.$ranking."'
                                                WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                                if(!$res_u_won = $mysqli->query($sql_u_won))
                                                {

                                                    sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                                    notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 102 op datum '.date("Y-m-d").'');
                                                }

                                                else
                                                {
                                                    goed('De misdaad is gelukt. Je hebt &euro; '.nummer($cash).' gestolen.<meta http-equiv=refresh content=2;URL= />');
                                                }    
                                            }
                                        }

                                        else
                                        {
                                            $jail_kans = mt_rand(0,3);
                                            $kans_verlies = mt_rand(1,2);
                                            
                                            if($jail_kans == 3)
                                            {

                                                $sql_jail = "UPDATE tijden SET
                                                crimetime = NOW(),
                                                jailtime = NOW(),
                                                maxjailtime = '60'
                                                WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                                if(!$res_jail = $mysqli->query($sql_jail))
                                                {

                                                    sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                                    notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 124 op datum '.date("Y-m-d").'');    
                                                }

                                                else
                                                {
                                                    $sql_crime = "UPDATE crime SET aantal_m = aantal_m + '1' WHERE login = '".$mysqli->real_escape_string($data->login)."'";
                                                    if(!$res_crime = $mysqli->query($sql_crime))
                                                    {

                                                        sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                                        notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 134 op datum '.date("Y-m-d").'');
                                                    }

                                                    else
                                                    {
                                                        fout("De misdaad is niet gelukt. Je bent in de gevangenis beland voor 1 minuut.<meta http-equiv=refresh content=2;URL=/jail />");
                                                    }
                                                }
                                            }

                                            else
                                            {
                                                $sql_crime = "UPDATE crime SET
                                                crime"
.$crime->crimenr." = crime".$crime->crimenr." + '".$kans_verlies."'
                                                WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                                if(!$res_crime = $mysqli->query($sql_crime))
                                                {

                                                    sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                                    notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 148 op datum '.date("Y-m-d").'');
                                                }

                                                else
                                                {
                                                    $sql_tijd = "UPDATE tijden SET crimetime = NOW() WHERE login = '".$data->login."'";
                                                    if(!$res_tijd = $mysqli->query($sql_tijd))
                                                    {

                                                        echo $mysqli->error;
                                                    }

                                                    fout("De misdaad is niet gelukt. Je hebt niks gestolen.<meta http-equiv=refresh content=2;URL= />");
                                                }
                                            }
                                        }
                                    }
                                }
                            }
                        }
                    }
                }

                else
                {
                    echo '
                    <form name="spelform" method="post" action=""><table class="content_table"><tr>'
;
                    
                    $sql_crime = "SELECT
                    crimenr,
                    imglink,
                    crimetekst
                    FROM crimepage"
;
                    if(!$res_crime = $mysqli->query($sql_crime))
                    {

                        sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                        notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 176 op datum '.date("Y-m-d").'');
                    }

                    else
                    {
                        for($j=1; $crime = $res_crime->fetch_object(); $j++)
                        {

                            $sql_crime3 = "SELECT crimenr,crimekans FROM crimekans WHERE crimenr = '".$mysqli->real_escape_string($crime->crimenr)."'";
                            if(!$res_crime3 = $mysqli->query($sql_crime3))
                            {

                                sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 190 op datum '.date("Y-m-d").'');
                            }

                            else
                            {
                                $crime3 = $res_crime3->fetch_object();
                                $sql_crime2 = "SELECT
                                login,
                                crime"
.$j."
                                FROM crime
                                WHERE login = '"
.$mysqli->real_escape_string($data->login)."'";
                                if(!$res_crime2 = $mysqli->query($sql_crime2))
                                {

                                    sqlError('Er is een fout gevonden bij het verwerken van de data.<br />Onze excuses voor het ongemak.<br />De beheerder is op de hoogte gesteld.');
                                    notify('Probleem met de query:'.$mysqli->error.' in crime2.php op lijn 199 op datum '.date("Y-m-d").'');
                                }

                                else
                                {
                                    $crime2 = $res_crime2->fetch_assoc();
                                    $kans = $crime2['crime'.$j]/$crime3->crimekans;
                                    if($kans > 99)
                                    {

                                        $kans = 99;
                                    }

                                    echo '
                                    <td><table class="content_table">
                                    <tr>
                                        <td class="tsub">'
.$crime->crimetekst.'</td>
                                    </tr>
                                    <tr>
                                    <td align="center" class="tcell"><input type="image" src="'
.$crime->imglink.'" width="100" height="100" name="crime['.$j.']" value="'.$j.'"/><br/>
                                    
                                    Uw kans is '
.floor($kans).'%</td>
                                    </tr>
                                    </table>'
;
                                    if($j % 4 == 0)
                                    {

                                        echo '</tr><tr>';
                                    }
                                }
                            }
                        }

                        echo '</tr></table>';
                    }
                }
            }
        }
    }
}

?>


Graag positieve feedback zodat ik ermee aan de slag kan.
P.s. De reden dat ik dit vraag is als dit netjes en goed is dat ik het verder kan uitbouwen. zodat ik mij zelf niet te veel op de nek haal.

Met vriendelijke groet,

Thomas de Vries.
 
PHP hulp

PHP hulp

26/04/2024 18:34:00
 
- SanThe -

- SanThe -

17/05/2012 09:40:44
Quote Anchor link
Ik mis de session_start() op de eerste regel. Zonder dat werkt $_SESSION niet.

Dit: if(isset($_GET['p'])=='crime') is geen goede vergelijking. De isset() geeft een true of een false retour en dat ga je nu vergelijken met de tekst 'crime'. Dat werkt niet natuurlijk.

En dit: if($_POST['crime'][$crime->crimenr]) vind ik ook een vreemde vergelijking. Hier kijk je eigenlijk of de $var true of false is.
 
Thomas de vries

thomas de vries

17/05/2012 09:51:24
Quote Anchor link
Deze session start hoeft er niet boven dit omdat het script wordt geinclude
Dus deze werkt zonder de session start.

@if(isset($_GET['p']=='crime') komt omdat ik de url opbouw zo heb

<a href="/crimes/crime">
<a href="/crimes/carboost">

Zonder die if isset krijg ik ook een een lege pagina terug.
Die andere vergelijking komt hiervan
<td align="center" class="tcell"><input type="image" src="'.$crime->imglink.'" width="100" height="100" name="crime['.$j.']" value="'.$j.'"/><br/>

Daar post die de crime['.$j.'] met nummer en dat nummer komt ook weer uit de database.
Zonder die vergelijking lopen de nummers met de crime en de crimekansen die in de database staan niet met elkaar door.
In feite plakt deze het achter elkaar.
Mijn database crime heeft deze rows
crime1 crime2 crime3 enz.

Dit heb ik zo gedaan zodat ik de crime's via een admin pagina kan bewerken/toevoegen/verwijderen zonder dat ik daarvoor het script hoef aan te passen.
Gewijzigd op 17/05/2012 09:55:39 door thomas de vries
 
Erwin H

Erwin H

17/05/2012 10:11:11
Quote Anchor link
Thomas de vries op 17/05/2012 09:51:24:
@if(isset($_GET['p']=='crime') komt omdat ik de url opbouw zo heb

<a href="/crimes/crime">
<a href="/crimes/carboost">

Zonder die if isset krijg ik ook een een lege pagina terug.

Maar het is nog steeds niet correct. Lees de opmerking van Santhe over het gebruik van isset().

Verder zou mijn eerste opmerking zijn dat als je meer dan 10 regels hebt, je moet gaan kijken naar het gebruik van functies. Zo'n grote lap code kan nooit optimaal zijn. Kijk ook eens naar een andere opbouw van je if/else structuren. Het ongebreideld nesten van if/else condities maakt je code niet leesbaarder of robuster.
 
Chris PHP

Chris PHP

18/05/2012 08:46:32
Quote Anchor link
Thomas de vries op 17/05/2012 09:51:24:
@if(isset($_GET['p']=='crime') komt omdat ik de url opbouw zo heb

<a href="/crimes/crime">
<a href="/crimes/carboost">


MEt @ onderdruk je foutmeldingen, is niet aan te raden, en zoals SanYhe en Erwin aangeven is die statement niet geldig, voer hem zo uit.

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
<?php
if (isset($_GET['p']) && $_GET['p'] == "crime") {
// De rest van het script
}
?>


Tevens zijn je <a> geen geldige $_GET links.
Hieronder is een voorbeeld link die een $_GET waarde meegeeft.

<a href="/crimes/je_pagina.php?p=crime>Crime</a>
Gewijzigd op 18/05/2012 08:49:24 door Chris PHP
 
Thomas de vries

thomas de vries

18/05/2012 12:19:46
Quote Anchor link
Die @ was daar niet voor bedoeld maar was als verwijzing daar naar bedoeld
zoals @Chris

Die hyperlink zijn geldig dit omdat het wordt afgevangen met een .htaccess
En zal het zoals wat jij hebt eens doen.
Bedankt voor de tip
 
Chris PHP

Chris PHP

18/05/2012 12:23:38
Quote Anchor link
@Thomas

Aaah ok haha, ik ben hier alleen @persoon gewent ;) Aangezien @ in PHP onderdrukking is, een klein misverstandje :D

Ok ik wist niet dat je dat aangepast hebt in de .htaccess.

Graag gedaan, succes
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.