Не работает код, когда поместил его в функцию php. Почему так?

Question

[TostPOST]

Делаю бота администратора бесед. Для того, чтобы проверить пользователя на админство написал код, но так как проверка будет использоваться многократно в коде сделал функцию и залил туда этот код. Сам по себе этот код работает, я вот в функции - нет.

function check($a,$b,$c){
	$info = $vk->request('messages.getConversationMembers',['peer_id' => $a, 'group_id' => $b]);
	$count = $info['count'] - count($info['count']) - 1;
	for ($i=0; $i < $count; $i++) { 
		$idd = $info['items'][$i]['member_id'];
		if ($idd !== $c) continue;
		if ($idd == $c) {
			$is = $info['items'][$i]['is_admin'];
			if($is == true){
				$ist = true;
			}else{
				$ist = false;
			}
			break;
		}

	}
	return $ist;
	
}

Comments

[edward_freedom]

$count = $info['count'] - count($info['count']) - 1;

Что тут происходит? Ты в функцию count, которая принимает массив, передаешь число

[TostPOST]

Я ошибся, там должно быть groups

[edward_freedom]

TostPOST, ты мне ответил?

Answer 1

[green_goo]

Используется неопределенная переменная $vk

Comments

[TostPOST]

Блин, я же читал документацию, специально все переменные менял, но про одну забыл

[TostPOST]

Иван, всм, sublime ниче не показывал, как б в просто ее из вида упустил. Бывает.

[JhaoDa]

TostPOST, мыши плакали, кололись, но продолжали жрать кактус. PHPStorm надо юзать.

[edward_freedom]

TostPOST, сублайн для верстки предназначен, используй пхпшторм или vs code

[Антон Р.]

edward_freedom, хех, я вообще сижу в браузерной ide от таймвеба, которая отличается от Блокнота только подсветкой синтаксиса ))

[Александр Маргинал]

edward_freedom, он даже для верстки не предохранитель, нужен только чтобы через судо открывать файлы

[TostPOST]

JhaoDa, ещё бы юзать пхпсторм на ноуте 2к10, у меня комп сломался, так что пока приходится работать на сублайме. Так то сублайм очень даже неплох.

[Konata Izumi]

TostPOST,

сублайм очень даже неплох

А ошибку подсветить не смог...

Answer 2

[Randewoo]

yette? gota ahhr!

function check($a,$b,$c){
  global $vk;
  $info = $vk->request('messages.getConversationMembers',['peer_id' => $a, 'group_id' => $b]);
  $count = $info['count'] - count($info['count']) - 1;
  for ($i=0; $i < $count; $i++) { 
    $idd = $info['items'][$i]['member_id'];
    if ($idd !== $c) continue;
    if ($idd == $c) {
      $is = $info['items'][$i]['is_admin'];
      if($is == true){
        $ist = true;
      }else{
        $ist = false;
      }
      break;
    }

  }
  return $ist;
  
}

Comments

[edward_freedom]

использование глобальной переменной еще больше зашквар

[Пётр Грибанов]

edward_freedom, тогда передавайте её в аргументах. У вас не существует переменная $vk в теле функции

[edward_freedom]

Пётр Грибанов, не я автор, просто не нужно советовать дичь какую то. Можно же создать класс, определить в нем этот вк и в конструкторе инициализировать

Answer 3

[Fosteev]

В переменой $ist нет смысла

Answer 4

[dimass55]

Дружище, какую ошибку выдает? И, действительно, передай объект vk в параметр функции))

Comments

[TostPOST]

Та я уже все сделал, там немножко неправильно в for было и нужно было передавать сразу info.