Fatal error: Call to a member function updatePower() on a non-object in

Frenzo.Brouwer

Hallo,

Momenteel werk ik aan een website waar ik de volgende melding bij krijg:
Fatal error: Call to a member function updatePower() on a non-object in ....

De functie heb ik in een class staan, en overige functie's in de class werken wel gewoon.

Code

public function updatePower($pwrlvl, $waarde, $waarde2, $user2)
        {
            global $db;
            
            $power = array();
            
            $pwr = json_decode($pwrlvl, false);
            
                $power['aanval'] = $pwr->aanval + $waarde;
                $power['defense'] = $pwr->defense + $waarde2;
            
            $array = array('aanval' => $power['aanval'],'defense' => $power['defense']);
            $power_nieuw = json_encode($array);
            
            $items = array(':username' => $user2, ':power' => $power_nieuw);
            return $db->query('UPDATE '.TBL_USERS.' SET `powerlevel` = :power WHERE `username` = :username', $items);
        
        }

Toon Meer

Zo roep ik het op:

Code

$user->updatePower($speler2->powerlevel, $power_beloning, $power_beloning, $speler['voor']);

Wat gaat er fout, dat ik deze error krijg?

stijnhau

die $user waarop je die functie oproept is niet van een klasse.
doe eens een var_dump er op om te bekijken wat die exact bevat dan.

WHMCSAddons

Simpel vertaald:
Je roept een functie aan op een object dat niet bestaat of geen object is.

Frenzo.Brouwer

Als ik de functie doe var_dumpen in mijn userclass met standaart invoer waardes:
object(PDOStatement)#6 (1) { ["queryString"]=> string(67) "UPDATE users SET `powerlevel` = :power WHERE `username` = :username" }

stijnhau

dus die $user is van PDOStatement class dus.
En heb je daar dan een functie op genaamt updatePower ?
vermoedelijk niet.

Frenzo.Brouwer

Jawel in mijn User Class, heb ik een functie updatePower().

stijnhau

Maar je var dump zegt dat het geen userclass is die user.
Post anders je volledige script eens.

Frenzo.Brouwer

Untitled PHP :: Code Dumper

Dit is heel mijn user class.

Alles werkt gewoon naar behoren, alleen bij deze functie geeft hij deze melding.
Wellicht zou het ook door de pagina komen waarop ik de functie uitvoer??

stijnhau

Vermoedelijk is het je pagina en overschijf je $usqer ergens op die pagina.

Frenzo.Brouwer

Dit is de pagina waar ik de error krijg:
Untitled PHP :: Code Dumper
Het gaat in dit geval om regel: 197.

Kan jij zien waar het fout gaat?

Stefan.J

De variabele $user wordt in elk geval niet gedefinieerd in het script dat je postte. Dat is waarschijnlijk ook waar het mis gaat: de variabele is denk ik niet gedefineerd. Als je een een var_dump doet van $user net boven de regel waar het misgaat.

TIP: Gebruik geen global statements. Hierdoor leg je allemaal afhankelijkheden in je code die niet beheerbaar zijn waardoor je een gigantische spagetticode krijgt.

Frenzo.Brouwer

Citaat van Stefan.J

De variabele $user wordt in elk geval niet gedefinieerd in het script dat je postte. Dat is waarschijnlijk ook waar het mis gaat: de variabele is denk ik niet gedefineerd. Als je een een var_dump doet van $user net boven de regel waar het misgaat.
TIP: Gebruik geen global statements. Hierdoor leg je allemaal afhankelijkheden in je code die niet beheerbaar zijn waardoor je een gigantische spagetticode krijgt.

Op het moment als ik:

Code

var_dump($user)

uitvoer, voor ik de functie wil uitvoeren. Dan ontvang ik heel mijn tabel van mijn gebruiker.

Op het moment als ik de bovenstaande code vervang naar

Code

var_dump($user->updatePower())

Krijg ik dus weer die melding???

En dan even een aanvulling op je reactie tot mijn code.
Hoe zou ik dan een andere variabel kunnen gebruiken in een functie? Zoals ik nu dus doe?

Frenzo.Brouwer

Het werkt nu, k heb er een static function van gemaakt.

stijnhau

Heb even de code bekeken en zie nergens waar je die class include.
Dit is ook de plaats waar je dei user aanmaakt.

Frenzo.Brouwer

Citaat van stijnhau

Heb even de code bekeken en zie nergens waar je die class include.
Dit is ook de plaats waar je dei user aanmaakt.

Dat wordt in de index gedaan. Maar de functie werkt perfect, precies zoals ik wil! Alleen weet ik niet waarom het alleen als een static function werkt?

Stefan.J

Waarschijnlijk omdat $user dus geen instantie van User is. Static is sowieso een slecht idee, vooral in het geval van een class als User, er kunnen tenslotte meerdere users zijn (zijn er zelfs waarschijnlijk).

Patrick

Citaat van Frenzo.Brouwer

Dat wordt in de index gedaan. Maar de functie werkt perfect, precies zoals ik wil! Alleen weet ik niet waarom het alleen als een static function werkt?

Door dat je een static functie van maakt hoeft hij maar één keer de gegevens op te vragen en dan blijven de gegevens in de class beschikbaar tot dat de class wordt afgesloten of dat het script afloopt.

Daardoor zal waarschijnlijk jou script nu wel resultaat geven, dit vanwege dat je ergens anders deze class ook al een keer aanroept die wel correct wordt aangeroepen. Hierdoor worden de gegevens statisch gemaakt en beschikbaar gesteld voor gebruik verder in het script.

Zoals eerder aangegeven is het geen slim idee om een functie in een "user" class statisch te maken, hierdoor bind je jezelf vast in een class waar je dat helemaal niet wilt. de variabel $user wordt niet correct aangemaakt waardoor $user niet het object van een class gebruikt. (ergens waar in het script staat $user = new class user of een autoload functie die daar voor zorgt).

In je getoonde script zie ik ook nergens dat $user wordt aangeroepen doormiddel van een class, je krijgt wellicht wel result, wat een restant van een andere aanroep is, maar vanwege dat $user geen onderdeel van het object is krijg je die melding.

Frenzo.Brouwer

Onderaan mijn user class zie je dat ik de class aan maak en defineer als $user.
Deze file wordt geladen in de index.php.

Patrick

Citaat van Frenzo.Brouwer

Onderaan mijn user class zie je dat ik de class aan maak en defineer als $user.
Deze file wordt geladen in de index.php.

en in je aangegeven script wissel je ook met werkingen van $user blijkbaar:

Code

user::login

&

Code

$user->updatePower

is ook niet geheel hoe je een stevige class opbouwt. En ik kan je class niet zien / index aangezien ik die nergens kan vinden...

Frenzo.Brouwer

user::login() gebruik ik om waardes aan te roepen van de gebruiker, dit heb k zo gemaakt omdat het fijn werkt anders heel de tijd een prepare -> array -> execute -> fetch. Nu is het een functie die dit allemaal automatisch doet.

ICTscripters

Dé plek voor IT

Dé plek voor IT

Beta-testers gezocht voor Crypto-oefenplatform

Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

Developer Gezocht

Op zoek naar de legends

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

350 Nieuwe Domeinnamen Januari 2026

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

Fatal error: Call to a member function updatePower() on a non-object in

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken