PHPhulp.nl PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
D

call to a member function children() on a non-object

Door Daniel van Seggelen op 22-08-2018 15:07
2.108 views
Als ik onderstaande code run, dan krijg ik de foutmelding in de titel.
Ik probeer dat te omzeilen, maar dat gaat niet. Weet iemand wat ik verkeerd doe?




foreach($detail_html->find('div[class=row grid]') as $gg) 
							{							
						
								$oppervlakte =  $gg->children(0)->next_sibling()->next_sibling()->children(0)->next_sibling()->children(0)->children(0)->children(0)->next_sibling();
								$house_type = $gg->children(0)->next_sibling()->children(0)->next_sibling()->children(0)->children(0)->children(0)->next_sibling();



    $isObject = false;
    if ( false !== $gg->children(0)->next_sibling()->next_sibling()->next_sibling()->children(0)->next_sibling()->children(0)->children(0)->children(0)->next_sibling()  ) {
        $isObject = true;
		$aantal = $gg->children(0)->next_sibling()->next_sibling()->next_sibling()->children(0)->next_sibling()->children(0)->children(0)->children(0)->next_sibling();
    }
    if($isObject){
	$arr_test[] = array('rooms',trim(strip_tags( $aantal)));	
	}
Thomas van den Heuvel
22-08-2018 15:39
Allereerst, fix je inspring eens. Een treinongeluk zit er nog overzichtelijker uit.

Vervolgens, voorzie je code eens van commentaar, zodat i.i.g. voor een ongeïnitieerd persoon duidelijk is wat je probeert te doen.

Dan over het topic zelf: is dit een soort van webscraper ofzo? Uitgaan van een specifieke positie van een bepaald element is nogal foutgevoelig lijkt mij. Op het moment dat de layout verandert of varieert gaat dit al snel stuk. Ik zou je dan ook aanraden om deze informatie gestructureerd uit te lezen via een feed of API.

We hebben ook niet zoveel aan de foutmelding omdat:
- er ik weet niet hoeveel keer er in het bovenstaande fragment "children" staat, bij welke children loopt deze vast?
- we niet weten hoe "$gg" er uitziet

Dus, eh. Gg.
D
Daniel van Seggelen
22-08-2018 16:57

Allereerst, fix je inspring eens. Een treinongeluk zit er nog overzichtelijker uit.


Voor jou wellicht, maar het is nogal duidelijk wat zich hier afspeelt.

1) Het is duidelijk dat het in en loop werkt.Het is ook duidelijk dat de childrens en nextsibblings staan. Even tellen dat weet je hoeveel. Daar gaat het niet om. Het komt erop neer dat in sommige lussen deze informatie wel bestaat en bij andere niet.

2) Het enige wat ik dus wil is wanneer deze " $aantal" dus niet bestaat dat hij niet vastloopt op een FATAL error, maar dat hij de lus gewoon door blijft lopen.

Daniel
R
Rob Doemaarwat
22-08-2018 17:14
Daniel van Seggelen op 22/08/2018 16:57:42

Voor jou wellicht, maar het is nogal duidelijk wat zich hier afspeelt.

Voor jou wellicht. Maar je vraagt ons hier om even voor nop jouw probleem op te lossen. Dan is misschien niet al teveel moeite om het een beetje netjes te presenteren?

Als ik snel kan zien wat er precies gebeurt, dan wil ik er best even naar kijken. Maar als ik eerst uit moet zoeken wat bij wat hoort, en daarvoor ook nog steeds van links naar rechts moet scrollen dan klik ik gewoon verder.
D
Daniel van Seggelen
22-08-2018 17:30

Als ik snel kan zien wat er precies gebeurt, dan wil ik er best even naar kijken. Maar als ik eerst uit moet zoeken wat bij wat hoort, en daarvoor ook nog steeds van links naar rechts moet scrollen dan klik ik gewoon verder.


Dat hoeft helemaal niet. Ik zeg dus ook dat deze children , nextsiblings part soms wel bestaat en soms niet. Ik wil gewoon dat hij de fatal error skipt als hij niet bestaat en dat schijnt me niet te lukken
Ward van der Put
23-08-2018 09:14
Splits de method chain ...->...->...->... op in losse stappen en voeg daaraan een if toe die hetzij controleert of een methode bestaat, hetzij of er überhaupt een object van het juiste type ergens uit komt.
Ivo P
23-08-2018 12:46 gewijzigd op 23-08-2018 12:47
method_exit() hoef je mi. niet te gebruiken: de foutmelding zegt "on a non-object".

$gg is dus geen object. (zal wel FALSE of NULL zijn). Maar de controle of $gg een object is (en eventueel ook nog van de class X) zou voldoende moeten zijn.

Dwz: dat lost het eerste deel op, maar ik schat in dat als $gg wel een object is, het ook wel een van het verwachte type zal zijn.
Paul Ulje
23-08-2018 15:53 gewijzigd op 23-08-2018 16:00
"Voor jou wellicht, maar het is nogal duidelijk wat zich hier afspeelt."

Ja, dat denk ik ook.
Hier wordt gepoogd van een webpagina een veldje uit te lezen.
Ik heb tevens de indruk dat er JS(on) wordt geplengd in een PHP syntax.

Maar gelukkig heb ik meer verstand van verzekeren. :-)

Reageren

Inloggen om te reageren