[PHP] funkcja array_keys i zmienne w klasach Opcje

[Zdunek]

Witam

Robię sobie mały portal na bazie IPB
oto funkcja klasy

[PHP] pobierz, plaintext 
<?php
public function exist($id){
    $exist=strip_tags($id);
    $idzapytania = mysql_query(&#092;"SELECT `forum_id` FROM `ipb_topics` WHERE `tid`='\".$exist.\"'\");
    $wiersz = mysql_fetch_row($idzapytania); 
    if ( array_keys($news[$lang],7) || array_keys($arts[$lang],7))
    {
        echo 'Element istnieje w tablicy!';
    }
    else
    {
    echo &#092;"nie ma takiego elementu\";
    }
    return $wiersz[0];
    }
?>
[PHP] pobierz, plaintext 

przy wywołaniu funkcji wywala mi taki błąd
Warning: array_keys() [function.array-keys]: The first argument should be an array in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 61

Warning: array_keys() [function.array-keys]: The first argument should be an array in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 61
nie ma takiego elementu6

tablica asocjacyjna dwuwymiarowa $news i $arts jest stworzona poza klasą i jest wczytana przed wywołaniem funkcji
podobnie jak zmienna $lang

Z góry dzięki za pomoc

Ten post edytował Zdunek 19.11.2008, 19:43:57

[Mize]

http://php.net.pl/manual/pl/function.array-keys.php - zwraca tablicę zawierającą klucze tablicy macierzystej.
A nie wartość BOOLEAN. Jeśli chcesz sprawdzić czy element znajduję się w tablicy użyj...

http://php.net.pl/manual/pl/function.in-array.php
http://php.net.pl/manual/pl/function.array-key-exists.php

Pozdrawiam.

Ten post edytował Mize 19.11.2008, 19:53:23

[Zdunek]

zmieniłem na
if (array_key_exists(7,$news[$lang]) || array_key_exists(7,$arts[$lang]))
i teraz wywala błąd
Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 61

Warning: array_key_exists() [function.array-key-exists]: The second argument should be either an array or an object in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 61

tablice $news i $arts są takie
$news = array(array());
$arts = array(array());
$news["pl"][0]=6;
$arts["pl"][0]=7;


Edit:
Wcześniej sobie robiłem w jednym pliku php bez klasy

[PHP] pobierz, plaintext 
<?php
$news = array(array());
$arts = array(array());
$lang=&#092;"en\";
$news[&#092;"pl\"][0]=6;
$news[&#092;"en\"][0]=7;
$arts[&#092;"en\"][0]=7;
 
    $array = Array('' => 'abc', 'b' => 'bcd');
   
    if ( array_keys($news[$lang],7) )
    {
        echo 'Element istnieje w tablicy!';
    }
    else
    {
    echo &#092;"nie ma takiego elementu w tablicy\";
    }
?>
[PHP] pobierz, plaintext 

i dobrze sprawdzało czy jest w tablicy asocjacyjnej, dwuwymiarowej, z indexem pierwszym o zmiennej $lang jest gdzieś 7
przy array_key_exists to już niestety nie działa

więc array_keys jest dobrą w tym przypadku metodą, lecz niestety jest coś nie tak z odwołaniem do zmiennych z poza klasy...

Edit2: przy użyciu in_array w jednym pliku php działa jak array_keys lecz przy klasie także wywala błąd
Warning: in_array() [function.in-array]: Wrong datatype for second argument in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 61

Ten post edytował Zdunek 20.11.2008, 13:41:08

[nevt]

twoje zmienne globalne $news, $arts i $lang nie są widoczne w ciele klasy. musisz je zadeklarować wewnątrz metody klauzulą global:

[PHP] pobierz, plaintext 
<?php
public function exist($id){
  global $news;
  global $arts;
  global $lang;
  $exist=strip_tags($id);
  ...
}
?>
[PHP] pobierz, plaintext 

ale to jest "obejście" nie w duchu OOP. najlepiej gdybyś te trzy zmienne (lub referencje do nich) przekazywał do konstruktora swojej klasy, a tam przepisał je do właściwości klasy - wtedy wszystkie metody będą miały do nich dostęp przez operator $this-> w zgodzie z zasadami OOP.

[Zdunek]

twoje zmienne globalne $news, $arts i $lang nie są widoczne w ciele klasy. musisz je zadeklarować wewnątrz metody klauzulą global:

[PHP] pobierz, plaintext 
<?php
public function exist($id){
  global $news;
  global $arts;
  global $lang;
  $exist=strip_tags($id);
  ...
}
?>
[PHP] pobierz, plaintext 

zrobiłem tak i wywala mi taki błąd:
Fatal error: Cannot use object of type news_arts as array in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 64
linia 64:
if( array_keys($news[$lang],7) || array_keys($arts[$lang],7))

ale to jest "obejście" nie w duchu OOP. najlepiej gdybyś te trzy zmienne (lub referencje do nich) przekazywał do konstruktora swojej klasy, a tam przepisał je do właściwości klasy - wtedy wszystkie metody będą miały do nich dostęp przez operator $this-> w zgodzie z zasadami OOP.

można jakiś link opisujący to ?

[nevt]

Fatal error: Cannot use object of type news_arts as array in C:\xampp\htdocs\test\poziomkaciecie\php\model\classes\class.np.php on line 64

czyli twoje zmienne $news i $arts nie są tablicami a instancjami klasy news_arts...
jednym słowem - pokazujesz nam jeden kod - a testujesz zupełnie inny i dziwisz się, że nasze podpowiedzi nie działają... pokaż cały kod - może wtedy da się wywnioskować coś rozsądnego.

co do OOP - widzę, że nie kojarzysz o co chodzi. na wrzuć w google 'podstawy OOP' i poczytaj coś na ten temat...

[Zdunek]

klasa news_arts

[PHP] pobierz, plaintext 
<?php
class news_arts{
public $all_id=array();
public $dane=array();
  public function __construct($host, $dbname, $user,  $pass = ''){ // 1
        $sql_conn = mysql_connect($host, $user, $pass)
        // w przypadku niepowodzenia poĹ‚Ä…czenia zakoĹ„cz aplikacjÄ™
        or die('Nie mogĹ‚em poĹ‚aczyć siÄ™ z  bazÄ… danych');
        mysql_select_db($dbname)or die("WystÄ…piĹ‚ bĹ‚Ä…d podczas wybierania bazy danych");
    }
  public function __destruct(){ // 2
        mysql_close();
         }
public function exist($id){
    global $news;
    global $arts;
    global $lang;
    $exist=strip_tags($id);
    $idzapytania = mysql_query("SELECT `forum_id` FROM `ipb_topics` WHERE `tid`='".$exist."'");
    $wiersz = mysql_fetch_row($idzapytania); 
    if( array_keys($news[$lang],7) || array_keys($arts[$lang],7))
    {
        echo 'Element istnieje w tablicy!';
    }
    else
    {
    echo "element nie istnieje w tablicy";
    }
    return $wiersz[0];
    }
 
}
?>
[PHP] pobierz, plaintext 

plik z tablicami

[PHP] pobierz, plaintext 
<?php
$news = array(array());
$arts = array(array());
$news["pl"][0]=6;
$arts["pl"][0]=7;
$news["en"][0]=6;
$arts["en"][0]=7;
$lang=$_COOKIE[lang];
?>
[PHP] pobierz, plaintext 

w indexie

[PHP] pobierz, plaintext 
<?php
$news=new news_arts($INFO['sql_host'],$INFO['sql_database'],$INFO['sql_user'],$INFO['sql_pass']);
echo $news->exist($_GET["id"]);
?>
[PHP] pobierz, plaintext 

wszystko jest na pewno połączone i wczytuje się w odpowiedniej kolejności

...

[MWL]

może poużywaj $this->news zamiast zmiennych global. Jesli chcesz ja wysłać dalej mozesz przecież przesłać to przez funkcję lub użyć singleatona

[Mize]

W tym kodzie nie ma co poprawiać, lepiej go napisać od nowa bo to kiepska struktura zamknięta w klasie.

[Zdunek]

no cóż każdy kiedyś zaczynał, z tego powodu piszę w przedszkolu