Как правильно сделать Callback в методе класса своего модуля для 1С-Битрикс?

Question

[Тимур Калимуллин]

Добрый день,

для модуля, которым пользуются пользователи из marketplace (1С-Битрикс) необходимо добавить возможность изменять в случае необходимости результаты работы методов класса.

Для реализации callback воспользовался функцией call_user_func. Возможно в 1С-Битрикс уже есть свое Event'ы, которые можно зарегистрировать и присвоить к своему модулю?

На данный момент код имеет следующую структуру:

class myClass {
	public function import()
	{
		$arResult = array();

		$arData = array(
			'result1',
			'result2',
			'result3',
		);

		foreach($arData AS $v)
		{
			$arResult[] = 'Строка: '.$v;
		}

		if(function_exists('call_import'))
			$arResult = call_user_func('call_import', $arResult);
		
		return $arResult;
	}
}

function call_import($arData = array())
{
	$arResult = array();
	foreach($arData AS $v)
	{
		$arResult[] = $v.' что то изменили';
	}
	return $arResult;
}

данный способ позволит пользователю создать свою функцию call_import и изменить результат работы метода import.

На сколько данное решение верно?

Спасибо.

Answer 1

[Алексей Уколов]

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

Comments

[Тимур Калимуллин]

Алексей, спасибо, верно понял?

class myClass {

	static $module_id = "modulename";

	public function import()
	{
		$arResult = array();

		$arData = array(
			'result1',
			'result2',
			'result3',
		);

		foreach($arData AS $v)
		{
			$arResult[] = 'Строка: '.$v;
		}

		$rsHandlers = GetModuleEvents(self::$module_id, "MyNameEvent");
		while($arHandler = $rsHandlers->Fetch())
		{
			$arResult = ExecuteModuleEvent($arHandler, $arResult));
		}

		return $arResult;
	}
}

AddEventHandler("modulename", "MyNameEvent", "MyFnCall");
function MyFnCall($arData = array())
{
	$arResult = array();
	foreach($arData AS $v)
	{
		$arResult[] = $v.' что то изменили';
	}
	return $arResult;
}

[Алексей Уколов]

Тимур Калимуллин: мне навскидку сложно сказать все ли правильно. Вроде, похоже на правду.
Если все работает, значит, вы все верно сделали :) если нет - обратно в документацию.

Answer 2

[Андрей Буров]

class myClass {
  public function import($callback = null)
  {
    $arResult = array();

    $arData = array(
      'result1',
      'result2',
      'result3',
    );

    foreach($arData AS $v)
    {
      $arResult[] = 'Строка: '.$v;
    }

    if (is_callable($callback)) {
        call_user_func($callback, $arResult);
    }
    
    return $arResult;
  }
}

Comments

[Тимур Калимуллин]

Проблема в том, что сам метод import пользователи вручную вызывать не могут, более того, он вызывается по крону.

[Андрей Буров]

Тимур Калимуллин: Ну жестко привязываться к имени функции как-то не очень.

[Тимур Калимуллин]

это точно.