unexpected }

Gestart doorOne2xs26 reacties81 weergaven

27 berichten · pagina 1 van 2

One2xs

Bot · Topicstarter

Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
Ik krijg de volgende foutmelding:

Parse error: syntax error, unexpected } in /home/*****/domains/*****/public_html/*****.php on line 7


En ik begrijp niet wat ik fout doe, dit is het stukje code:

[COLOR= #0000BB]<?php
$ram[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]];[/COLOR] 

[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]256[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR] 
[COLOR= #007700]}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]512[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR] 
[COLOR= #007700]}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1024[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR] 
[COLOR= #007700]}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1536[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR] 
[COLOR= #007700]}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]2048[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR] 
[COLOR= #007700]}[/COLOR] 
[URL="http://www.php.net/else"]else[/URL] 
[COLOR= #007700]{[/COLOR] 
[URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700]; 
}[/COLOR]
?>


Oplossing:

[COLOR= #0000BB]<?php
$ram[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]];[/COLOR] 

[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]256[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR][COLOR= #007700];
}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]512[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR][COLOR= #007700];
}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1024[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR][COLOR= #007700];
}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1536[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR][COLOR= #007700];
}[/COLOR] 
[URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]2048[/COLOR][COLOR= #007700]) 
{[/COLOR] 
[COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR][COLOR= #007700];
}[/COLOR] 
[URL="http://www.php.net/else"]else[/URL] 
[COLOR= #007700]{[/COLOR] 
[URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700]; 
}[/COLOR]
?>


Wat je ook kan gebruiken:

[COLOR= #0000BB]<?php[/COLOR]

    [URL="http://www.php.net/switch"]switch[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]])
    {
        case[/COLOR] [COLOR= #DD0000]256[/COLOR][COLOR= #007700]:[/COLOR] [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        case[/COLOR] [COLOR= #DD0000]512[/COLOR][COLOR= #007700]:[/COLOR] [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        case[/COLOR] [COLOR= #DD0000]1024[/COLOR][COLOR= #007700]:[/COLOR] [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        case[/COLOR] [COLOR= #DD0000]1536[/COLOR][COLOR= #007700]:[/COLOR] [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        case[/COLOR] [COLOR= #DD0000]2048[/COLOR][COLOR= #007700]:[/COLOR] [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
        default:[/COLOR] [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700];[/COLOR] [URL="http://www.php.net/break"]break[/URL][COLOR= #007700];
    }[/COLOR]

?>
0
Berichten
4.451
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
$bedrag = 3.0;


Na elk bedrag afsluiten met ; wink
0
One2xs

Bot · Topicstarter

Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
Bartv schreef:
$bedrag = 3.0;
Na elk bedrag afsluiten met ; wink
Oja, bedankt smile
0
Berichten
3.112
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Kun je er trouwens niet beter een array van maken?
0
Berichten
10.409
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Uitlijnen van je script is ook niet verkeerd.

[COLOR= #0000BB]<?php
    $ram[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]];[/COLOR] 

    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]256[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]512[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1024[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1536[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR][COLOR= #007700];
    }[/COLOR]     [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]2048[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] 
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700]; 
    }[/COLOR]
?>
0
Berichten
5.274
Reputatie
1
Punten
0,01
Lid sinds
Laatst gezien

Eindhoven, Nederland

Zoiets zou ik je adviseren om uit een db te halen. 100x simpeler.
0
Berichten
6.485
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
@Wouterr: bij < 10 opties is het echt onmogelijk sneller haha
Kingunit schreef:
Uitlijnen van je script is ook niet verkeerd.
[COLOR= #0000BB]<?php
    $ram[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]];[/COLOR] 

    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]256[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]512[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1024[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1536[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR][COLOR= #007700];
    }[/COLOR]     [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]2048[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] 
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700]; 
    }[/COLOR]
?>


Dit is goeie uitlijning wil je zeggen?

@Jellesch: Er is een kleine vragen topic voor dit soort foutjes.
0
Berichten
3.112
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Wouterr schreef:
Zoiets zou ik je adviseren om uit een db te halen. 100x simpeler.
Waarom?
Zie hier niet echt een voordeel in.
0
One2xs

Bot · Topicstarter

Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
Wouterr schreef:
Zoiets zou ik je adviseren om uit een db te halen. 100x simpeler.
Als je de producten nooit/bijna nooit wijzigt heeft dat geen meerwaarden.
0
Berichten
6.485
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Als je producten meer dan 1 kenmerk hebben wel, maar anders niet. En simpeler/sneller is het sowieso niet.
0
Berichten
10.409
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Rogier schreef:
@Wouterr: bij < 10 opties is het echt onmogelijk sneller haha
Kingunit schreef:
Uitlijnen van je script is ook niet verkeerd.
[COLOR= #0000BB]<?php
    $ram[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$_GET[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]product[/COLOR][COLOR= #007700]];[/COLOR] 

    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]256[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]3.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]512[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]5.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1024[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]10.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]1536[/COLOR][COLOR= #007700])  {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]15.0[/COLOR][COLOR= #007700];
    }[/COLOR]     [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$ram[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #0000BB]2048[/COLOR][COLOR= #007700]) {[/COLOR] 
        [COLOR= #0000BB]$bedrag[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]25.0[/COLOR][COLOR= #007700];
    }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] 
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Er is geen product gevonden![/COLOR][COLOR= #007700]; 
    }[/COLOR]
?>


Dit is goeie uitlijning wil je zeggen?

@Jellesch: Er is een kleine vragen topic voor dit soort foutjes.

Iedereen heeft een eigen manier van uitlijnen, ga ik geen discussie over beginnen.
0
One2xs

Bot · Topicstarter

Berichten
93.699
Reputatie
0
Punten
1,27
Lid sinds
Laatst gezien
// laat maar
0
Berichten
4.214
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Waarom geen array?

$bedrag = array("256" => "3.0", "512" => "5.0");

Dan hoef je alleen maar $bedrag[aantal ram geheugen]; en je hebt het bedrag.

Edit: om te checken of het product wel bestaat kun je de in_array functie gebruiken.
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren