Постоянное проговаривание свойства.

[Denis_k]

писал по памяти, а её нету(

$params['NEW_VALUE'] , $params['OLD_VALUE']

отлично, спасибо, такая конструкция работает

Код: Выделить всё

If($params['NEW_VALUE']<>$params['OLD_VALUE'])
{
$open=gg("OpenClose01.status");
if ($open==1) 
{say ("Входная дверь закрыта",2);
}
else 
{say ("Входная дверь открыта",2);
}
}

[xor]

писал по памяти, а её нету(

$params['NEW_VALUE'] , $params['OLD_VALUE']

отлично, спасибо, такая конструкция работает

Код: Выделить всё

If($params['NEW_VALUE']<>$params['OLD_VALUE'])
{
$open=gg("OpenClose01.status");
if ($open==1) 
{say ("Входная дверь закрыта",2);
}
else 
{say ("Входная дверь открыта",2);
}
}

ну, можно подсократить, тут $params['NEW_VALUE'] у вас и есть новое значение OpenClose01.status

Код: Выделить всё

If($params['NEW_VALUE']<>$params['OLD_VALUE'])
{
if ($params['NEW_VALUE']==1) 
{say ("Входная дверь закрыта",2);
}
else 
{say ("Входная дверь открыта",2);
}
}

[madbob]

Скорее всего при while у вас "набросало" сообщений в очередь и будет проговаривать до второго пришествия (хотя сам метод фактически закончил выполнение).
Попробуйте, как правильно подсказали, добавить в код строчку с SetTimeOut, скажем, с 10-секундной задержкой выполнения).

Код: Выделить всё

If  ($this->getProperty('status')) {
 say("пожар в гараже",3);
 SetTimeOut("MyTimer_33","$this->callMethod('ИМЯ_ТЕКУЩЕГО_МЕТОДА');",10);
}

Все равно проговаривает 1 раз, метод не перезапусается по окончании таймера.

[Chainik]

...

Код: Выделить всё

If  ($this->getProperty('status')) {
 say("пожар в гараже",3);
 SetTimeOut("MyTimer_33","$this->callMethod('ИМЯ_ТЕКУЩЕГО_МЕТОДА');",10);
}

Все равно проговаривает 1 раз, метод не перезапусается по окончании таймера.

Значит, наверняка проблема с правильностью синтаксиса в том коде, что я предложил. Попробуйте указать имя Объекта.Метода в явном виде.

Код: Выделить всё

 SetTimeOut("MyTimer_33","callMethod('Relay19.statusUpdated');",10);

Если вам все же нужен классовый метод, можно написать так:

Код: Выделить всё

$ot=$this->object_title;
SetTimeOut("MyTimer_33","callMethod('".$ot.".statusUpdated');",10);

[Logrus]

SDevices_statusUpdated не предназначен для такого, есть SDevices_logicAction

[madbob]

Код: Выделить всё

if ($this->getProperty('status')) {
 say("пожар в гараже",3);
 SetTimeOut("Timer_fire_garage","callMethod('Relay19.statusUpdated');",10);
}

Ура! Так заработало в statusUpdated. Даже теща прибежала ))

SDevices_statusUpdated не предназначен для такого, есть SDevices_logicAction

Почему? Чем грозит использование этого кода в методе statusUpdate объекта в Sdevices ?
Спасибо всем за помощь!

[Logrus]

Код: Выделить всё

if ($this->getProperty('status')) {
 say("пожар в гараже",3);
 SetTimeOut("Timer_fire_garage","callMethod('Relay19.statusUpdated');",10);
}

Ура! Так заработало в statusUpdated. Даже теща прибежала ))

SDevices_statusUpdated не предназначен для такого, есть SDevices_logicAction

Почему? Чем грозит использование этого кода в методе statusUpdate объекта в Sdevices ?
Спасибо всем за помощь!

в инжекции есть много лишнего

[madbob]

в инжекции есть много лишнего

Мне пока не понять катастрофы, надеюсь, не будет?

[Logrus]

не должно, но лишняя не нужная нагрузка

[xor]

в инжекции есть много лишнего

Мне пока не понять катастрофы, надеюсь, не будет?

я б вообще вынес крик о пожаре в отдельный скрипт, допустим, FireAlarm

Код: Выделить всё

//script FireAlarm
if (gg('Object.status')) { //кричать, пока свойство=1
 say("пожар в гараже",3);
 SetTimeOut("Timer_fire_garage","runscript('FireAlarm');",10);
}

а инициировал бы его один раз вызовом из откуда он там вызывается

Код: Выделить всё

//say("пожар в гараже",3);
 runscript('FireAlarm');