all to a member function execute() on a non-object

Peter2014 · 7. März 2015

Hallo,

sorry aber ich bin langsam am verzweifeln. Auf einer Seite läuft mein Script auf einer anderen einfach nicht. Ich bekomme diese Meldung:

Was hat dieses zu bedeuten? Mein Code sieht so aus, an dieser Stelle nur das Update, da das insert ohne Probleme klappt.

PHP

if(isset($_POST['abschicken']) && empty($errors)) {

	$test 	= 	$l_header;
	$test1 	=	$l_teaser_img;




	if(isset($_GET['leistungID'])) {




	// Titel Bild löschen
	if($_POST['del_bild'] == 'ja') {
		$h_bild_pfad  = 'img/header/' . $l_header;	
		@unlink($h_bild_pfad);	
		$l_header = $test = "";
	}

	if ($nmb->file_dst_name != ''){
		$test = $nmb->file_dst_name;				
		}




	// Teaser Bild löschen
	if($_POST['del_bild_teaser'] == 'ja') {
		$h_bild_teaser  = 'img/teaser/' . $l_teaser_img;	
		@unlink($h_bild_teaser);	
		$l_teaser_img = $test1 = "";
	}

	if ($nmb_f->file_dst_name != ''){
		$test1 = $nmb_f->file_dst_name;				
		}




		if ($stmt = $mysqli->prepare("Update 
										web_leistungen SET 
										l_startseite=?,
										l_titel=?,
										l_teaser=?
										l_titel_url=?,
										l_header=?,
										l_header_titel=?,
										l_teaser_img=?,
										l_inhalt=?,
										l_sort=?,
										l_status=?

									WHERE l_id = ?"))
		 {    
				$l_startseite 			= $_POST["l_startseite"];
				$l_titel 				= $_POST["titel"];
				$l_teaser 				= $_POST["l_teaser"];
				$l_titel_url 			= $_POST["titel_url"];
				$l_header 				= $test;
				$l_header_titel 		= $_POST["l_header_titel"];
				$l_teaser_img	 		= $test2;
				$l_inhalt 				= $_POST["l_inhalt"];
				$l_sort			 		= $_POST["l_sort"];
				$l_status 				= $_POST["l_status"];

			$stmt->bind_param("sssssssssss",	
										$l_startseite,
										$l_titel,
										$l_teaser,
										$l_titel_url,
										$l_header,
										$l_header_titel,
										$l_teaser_img,
										$l_inhalt,
										$l_sort,
										$l_status,
										$l_id
		 );
	}} // Ende




	$stmt->execute();
				header("Location: http://$host$uri/$extra");
						}
				else {
						echo $mysqli -> error;
					}
	};

Alles anzeigen

Ich habe es jetzt immer wieder mit dem anderen Script vergleichen und es ist so wie ich das sehe, absolut gleich. Vielleicht sehr ihr ja den Fehler

Peter2014 · 7. März 2015

Ich hab den Fehler gefunden, bei:

l_teaser=?

hat das , gefehlt. Warum kann PHP nicht einfach vernünftige Fehlermeldungen ausgegeben? Ganz ehrlich, ich habe jetzt bestimmt gut 2 Stunden den Fehler gesucht

wolf · 7. März 2015

Hier wird eine Bedingung gestellt:

PHP: 26

if ($nmb_f->file_dst_name != ''){

Wenn diese Bedingung nicht erfüllt wird, wird auch kein

PHP: 30

$stmt = $mysqli->prepare()

Ausgeführt. Dadurch ist '$stmt' 'undefined' und kein Objekt, was wiederum bedeutet das man keine Methoden(in deinem fall execute() ) des Objekts benutzen kann.
Pack das '$stmt->execute()' nach Zeile 68 rein (also zwischen 68 und 69) und dann sollten keine Fehler entstehen (allerdings scheint die Bedingung aus Zeile 26 false zu sein, was die Folge hat das du keine Daten bekommen wirst..).

Peter2014 · 7. März 2015

Danke dir für deine schnelle Antwort, ich habe den Fehler allerdings schon gefunden, schau mal ein Posting über deinem. Es läuft jetzt auch wieder. Soll ich dennoch das stmt->execute() eine Zeile höher schieben und würde dann das header("Location: http://$host$uri/$extra"); ebenfalls mit nach oben wandern?

cottton · 7. März 2015

Zitat von Peter2014

Warum kann PHP nicht einfach vernünftige Fehlermeldungen ausgegeben?

Tuts doch =)
Du musst English lesen können - klar.
Wenn Du dann nach ner Weile die Zusammenhänge verstehst (hier zB object / methode ...), dann verstehst Du die Meldungen auf Anhieb.

Zitat

Call to a member function execute() on a non-object

Call = Aufrufen
member function = Methode einer Klasse
execute() = der Name der aufgerufenen Methode
on a non object = auf ein nicht object

=)

