[PHP] Unexpected { (eerste stappen in coding) - Softwareontwikkeling

woensdag 16 december 2015 08:40

Acties:

0 Henk 'm!

Topicstarter

Ik ben sinds een paar dagen rustig begonnen met PHP, CSS en HTML. Dit doe ik via Codecademy & Freecodecamp.

Nu is 1 van deze dingen, een opdracht om het volgende te doen:

Bij > 5 items, een 10% discount
Bij < 5 items, een 5% discount.

Dit werkt:

code:

        $items = 3;
        
        if($items > 5) {
          echo "You get a 10% discount!";
        }

Echter, als ik dan het volgende toevoeg (volgens de opdracht moet het met een else statement), krijg ik de error:

Parse error: syntax error, unexpected '{' on line 13

code:

      <?php
        $items = 3;
        
        if($items > 5) {
          echo "You get a 10% discount!";
        }
        
        else($items < 5) {
          echo "You get a 10% discount!";
        }

Vooral omdat de site in dit geval geen hulp biedt met de fout, zou ik graag willen weten wat ik fout doe, zodat ik dit in het vervolg kan voorkomen.

Voor de duidelijkheid, de complete code:

code:

<html>
  <head>
  </head>
  <body>
    <p>
      <?php
        $items = 3;
        
        if($items > 5) {
          echo "You get a 10% discount!";
        }
        
        else($items < 5) {
          print "You get a 10% discount!";
        }
        
      ?>
    </p>
  </body>
</html>

[ Voor 16% gewijzigd door corset op 16-12-2015 08:41 ]

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 08:43

Acties:

0 Henk 'm!

emnich

kom je hier vaker?

corset schreef op woensdag 16 december 2015 @ 08:40:
Parse error: syntax error, unexpected '{' on line 13

Lees de error eens goed, daar staat alles in wat je nodig hebt.

Er is een onverwachte '{' op regel 13. Dus PHP treft een { aan op regel 13 die hij niet verwacht had.

woensdag 16 december 2015 08:43

Acties:

+2 Henk 'm!

Salmon

.NET developer

Het is of

PHP:

 if(voorwaarde){

}else{

}

Wat jij wilt is zoiets als:

PHP:

if ($a > $b) {
    echo "a is bigger than b";
} elseif ($a == $b) {
    echo "a is equal to b";
} else {
    echo "a is smaller than b";
}

Zie de PHP documentatie

[ Voor 66% gewijzigd door Salmon op 16-12-2015 08:46 ]

woensdag 16 december 2015 08:45

Acties:

+1 Henk 'm!

corset

Topicstarter

emnich schreef op woensdag 16 december 2015 @ 08:43:
[...]

Lees de error eens goed, daar staat alles in wat je nodig hebt.

Er is een onverwachte '{' op regel 13. Dus PHP treft een { aan op regel 13 die hij niet verwacht had.

Klopt, echter wat ik tot nu toe heb begrepen, klopt dit gewoon. Daarom zit ik dus vast

@Salmon, thanks, ga ik even naar kijken.

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 08:48

Acties:

0 Henk 'm!

Ypho

Allround Nerd

De

Parse error: syntax error, unexpected '{' on line 13

is eigenlijk een beetje misleidend. Omdat je bij een else wel degelijk een { kunt gebruiken. Waar het fout gaat is dat je in de else ook een conditie zet, en dat kan niet.

Je zegt zelf ook: "Als je meer dan 5 items hebt krijg je 10% korting, en anders ..." (if/else)
Je zegt niet: "Als je meer dan 5 items hebt krijg je 10% korting, en anders minder dan 5 items ..." (jouw foute code)
Je zegt dan eerder: "Als je meer dan 5 items hebt krijg je 10% korting, en anders als je minder dan 5 items hebt..." (if/elseif)

[ Voor 4% gewijzigd door Ypho op 16-12-2015 08:48 ]

🃏 TCG Codex - Je volledige TCG verzameling in je broekzak ::: 🍏 TCG Codex for iOS ::: 🤖 TCG Codex for Android

woensdag 16 december 2015 08:51

Acties:

0 Henk 'm!

corset

Topicstarter

Ypho schreef op woensdag 16 december 2015 @ 08:48:
De
[...]
is eigenlijk een beetje misleidend. Omdat je bij een else wel degelijk een { kunt gebruiken. Waar het fout gaat is dat je in de else ook een conditie zet, en dat kan niet.

Je zegt zelf ook: "Als je meer dan 5 items hebt krijg je 10% korting, en anders ..." (if/else)
Je zegt niet: "Als je meer dan 5 items hebt krijg je 10% korting, en anders minder dan 5 items ..." (jouw foute code)
Je zegt dan eerder: "Als je meer dan 5 items hebt krijg je 10% korting, en anders als je minder dan 5 items hebt..." (if/elseif)

Dit was hem! Bedankt. Ik dacht echt dat je het moest definieren dat het < 5 moest zijn.

zoals ik al zei, eerste stappen in coding.

Bedankt voor de hulp allemaal

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 09:41

Acties:

0 Henk 'm!

Caelorum

met >5 en <5 dek je de conditie dat het precies 5 is ook niet af. Tenzij je echt mensen wilt straffen omdat ze een 5e item erbij nemen, lijkt me dat niet echt de bedoeling.

woensdag 16 december 2015 10:02

Acties:

0 Henk 'm!

corset

Topicstarter

Caelorum schreef op woensdag 16 december 2015 @ 09:41:
met >5 en <5 dek je de conditie dat het precies 5 is ook niet af. Tenzij je echt mensen wilt straffen omdat ze een 5e item erbij nemen, lijkt me dat niet echt de bedoeling.

Zou je dan kunnen doen:

code:

<?php
        $items = 3;
        
        if($items > 5) {
          echo "You get a 10% discount!";
        }
        
        else {
          echo "You get a 10% discount!";
        }

    else($items == 5) {
       echo "You get a 50% discount!";
            
    }

Even puur ter test uiteraard

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 10:04

Acties:

0 Henk 'm!

Marco1994

corset schreef op woensdag 16 december 2015 @ 10:02:
[...]

Zou je dan kunnen doen:

code:

<?php
        $items = 3;
        
        if($items > 5) {
          echo "You get a 10% discount!";
        }
        
        else($items < 5) {
          echo "You get a 10% discount!";
        }

    else($items == 5) {
       echo "You get a 50% discount!";
            
    }

Even puur ter test uiteraard

Nee compiled ook niet, je mag geen conditie in de else hebben. Je moet else if gebruiken. Een switch zou ook kunnen in deze use case

woensdag 16 december 2015 10:13

Acties:

0 Henk 'm!

corset

Topicstarter

Marco1994 schreef op woensdag 16 december 2015 @ 10:04:
[...]

Nee compiled ook niet, je mag geen conditie in de else hebben. Je moet else if gebruiken. Een switch zou ook kunnen in deze use case

Thanks! Ik ga daar vandaag even mee vechten. Merk wel dat dit echt meer mijn ding is dan verwacht. Heerlijk kalmerend effect heeft dit.

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 10:43

Acties:

0 Henk 'm!

Alternative

Loop gelijk de rest van je script even door. Je zegt hier

Bij > 5 items, een 10% discount
Bij < 5 items, een 5% discount.

Maar in je script staat overal 10% discount

woensdag 16 december 2015 10:48

Acties:

0 Henk 'm!

CH4OS

It's a kind of magic

De 10% korting is vanaf 5 items en meer? Zo ja, dan is je operator in de if-lus >= ipv enkel >.

Anders heb je t/m 4 artikelen 5% korting en vanaf 6 artikelen 10%...

Je kunt voor de middelste de elseif() gebruiken, of gebruik gaan maken van de switch() functie.

Check ook even de docs van PHP zelf.

If:
http://php.net/manual/en/control-structures.if.php
http://php.net/manual/en/control-structures.elseif.php

Switch:
http://php.net/manual/en/control-structures.switch.php

[ Voor 50% gewijzigd door CH4OS op 16-12-2015 10:51 ]

woensdag 16 december 2015 11:51

Acties:

0 Henk 'm!

NMe

Quia Ego Sic Dico.

Marco1994 schreef op woensdag 16 december 2015 @ 10:04:
[...]

Nee compiled ook niet, je mag geen conditie in de else hebben. Je moet else if gebruiken. Een switch zou ook kunnen in deze use case

Die switch zou dan wel heel raar uitzien voor een beginnende programmeur.

Even voor de volledigheid:

PHP:

switch (true)
{
    case $items < 5:
        echo "You get a 5% discount!";
        break;
    case $items > 5:
        echo "You get a 10% discount!";
        break;
    case $items == 5:
        echo "You get a 50% discount!";
        break;
}

... terwijl je meestal juist de andere kant op vergelijkt door de variabele in de switch-conditie te zetten en de constantes in de case.

[ Voor 10% gewijzigd door NMe op 16-12-2015 11:52 ]

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.

woensdag 16 december 2015 12:06

Acties:

0 Henk 'm!

corset

Topicstarter

NMe schreef op woensdag 16 december 2015 @ 11:51:
[...]

Die switch zou dan wel heel raar uitzien voor een beginnende programmeur. Even voor de volledigheid:
PHP:
1
2
3
4
5
6
7
8
9
10
11
12
switch (true)
{
    case $items < 5:
        echo "You get a 5% discount!";
        break;
    case $items > 5:
        echo "You get a 10% discount!";
        break;
    case $items == 5:
        echo "You get a 50% discount!";
        break;
}
... terwijl je meestal juist de andere kant op vergelijkt door de variabele in de switch-conditie te zetten en de constantes in de case.

En dat is dus PHP waar ik voorlopig nog de ballen verstand van heb. Ja ik snap wat het doet, en hoe het werkt. Maar het zelf bouwen? Oo

"Whatever their future, at the dawn of their lives, men seek a noble vision of man’s nature and of life’s potential."

woensdag 16 december 2015 12:57

Acties:

0 Henk 'm!

Marco1994

NMe schreef op woensdag 16 december 2015 @ 11:51:
[...]

Die switch zou dan wel heel raar uitzien voor een beginnende programmeur. Even voor de volledigheid:
PHP:
1
2
3
4
5
6
7
8
9
10
11
12
switch (true)
{
    case $items < 5:
        echo "You get a 5% discount!";
        break;
    case $items > 5:
        echo "You get a 10% discount!";
        break;
    case $items == 5:
        echo "You get a 50% discount!";
        break;
}
... terwijl je meestal juist de andere kant op vergelijkt door de variabele in de switch-conditie te zetten en de constantes in de case.

Klopt, zat in de bios (star wars is hyped

) de nutteloze reclames te kijken, kon dus niet even iets uit typen

woensdag 16 december 2015 16:20

Acties:

0 Henk 'm!

Candanz

Geflipt

Marco1994 schreef op woensdag 16 december 2015 @ 10:04:
[...]

Nee compiled ook niet, je mag geen conditie in de else hebben. Je moet else if gebruiken. Een switch zou ook kunnen in deze use case

[mierenliefdebedrijven]
php compiled zowiezo niet.
[/mierenliefdebedrijven]

woensdag 16 december 2015 16:27

Acties:

0 Henk 'm!

CH4OS

It's a kind of magic

Candanz schreef op woensdag 16 december 2015 @ 16:20:
[mierenliefdebedrijven]
php compiled zowiezo niet.
[/mierenliefdebedrijven]

offtopic:
Je kan het overigens wel compilen naar binary files, maar dat werkt geloof ik niet zo goed. Zie bijvoorbeeld http://www.phpcompiler.org/ of http://www.thefreecountry.com/compilers/php.shtml.