[PHP] Can't use function return value in write context

[virtualite]

Ik loop tegen een probleem aan in een script. Ik krijg de melding "[PHP] Can't use function return value in write context". Ik vermoed dat ik verderop in mijn php script ook nog tegen problemen aan ga lopen.

Het gaat om dit stuk php.

code:

if(empty(trim($_POST["username"]))){

De code heeft betrekking tot onderstaande stuk php.

code:

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

    if(empty(trim($_POST["username"]))){

        $username_err = 'Please enter username.';

    } else{

        $username = trim($_POST["username"]);

    }

     if(empty(trim($_POST['password']))){

        $password_err = 'Please enter your password.';

    } else{

        $password = trim($_POST['password']);

    }

Nu heb ik gekeken en een kleine aanpassing gedaan, echter resulteerd dit weer in een andere fout, namelijk: Syntax error, unexpected 'if' (T_IF)

De onderstaande code heb ik gebruikt.

code:

$trim_post = trim($_POST["username"]
if (!empty($trim_post)) {

Ik hoop dat jullie me kunnen helpen bij het oplossen van deze error(s) en me vertellen wat ik precies verkeerd doe.

[ Voor 3% gewijzigd door virtualite op 27-09-2017 14:35 ]

[nIghtorius]

je bent een ; vergeten achter trim($_POST["username"]

[virtualite]

nIghtorius schreef op woensdag 27 september 2017 @ 14:36:
je bent een ; vergeten achter trim($_POST["username"]

Aangepast, maar krijg dan de melding syntax error, unexpected ';'.

[Sadieka]

je hebt de functie ook niet gesloten:

trim($_POST["username"]);

[hellfighter87]

$trim_post = trim($_POST["username"]); <= zowel een `;` vergeten als een `)`

[juiced01]

@nIghtorius bedoelt dat je de ; bent vergeten in je laatste code voorbeeld.

virtualite schreef op woensdag 27 september 2017 @ 14:33:

code:

$trim_post = trim($_POST["username"]
if (!empty($trim_post)) {

Er mist een puntkomma en sluitend haakje achter trim($_POST["username"].

[ Voor 2% gewijzigd door juiced01 op 27-09-2017 14:42 . Reden: hellfighter87 was me voor :) ]

[NeFoRcE]

Sinds PHP wat stricter is geworden, loop je tegen dit soort fouten aan.

Je moet eerst de 'TRIM' in een variabele zetten. Die vervolgens op empty controleren, dus:

$trim = trim($_POST["username"]);

if (empty($trim)) {
blabla
}

[0stone0]

Waarom check je of iets leeg is met behulp van trim?

Als iemand een 'spatie' in zn wachtwoord heeft staan, filter je deze eruit.

Ik zal trim pas gebruiken nadat je checkt of het empty is;

code:

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

    if(empty($_POST["username"])) {

        $username_err = 'Please enter username.';

    } else{

        $username = trim($_POST["username"]);

    }

[virtualite]

0stone0 schreef op woensdag 27 september 2017 @ 14:46:
Waarom check je of iets leeg is met behulp van trim?

Als iemand een 'spatie' in zn wachtwoord heeft staan, filter je deze eruit.

Ik zal trim pas gebruiken nadat je checkt of het empty is;

code:

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

    if(empty($_POST["username"])) {

        $username_err = 'Please enter username.';

    } else{

        $username = trim($_POST["username"]);

    }

Goede tip, zal hier naar gaan kijken. Bedankt.

NeFoRcE schreef op woensdag 27 september 2017 @ 14:42:
Sinds PHP wat stricter is geworden, loop je tegen dit soort fouten aan.

Je moet eerst de 'TRIM' in een variabele zetten. Die vervolgens op empty controleren, dus:

$trim = trim($_POST["username"]);

if (empty($trim)) {
blabla
}

Klopt, dit had ik gelezen.

Bedankt allemaal voor het corrigeren. Voel me stom dat ik over ); heen heb gekeken.

[ Voor 40% gewijzigd door virtualite op 27-09-2017 14:49 ]

[TERW_DAN]

virtualite schreef op woensdag 27 september 2017 @ 14:46:
Bedankt allemaal voor het corrigeren. Voel me stom dat ik over ); heen heb gekeken.

Een goede IDE kan daarbij helpen. Dit soort suffe dingen over 't hoofd zien gebeurt iedereen, dus iets met goede syntaxhighlighting e.d. werkt erg prettig.

[juiced01]

Overigens zou ik je willen aanraden om te lezen op php.net wanneer je niet zeker weet wat eea doet. Dan leer je echt wat en waarom je aan kan gebruiken . Er is nog zo veel te leren!

Als je bijvoorbeeld op de pagina over empty kijkt, zie je de note staan:

quote: http://php.net/empty

Note:
Prior to PHP 5.5, empty() only supports variables; anything else will result in a parse error. In other words, the following will not work: empty(trim($name)). Instead, use trim($name) == false.

[NMe]

Als je de foutmelding letterlijk in Google stopt is dit de tweede hit met exact hetzelfde probleem dat jij had én de bijbehorende oplossing. Bovendien zijn syntaxfouten vrijwel altijd vergeten puntkomma's of haakjes (of zoals hier beide) en dus altijd makkelijk door je zelf op te lossen. Een klein beetje meer zelfredzaamheid is in dit geval wel welkom.

[virtualite]

juiced01 schreef op woensdag 27 september 2017 @ 14:49:
Overigens zou ik je willen aanraden om te lezen op php.net wanneer je niet zeker weet wat eea doet. Dan leer je echt wat en waarom je aan kan gebruiken . Er is nog zo veel te leren!

Als je bijvoorbeeld op de pagina over empty kijkt, zie je de note staan:


[...]

Ik ben pas twee weken bezig met het leren van PHP (en programmeren in geheel). Ik zal die site zeker gaan raadplegen.

NMe schreef op woensdag 27 september 2017 @ 14:51:
Als je de foutmelding letterlijk in Google stopt is dit de tweede hit met exact hetzelfde probleem dat jij had én de bijbehorende oplossing. Bovendien zijn syntaxfouten vrijwel altijd vergeten puntkomma's of haakjes (of zoals hier beide) en dus altijd makkelijk door je zelf op te lossen. Een klein beetje meer zelfredzaamheid is in dit geval wel welkom.

Klopt, hier had ik ook mijn aangepaste stuk vandaan. Echter inderdaad stom van mij dat ik de puntkomma en haakje vergat.

[ Voor 39% gewijzigd door virtualite op 27-09-2017 14:53 ]

[juiced01]

Zoals hierboven al gemeld, kijk eens naar een goede IDE (Integrated Development Environment). Deze tools hebben o.a. goeie syntax highlighting (en veul meer handige/essentiële tools), en daarmee zie je direct/snel of je iets verkeerd doet in je syntax. Kijk bijvoorbeeld in deze lijst en probeer er een paar uit.

[DJMaze]

Als je wat verder had gelezen geeft empty() ook true bij '0'.
Nou begrijp ik dat je geen username wil hebben die '0' heet.
Maar de gebruiker '1' kan dan weer wel.

Gebruik gewoon http://php.net/mb_strlen

PHP:

    $username = trim($_POST["username"]);
    // username minimaal 4 karakters
    // geen strlen() omdat dit in Kanji één UTF-8 karakter is (4 bytes)
    if (4 > mb_strlen($username)) {
        $username_err = 'Please enter username.';
    }

[ Voor 7% gewijzigd door DJMaze op 27-09-2017 15:08 ]

[Harrie_]

NeFoRcE schreef op woensdag 27 september 2017 @ 14:42:
Sinds PHP wat stricter is geworden, loop je tegen dit soort fouten aan.

Je moet eerst de 'TRIM' in een variabele zetten. Die vervolgens op empty controleren, dus:

$trim = trim($_POST["username"]);

if (empty($trim)) {
blabla
}

Sorry maar je comment rommelt van alle kanten.

Dit is de code van TS:

PHP:

if(empty(trim($_POST["username"]))){
    $username_err = 'Please enter username.';
}

• 100% correct, geen errors, geen notices. Of het de meest geschikte oplossing is voor wat TS wil is een ander verhaal, maar feitelijk is er niets mis mee.
• Je geeft zelf een voorbeeld waarbij je zelf de var $trim gebruikt. Ik denk dat er geen enkele coding convention is waarbij dat een goed idee lijkt dus leer het alsjeblieft ook niet zo aan TS aan.
• Je hebt een +1 voor die post gekregen, ik vermoed van TS. Wie het ook is geweest, ik vind het jammer, want dat betekent dat tenminste 1 persoon nu waarheid ziet in iets wat absoluut niet klopt.

Edit:

NeFoRcE schreef op woensdag 27 september 2017 @ 15:30:
Het was dan ook meer als een rap 'pseudo' voorbeeld bedoeld

Dat snap ik en dat je je var in de vluggigheid zo'n brakke naam geeft is 1 ding maar de opmerking erboven dat die string modifier niet in je if-clause mag staan en eerst naar een andere var geduwd moet worden dat slaat echt helemaal nergens op en heeft bovendien ook niets te maken met zaken die tegenwoordig in PHP stricter zijn dan voorheen. Gevalletje klok/klepel

[ Voor 32% gewijzigd door Harrie_ op 27-09-2017 15:48 ]

[NeFoRcE]

Harrie_ schreef op woensdag 27 september 2017 @ 15:26:
[...]


Sorry maar je comment rommelt van alle kanten.

Dit is de code van TS:

PHP:

if(empty(trim($_POST["username"]))){
    $username_err = 'Please enter username.';
}

• 100% correct, geen errors, geen notices. Of het de meest geschikte oplossing is voor wat TS wil is een ander verhaal, maar feitelijk is er niets mis mee.
• Je geeft zelf een voorbeeld waarbij je zelf de var $trim gebruikt. Ik denk dat er geen enkele coding convention is waarbij dat een goed idee lijkt dus leer het alsjeblieft ook niet zo aan TS aan.
• Je hebt een +1 voor die post gekregen, ik vermoed van TS. Wie het ook is geweest, ik vind het jammer, want dat betekent dat tenminste 1 persoon nu waar ziet in iets wat absoluut niet klopt.

Het was dan ook meer als een rap 'pseudo' voorbeeld bedoeld

Harrie_ schreef op woensdag 27 september 2017 @ 15:26:

Dat snap ik en dat je je var in de vluggigheid zo'n brakke naam geeft is 1 ding maar de opmerking erboven dat die string modifier niet in je if-clause mag staan en eerst naar een andere var geduwd moet worden dat slaat echt helemaal nergens op en heeft bovendien ook niets te maken met zaken die tegenwoordig in PHP stricter zijn dan voorheen. Gevalletje klok/klepel

Heuu, zeker weten? Ik meen te toch echt te herinneren dat er meer(dere) functies in php zijn nu, die niet meer genest aangeroepen mogen worden, maar eerst in een variabele gestopt moeten worden voor er weer iets mee gedaan kan worden?

[ Voor 28% gewijzigd door NeFoRcE op 27-09-2017 15:50 ]

[NMe]

[quote]Harrie_ schreef op woensdag 27 september 2017 @ 15:26:
• 100% correct, geen errors, geen notices. Of het de meest geschikte oplossing is voor wat TS wil is een ander verhaal, maar feitelijk is er niets mis mee.
Dat is dus wel zo, in PHP 5.4 of ouder. En dat wordt stiekem nog best veel gebruikt.
[quote]• Je geeft zelf een voorbeeld waarbij je zelf de var $trim gebruikt. Ik denk dat er geen enkele coding convention is waarbij dat een goed idee lijkt dus leer het alsjeblieft ook niet zo aan TS aan.
• Je hebt een +1 voor die post gekregen, ik vermoed van TS. Wie het ook is geweest, ik vind het jammer, want dat betekent dat tenminste 1 persoon nu waarheid ziet in iets wat absoluut niet klopt.
Ik vind dat je nogal hard van stapel loopt vanwege een simpele naamgevingissue.

Dat snap ik en dat je je var in de vluggigheid zo'n brakke naam geeft is 1 ding maar de opmerking erboven dat die string modifier niet in je if-clause mag staan en eerst naar een andere var geduwd moet worden dat slaat echt helemaal nergens op en heeft bovendien ook niets te maken met zaken die tegenwoordig in PHP stricter zijn dan voorheen. Gevalletje klok/klepel

Het gevalletje klok/klepel zit er eerder bij jou, want hij zegt niks over wat wel en niet in een if mag staan. Hij zegt wel dat empty in bepaalde PHP-versies alleen variabelen accepteert en dat is een waarheid als een koe. Het enige dat hij verkeerd ziet is dat PHP juist soepeler is geworden op dit vlak: het werkt vanaf 5.5 juist wel op de manier die de TS gebruikt.

NeFoRcE schreef op woensdag 27 september 2017 @ 15:30:
Heuu, zeker weten? Ik meen te toch echt te herinneren dat er meer(dere) functies in php zijn nu, die niet meer genest aangeroepen mogen worden, maar eerst in een variabele gestopt moeten worden voor er weer iets mee gedaan kan worden?

Empty is geen functie maar een language construct. En language constructs hebben inderdaad dit soort restricties. Zelfde geldt voor isset en dergelijken.

[ Voor 14% gewijzigd door NMe op 27-09-2017 16:41 ]

[Morax]

NeFoRcE schreef op woensdag 27 september 2017 @ 15:30:
[...]

Heuu, zeker weten? Ik meen te toch echt te herinneren dat er meer(dere) functies in php zijn nu, die niet meer genest aangeroepen mogen worden, maar eerst in een variabele gestopt moeten worden voor er weer iets mee gedaan kan worden?

Die zijn er zeker weten, maar gaan dan vooral over array modifiers zoals "array_pop" en "array_shift"