[PHP] Problem z foreach Opcje

[Karoll]

Witam, mam problem z moim botem napisanym w php na wczesniejszym serwerze chodził bez problemu, aktualnie wywala błąd foreach :

PHP Warning: Invalid argument supplied for foreach() in /home/ts3/ts3/pokebot/core.php on line 38

38,39,40 linijka kodu

[PHP] pobierz, plaintext 
foreach ($users as $client) {
        if ($client['client_nickname'] != $config['bot']['name']) {
            $w = 0;
[PHP] pobierz, plaintext 

Wersja PHP na nowej maszynie (php -v)

PHP 5.4.4-14+deb7u8 (cli) (built: Feb 17 2014 09:18:47)
Copyright © 1997-2012 The PHP Group
Zend Engine v2.4.0, Copyright © 1998-2012 Zend Technologies
with the ionCube PHP Loader v4.2.2, Copyright © 2002-2012, by ionCube Ltd.

Wersja PHP na starej maszynie(php-v)

PHP 5.4.27-1~dotdeb.1 (cli) (built: Apr 6 2014 01:32:22)
Copyright © 1997-2014 The PHP Group
Zend Engine v2.4.0, Copyright © 1998-2014 Zend Technologies

Może to być wina innej wersji PHP?

Za rozwiąznie oczywiście ręka królewny i pół królestwa;D

Powód edycji: [Kshyhoo]: MANUAL służy do linkowania funkcji manula!

[nospor]

$users nie jest tablicą

[Karoll]

$users nie jest tablicą

Czyli mam błąd w kodzie?

[nospor]

jejkus.... Napisz skad sie bierze te $users to bedziemy ci mogli cos powiedziec... No chyba nie sadzisz, ze wywrozymy z fusow jak wyglada Twoj kod?
Napisz skad sie bierze, pokaz kod

[Karoll]

[PHP] pobierz, plaintext 
<?php
 
/*
* PokeBOT on TeamSpeak 3
* Przemysław Thel || admin@thel.pl
* If you wont more info check: <a href="http://www.thel.pl/" target="_blank">http://www.thel.pl/</a>
*/
 
date_default_timezone_set('Europe/Warsaw');
 
require_once 'configs/global_config.php';
require_once 'configs/lib/ts3admin.class.php';
require_once 'configs/lib/helper.php';
 
$query = new ts3admin($config['connect']['address'], $config['connect']['tcp']);
 
$cache = array();
$welcome = array();
 
if($query->getElement('success', $query->connect())) {
 
    $query->login($config['connect']['login'],$config['connect']['password']);
 
    $query->selectServer($config['connect']['udp']);
 
    $query->setName($config['bot']['name']);
 
	while (true) {
 
	        $core = $query->getElement('data',$query->whoAmI());
	        $query->clientMove($core['client_id'],$config['bot']['default_channel']);
 
        	$users = $query->getElement('data',$query->clientList('-groups -voice -away -times'));
 
        	$pokes = array();
        	$admins = array();
 
        foreach ($users as $client) {
		if ($client['client_nickname'] != $config['bot']['name']) {               
			$w = 0;
				if ($client['cid'] == $config['message']['channel_id'] && !in_array($client['clid'],$welcome)) {
                    			$info = $query->getElement('data',$query->clientInfo($client['clid']));
					$welcomeMsg = str_replace('{IP}', $info['connection_client_ip'], file_get_contents('messages/welcome.txt', true));
 
                    			$query->sendMessage(1, $client['clid'], $welcomeMsg);
                    			$welcome[time()-$w] = $client['clid'];
                    			$w++;
                }
 
$user_groups = explode(',',$client['client_servergroups']);
 
	if (isInGroup($user_groups,$admins_groups) && !in_array($client['clid'],$cache)) {
		$admins[$client['clid']] = $user_groups;
	}
 
	if (array_key_exists($client['cid'],$messages) && !isInGroup($user_groups, $admins_groups) && !$client['client_is_talker']) {
 
		$pokes[] = $client['cid'];
 
	if (!in_array($client['clid'],$cache)) {
		$query->sendMessage(1, $client['clid'], file_get_contents('messages/'.$messages[$client['cid']]['user_poke'],true));
		$cache[time()-1] = $client['clid'];
                    }
                }   
            }
        }
 
$counter = 0;
 
 
	foreach ($pokes as $poke) {
		foreach ($admins as $clid => $admin) {
			if (isInGroup($admin, $messages[$poke]['groups'])) {
				if (!in_array($clid,$cache)) {
					$cache[time()+$counter] = $clid;
					$query->clientPoke($clid, file_get_contents('messages/'.$messages[$poke]['admin_poke']));
					$counter++;
                    }
                }
            }
        }
 
 
	foreach ($cache as $time => $user) {
		if (time() - $time > $config['delete']['cache']) {
			unset($cache[$time]);
            }
        }
 
        if (date('H') == '23') {
		foreach ($welcome as $time => $user) {
			if (time() - $time > 60*60*24) { //1 day
				unset($welcome[$time]);
                }
            }
        }
 
	$admins = array();
	sleep(1);
    }
}
?>
[PHP] pobierz, plaintext 

[nospor]

$users = $query->getElement('data',$query->clientList('-groups -voice -away -times'));
Tu masz pobierane $users. Z jakiegos powodu $query->getElement źle zwraca dane. Dlaczego? nie wiem. Nie mam bladego pojecia co robi klasa ts3admin.

Wlacz wyswietlanie wszystkich bledow, to moze pojawią się jakies dodatkowe informacje. Tu masz napisane jak
Temat: Jak poprawnie zada pytanie

[Karoll]

Warning: Invalid argument supplied for foreach() in /home/ts3/ts3/pokebot/core.php on line 40
i tylko tylko pokazuje screen -r czyli w sumie nic się po wklejeniu na początek kodu tych paru linijek nie zmieniło oprócz tego że naturalnie linijka z błędem poleciała w dół...

[nospor]

A daj jeszcze
var_dump($users);
przed FOREACH i pokaz co wyswietli

Od biedy mozesz zrobic tak:
if (is_array($users))
foreach ($users as $client) {
.....

Dzieki temu nie bedziesz mial bledu. Ale czy to rozwiaze twoj problem?

[Karoll]

A daj jeszcze
var_dump($users);
przed FOREACH i pokaz co wyswietli

Od biedy mozesz zrobic tak:
if (is_array($users))
foreach ($users as $client) {
.....

Dzieki temu nie bedziesz mial bledu. Ale czy to rozwiaze twoj problem?

Warning: Invalid argument supplied for foreach() in /home/ts3/ts3/pokebot/core.php on line 41
bool(false)

Po zmianie na :

if (is_array($users))
foreach ($users as $client) {

Rzeczywiście nie ma błędu, ale bot w dalszym ciągu nie działa

[nospor]

No to niestety bardziej ci nie pomogę. Nie wiem co mają robić funkcje ktorych uzywasz ani jak działają. A ze nie pojawiły się żadne dodatkowe bledy, pomimo ze wlaczyles ich wyswietlanie, wiec pech....

[Karoll]

No to niestety bardziej ci nie pomogę. Nie wiem co mają robić funkcje ktorych uzywasz ani jak działają. A ze nie pojawiły się żadne dodatkowe bledy, pomimo ze wlaczyles ich wyswietlanie, wiec pech....

Ale na poprzedniej maszynie z inną wersją PHP bot działał bez problemu, może w tym tkwi problem?

[nospor]

Moze. ALe jak juz mowilem: skoro nie wyswietlają sie zadne dodatkowe bledy i nie mam bladego pojecia jak dzialają twoje funkcje, wiec nie wiem jakaz zmiana konfiguracji moze miec na to wplyw.

[Turson]

nospor widzę stracił cierpliwość i się nie dziwię.
Inna możliwość, to $clients jest tablicą, ale jest puste.

Jak już wspomniano

A daj jeszcze
var_dump($users);
przed FOREACH i pokaz co wyswietli

jeśli i tym razem zrobisz coś innego, to nie oczekuj więcej odpowiedzi

[nospor]

@Turson nie stracilem cierpliwosci. Poprostu naprawde nie wiem co robią te funkcje i naprawde nie jestem w stanie pomóc

Zas var_dump($users); autor juz zrobil i pokazal. Tez bys zobaczyl, jakbys nie byl slepy