[PHP] problem z $this Opcje

[pawel06281990]

Witam,

Nie wiem dla czego działa mi $this na class mam taki komunikat.

Fatal error: Uncaught Error: Using $this when not in object context in /home/twoj-host/web/test.twoj-host.eu/public_html/wp-content/plugins/wp-zmarli/inc/function/function_licz.php:14 Stack trace: #0 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-content/plugins/wp-zmarli/admin/zmarli_list.php(72): liczenie::count_sql() #1 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-content/plugins/wp-zmarli/inc/instalation.php(78): require_once('/home/twoj-host...') #2 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-includes/class-wp-hook.php(303): wp_zmarli->list_zmarli() #3 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-includes/class-wp-hook.php(327): WP_Hook->apply_filters() #4 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-includes/plugin.php(470): WP_Hook->do_action() #5 /home/twoj-host/web/test.twoj-host.eu/public_html/wp-admin/admin.php(259): do_action() #6 {main} thrown in /home/twoj-host/web/test.twoj-host.eu/public_html/wp-content/plugins/wp-zmarli/inc/function/function_licz.php on line 14

W witrynie wystąpił błąd krytyczny. W wiadomości wysłanej na adres e-mail administratora strony umieszczono dodatkowe instrukcje

a tak wygląda function count_sql()

[PHP] pobierz, plaintext 
class liczenie extends wp_zmarli {
 
 function count_sql($name_db, $wartosc =''){
 
	$count_sql = $this->wpdb->get_var("SELECT COUNT($wartosc) FROM ".$name_db."");
 
	if(!$count_sql){
	$count_sql .='0';	
	}
	return $count_sql;
}
[PHP] pobierz, plaintext 

A tak wygląda główna classa

[PHP] pobierz, plaintext 
$wp_zmarli = new wp_zmarli();
class wp_zmarli {
	private $wpdb;
 
	private $wyszukiwarka_table;
	private $wygasa_table;
	private $parafianin_table;
	private $info_cmentarz_table;
	private $zapal_znicz_table;
	private $zapal_znicz_ile_table;
	private $groby_table;
	private $historia_zmarli_table;
	private $zamow_msze_email_templates_table;
	private $users;
 
 
function wp_zmarli(){
	global $wpdb;
        $this->wpdb = $wpdb;
        $this->wyszukiwarka_table = $wpdb->prefix . 'wyszukiwarka';
		$this->wygasa_table = $wpdb->prefix . 'wygasa';
		$this->parafianin_table = $wpdb->prefix . 'parafianin';
		$this->info_cmentarz_table = $wpdb->prefix . 'info_cmentarz';
	    $this->zapal_znicz_table = $wpdb->prefix . 'zapal_znicz';
		$this->zapal_znicz_ile_table = $wpdb->prefix . 'zapal_znicz_ile';
		$this->groby_table = $wpdb->prefix . 'groby';
		$this->historia_zmarli_table = $wpdb->prefix . 'historia_zmarli';
		$this->zamow_msze_email_templates_table = $wpdb->prefix . 'zamow_msze_email_templates';
		$this->users_table = $wpdb->prefix . 'users';
 
 
		require_once(WPNW_DIR.'/inc/function/function_licz.php');
}
[PHP] pobierz, plaintext 

I próbowałem już wszystkiego co robię źle??. Pomóżcie.

[Tomplus]

Paweł,
spójrz na twój komunikat z błędem (co prawda już go nie masz jak czytam) ale jasno ci mówi, że $this nie jest obiektem. Druga rzecz, wskazałeś tam linię z błędami, ale nam nie podałeś od razu która to linia w kodzie w głównym wątku. Możemy tylko zgadywać, gdzie jest problem.

Tak jak ktoś powiedział, używaj typowania, bardzo wyjaśnia czytelność kodu oraz nazywanie zmiennych np:

[PHP] pobierz, plaintext 
	public array $tablicaWyszukiwan; //$wyszukiwarka - czego?
	public array $wygasa; //Co wygasa?
	public array $listaParafian;
	public array $daneCmentrza;
	public array $listaZapalonychZniczy;
	public array $iloscZapalonychZniczy; //czym się to różni od poprzedniego?
	public array $listaGrobow;
	public array $historiaZmarlych; 
	public array $zapowiedziEmailZamawianychMszy;
	private array $users;
[PHP] pobierz, plaintext 

w PHP dobrym obyczajem jest stosowanie camelCase przy nazywaniu zmiennych i metod.

private/public/protected oznacza zasięg tych zmiennyc, więc warto z nich korzystać.