php - Jak naprawić ten niedozwolony błąd typu przesunięcia

90

Dostaję

niedozwolony typ offsetu

błąd dla każdej iteracji tego kodu. Oto kod:

$s = array();
for($i = 0; $i < 20; $i++){
    $source = $xml->entry[$i]->source;
    $s[$source] += 1;    
}

print_r($s)

php arrays

— Steven
źródło

10

Ostrzeżenie: prawie wszystkie odpowiedzi (oprócz zombata) zakładają, że $sourcejest to przykład SimpleXMLi dostarczają informacji, które dotyczą tylko tej konkretnej sytuacji. Chociaż ostatecznie tak się stało, w pytaniu tego nie stwierdzono, a ktokolwiek przyjeżdża tutaj w celach informacyjnych, powinien to wziąć pod uwagę.

— Álvaro González

157

Niedozwolone błędy typu przesunięcia występują podczas próby uzyskania dostępu do indeksu tablicy przy użyciu obiektu lub tablicy jako klucza indeksu.

Przykład:

$x = new stdClass();
$arr = array();
echo $arr[$x];
//illegal offset type

Twoja $xmltablica zawiera obiekt lub tablicę pod adresem $xml->entry[$i]->sourcedla pewnej wartości $i, a kiedy spróbujesz użyć tego jako klucza indeksu dla $s, otrzymasz to ostrzeżenie. Musisz upewnić się, że $xmlzawiera to, czego chcesz i że uzyskujesz do niego dostęp poprawnie.

— zombat
źródło

źródło zawiera html, czy ta klasa jest obiektem?

— Steven

Czy utworzyłeś $xmlzmienną używając jakiegoś parsera XML? simple_xml czy DOMDocument? W takim przypadku jest prawdopodobne, że węzeł źródłowy jest w rzeczywistości jakimś rodzajem obiektu elementu dom.

— zombat

używam simplexml_load_string. To pomaga?

— Steven

Twój kod HTML mógł zostać przeanalizowany jako XML i prawdopodobnie wszystkie tagi stały się węzłami. Na przykład, jeśli masz fragment kodu HTML „<div> Cześć </div>” jako właściwość źródłową, prawdopodobnie masz coś takiego jak $xml->entry[$i]->source->div. Jeśli chcesz przeanalizować HTML do struktury DOM, DomDocumentma loadHTML()funkcję, która obsługuje HTML znacznie lepiej niż SimpleXML. Sprawdź php.net/manual/en/domdocument.loadhtml.php

— zombat

Dziękuję za to. Użyłem str_replace, aby usunąć HTML i działa.

— Steven

26

Użyj trim($source)przed $s[$source].

— Zafer
źródło

3

Myślę, że to jest właściwa odpowiedź na to pytanie, a nie zombat

— Pmpr

5

Przycinanie obiektu jest po prostu nieoczywistym sposobem rzutowania go na łańcuch (najprostszy byłby (string)$source), a wyniki zależą całkowicie od jego implementacji __toString () . Działa, jeśli masz SimpleXMLobiekt (coś, co najwyraźniej wszyscy przyjęli, ale nigdy tak naprawdę nie zostało wyrażone w pytaniu).

— Álvaro González

Jeśli ten problem jest związany z implementacją __toString (), wywołanie trim () nie jest czystym rozwiązaniem. To wprowadza zamieszanie.

— Čamo

3

check $ xml-> pozycja [$ i] istnieje i jest obiektem przed próbą uzyskania jego właściwości

 if(isset($xml->entry[$i]) && is_object($xml->entry[$i])){
   $source = $xml->entry[$i]->source;          
   $s[$source] += 1;
 }

lub $ source może nie być prawidłowym przesunięciem tablicy, ale tablicą, obiektem, zasobem lub prawdopodobnie wartością null

— brian_d
źródło

2

Jedyna prawidłowa odpowiedź. Musisz sprawdzić istnienie elementu w tablicy. Jeśli nie istnieje, nie możesz uzyskać dostępu do jego właściwości.

— RWC

0

Prawdopodobnie w Twoim xml jest mniej niż 20 wpisów.

zmień kod na ten

for ($i=0;$i< sizeof($xml->entry); $i++)
...

— Byron Whitlock
źródło

4

Niezdefiniowany indeks całkowity nie generuje ostrzeżenia „Niedozwolone przesunięcie”, zamiast tego otrzymasz „Niezdefiniowany indeks” E_NOTICE.

— zombat

0

Miałem podobny problem. Ponieważ otrzymałem Character od mojego dziecka XML, musiałem najpierw przekonwertować go na String (lub Integer, jeśli się spodziewasz). Poniżej pokazano, jak rozwiązałem problem.

foreach($xml->children() as $newInstr){
        $iInstrument = new Instrument($newInstr['id'],$newInstr->Naam,$newInstr->Key);
        $arrInstruments->offsetSet((String)$iInstrument->getID(), $iInstrument);
    }

— user8387356
źródło