Fout niet te vinden

[Flash-N44B]

Warning: Invalid argument supplied for foreach() in /usr/export/www/hosting/ocg/clan.php on line 137

Die foutmelding krijg ik op mijn site maar ik zie geen fout in het php script

$lasttype = 1;
foreach($clanpower as $type => $info) {
if($type != $lasttype)
print " <tr><td><br></td></tr>\n";

Iemand die me verder kan helpen ( Regel 137 is met Foreach ).

Hartstikke bedankt

[Laiverd]

$clanpower moet een array zijn en is dat niet, of bestaat niet op het moment dat je de for each doet. Als $clanpower uit een databse komt, kan je query ook nog fout zijn (bv. foute kolomnamen gebruikt). Even checken met die (mysql_error()).

John

[Flash-N44B]

Sorry, ik ben niet zo'n PHP freak ik snap je niet hellemaal:$. Zou je het nog eens kunnen uitlegen. Alvast bedankt

[Bargok]

Foreach loopt door een array heen, een array is een multi dimensionale "container" waar je entrys in kunt opslaan.

De error zegt dat er een "invalid argument" is in de foreach, een argument is een meegestuurde waarde naar een functie, in dit geval is de functie foreach.

Foreach wilt als eerste waarde een van het type array, waar het nu probably een enkele waarde is (string/int).

Test dit maar eens, zet $clanpower = array(); boven je foreach, je zult zien dat het werkt.

[Flash-N44B]

Juist, hartstikke bedankt Bargok. Laiverd ook nog bedankt uiteraard.

[Flash-N44B]

Heb ik gelijk een nieuwe vraag:

Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in /usr/export/www/hosting/ocg/shop.php on line 76


en op Line 76 staat het volgende:
while($weapon = mysql_fetch_object($dbres)) {

[Dauntless]

Dan is er iets mis met $dbres. Maar zo'n regels hebben altijd te maken met vorige dingen. We kunnen niet zeggen wat er mis is met $dbres, tenzij dat jij alle regels geeft waarin je $dbres declareert / vult.

(Hetzelfde geldt voor de eerste post: Daar zou je ook moeten laten zien wat $clanpower is, anders kunnen we enkel gokken).

Laat dus even HEEL je script zien (of tenminste alle belangrijke delen mbt de error die je krijgt)

[Bargok]

Mysql_fetch_object vraagt een mysql resource als argument (dit is te controleren met is_resource($arg);. Een mysql resourcse wordt bijvoorbeeld gereturned door mysql_query.

Wat er waarschijnlijk aan de hand is, is dat je query niet klopt ;).

[Flash-N44B]

Deze fout is al opgelost toen ik mysql_fetch_object weghaalde bij alles deed hij het goed en werkte 100%.

Alweer harstikke bedankt. Dan kom ik weer met een nieuwe vraag:

Parse error: syntax error, unexpected T_STRING in /usr/export/www/hosting/ocg/clandonate.php on line 31

Ik heb het desbetreffende bestandje bijgevoegd.:$.. Ook deze fout kan ik niet vinden er zaten veel fouten in de site heb er al veel van opgelost maar deze 3 kan ik echt niet vinden.

[Bargok]

In die lijn staat een echo.
Met die echo wil jij een string weergeven, die string open je met ", en sluit je met ". Bij je html gebruik je ook "", bij name="" bijvoorbeeld. PHP ziet dit als het einde van je string, terwijl je dit niet zo bedoeld. Daarna komt hij een onbekende character tegen, namelijk de "s" van subTitle.

Je kan nu 2 dingen doen:
1. Escapen van tekens d.m.v. een \. De \ zegt tegen php dat hij het volgende character moet negeren.
print "<tr><td class=\"subTitle\"><b>Clan Donatie</b></td></tr>\n";
2. Het gebruiken van singlequotes, wat hier de beste oplossing is.
Nu vragen mensen zich misschien af waarom ik de html single quote: \n werkt enkel tussen dubbele quotes, dit voorkomt dat ik op het einde ."\n" hoef toe te voegen.
print "<tr><td class='subTitle'><b>Clan Donatie</b></td></tr>\n";


Hopelijk heb je hier iets aan ;)

[josko]

In die lijn staat een echo.
Met die echo wil jij een string weergeven, die string open je met ", en sluit je met ". Bij je html gebruik je ook "", bij name="" bijvoorbeeld. PHP ziet dit als het einde van je string, terwijl je dit niet zo bedoeld. Daarna komt hij een onbekende character tegen, namelijk de "s" van subTitle.

Je kan nu 2 dingen doen:
1. Escapen van tekens d.m.v. een \. De \ zegt tegen php dat hij het volgende character moet negeren.
print "<tr><td class=\"subTitle\"><b>Clan Donatie</b></td></tr>\n";
2. Het gebruiken van singlequotes, wat hier de beste oplossing is.
Nu vragen mensen zich misschien af waarom ik de html single quote: \n werkt enkel tussen dubbele quotes, dit voorkomt dat ik op het einde ."\n" hoef toe te voegen.
print "<tr><td class='subTitle'><b>Clan Donatie</b></td></tr>\n";


Hopelijk heb je hier iets aan ;)
Een derde optie, die ikzelf gebruik:

print '<tr><td class="subTitle"><b>Clan Donatie</b></td></tr>'."\n";
Nadeel / Voordelen:

Strings met "":
- Bij de HTML die je wilt weergeven, moet je de dubbele quotes escapen. (" -> \" )
+ Je kunt variablen er gewoon intypen

Strings met '':
- je kunt geen variablen intypen. dit los je op door de . operator, wiens naam ik even vergeten ben. Zal iets van append zijn ( echo 'blaat'. $var; )
+ Je kunt je HTML er gewoon intypen

[Bargok]

."\n";

Dat is dus precies het nadeel die ik bij 2 aangaf ;). Je moet overigens nooit 'zomaar' iets intypen, ook al kan het. Dat is het verschil tussen slordig en georganiseerd programmeren.

[Flash-N44B]

Ja, mijn broer kwam thuis en die heeft ook verstand van PHP. Alleen hij was een lange tijd niet thuis. En toen vroeg ik het hem en hij zei inderdaad precies het zelfde.

Dat het een PHP script is en ik daarin HTML had gezegt en dat dus anders moest oplossen hij heeft de haakjes gewoon in z'n geheel weggehaald dus:

print "<tr><td class=subTitle><b>Clan Donatie</b></td></tr>\n";

Werkte ook prima

Edit: Vergeten, bedankt voor alle hulp nog

[Bargok]

Die aanhalingstekens mag je niet weglaten, dit is ten eerste slordig en ten tweede in strijd met W3C standaarden.

[Laiverd]

Zie hieronder: oorspronkelijke post geedit omdat ik weer eens onzin zat te beweren ;)

John

[Bargok]

<b> is niet depricated. <b> is net zo 'depricated' als strong. Ze zijn beide gewoon verouderd omdat css meer kracht heeft.

[Laiverd]

Je hebt gelijk; <b> is niet deprecated, maar er is wel een voorkeur om de <strong> tag te gebruiken i.v.m. accessibility issue's. My bad.

John

[Flash-N44B]

Snap het neit maar oke, ik merk nu dat <b> er niet eens instaat:P