BTW
guck Dir mal die Klammern an:

PHP

<?
...
    }} // Ende




    $stmt->execute();
                header("Location: http://$host$uri/$extra");
                        }
                else {
                        echo $mysqli -> error;
                    }
    };

Alles anzeigen

Einrücken!
Da bricht man sich ja einen ab wenn man das lesen will

Zu Deiner letzten Frage: Ja und Nein.
execute() muss dort rein, wo es "funktionieren kann". Außerhalb wird es das nie.

PHP

<?




    if(isset($_POST['abschicken']) && empty($errors)) {

        $test = $l_header;
        $test1 = $l_teaser_img;




        if(isset($_GET['leistungID'])) {




            // Titel Bild löschen
            if($_POST['del_bild'] == 'ja') {
                $h_bild_pfad  = 'img/header/' . $l_header;    
                @unlink($h_bild_pfad);    
                $l_header = $test = "";
            }

            if ($nmb->file_dst_name != ''){
                $test = $nmb->file_dst_name;                
            }




            // Teaser Bild löschen
            if($_POST['del_bild_teaser'] == 'ja') {
                $h_bild_teaser  = 'img/teaser/' . $l_teaser_img;    
                @unlink($h_bild_teaser);    
                $l_teaser_img = $test1 = "";
            }

            if ($nmb_f->file_dst_name != ''){
                $test1 = $nmb_f->file_dst_name;                
            }




            if ($stmt = $mysqli->prepare("
                Update web_leistungen 
                SET 
                    l_startseite=?,
                    l_titel=?,
                    l_teaser=?
                    l_titel_url=?,
                    l_header=?,
                    l_header_titel=?,
                    l_teaser_img=?,
                    l_inhalt=?,
                    l_sort=?,
                    l_status=?
                WHERE l_id = ?
                ")
            ) {    
                $l_startseite   = $_POST["l_startseite"];
                $l_titel        = $_POST["titel"];
                $l_teaser       = $_POST["l_teaser"];
                $l_titel_url    = $_POST["titel_url"];
                $l_header       = $test;
                $l_header_titel = $_POST["l_header_titel"];
                $l_teaser_img   = $test2;
                $l_inhalt       = $_POST["l_inhalt"];
                $l_sort         = $_POST["l_sort"];
                $l_status       = $_POST["l_status"];

                $stmt->bind_param("sssssssssss",    
                    $l_startseite,
                    $l_titel,
                    $l_teaser,
                    $l_titel_url,
                    $l_header,
                    $l_header_titel,
                    $l_teaser_img,
                    $l_inhalt,
                    $l_sort,
                    $l_status,
                    $l_id
                );
                $stmt->execute();   
            } // /if ($stmt = $mysqli->prepare(
        } // /if(isset($_GET['leistungID']))




        
        /* -> weiß ich nicht wohin damit -> */header("Location: http://$host$uri/$extra");




    }
    else {
        echo $mysqli -> error; // <- warum so viel leerzeichen? 
    }
}; // die klammer ist "single" =)

Alles anzeigen

all to a member function execute() on a non-object

Jetzt mitmachen!

Teilen