помогите с кодом

[Ged]

Так и пришлось смотреть, как же у меня сделано...

Всю нашу магию (т.е. то что мы должны сделать) заносим в метод refresh.
Привязываем чтобы при изменении статуса вызывался метод refresh.

После этого, все управление объектом и в скриптах и в меню, сводится к изменению свойства статус.
В скриптах соответственно использование

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

getGlobal('Svet_3_spaln.status',1);

или

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

getGlobal('Svet_3_spaln.status',0);

Можно и через методы

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

callMethod("Svet_3_spaln.turnOn");

Работать будет в любом случае.

Соответственно, если вы прикрутили статус в переменную с названием "sostojanie", то через нее.
getGlobal('Svet_3_spaln.sostojanie');

Метод switch, служит только для переключение через методы turnOn и turnOff.
А иначе, действительно. При изменении статуса, запускаем метод и изменяем статус. Соответственно, кони бегают по кругу.

4 метода:
refresh - Получаем статус и вызываем магию. Получается, что у наследников нужно делать свою обработку этого блока

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 0);
} else {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 1);
}

switch - В зависимости от статуса вызываем противоположный метод turnOff/turnOn

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

$status=$this->getProperty("status");
if ($status) {
 $this->callMethod('turnOff');
} else {
 $this->callMethod('turnOn');
}

turnOff - Устанавливаем статус и вызываем метод refresh

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

$this->setProperty("status",0);
$this->callMethod("refresh");

turnOn - Устанавливаем статус и вызываем метод refresh

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

$this->setProperty("status",1);
$this->callMethod("refresh");

P.S. чой-то с постами я накосячил....

[DimSun75]

Можно и так. Только во втором getURL st=0 и в конце не 1, а 0. Чтобы каждый раз не переписывать метод refresh, добавляем в класс свойства IP и pin. Метод refresh такой:

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=1&pin='.$this -> getProperty("pin"), 0);
} else {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=0&pin='.$this -> getProperty("pin"), 0);
}

Теперь в каждом новом объекте нужно прописать адрес в IP и номер gpio в pin.

[Ged]

Можно и так. Только во втором getURL st=0 и в конце не 1, а 0. Чтобы каждый раз не переписывать метод refresh, добавляем в класс свойства IP и pin. Метод refresh такой:

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=1&pin='.$this -> getProperty("pin"), 0);
} else {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=0&pin='.$this -> getProperty("pin"), 0);
}
 

Теперь в каждом новом объекте нужно прописать адрес в IP и номер gpio в pin.

Уху. Чой-то как обычно, я написал много и сумбурно. Попробую коротко.

Вся наша чудо магия только в методе refresh, который привязываем к изменению свойства статус. Стандартные метода не трогаем.
Управление производим через изменение статуса в нужное состояние.

P.S. Метод switch полезен в скриптах помигай лампочкой.

[vitar78]

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 0);
} else {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 1);
} 
 

Я извеняюсь но не является ли ошибкой в обоих запросах gpio?st=1 и не надо ли в нижнем заменить на gpio?st=0

[Ged]

Я извеняюсь но не является ли ошибкой в обоих запросах gpio?st=1 и не надо ли в нижнем заменить на gpio?st=0

Естественно, я перепутал и поменял не то.
Вместо gpio?st=1, я поменял зачем-то в конце...

Ну и краше конечно будет:

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

getURL('http://192.168.2.10/gpio?st='.$state.'&pin=12', 0);

И можно еще раз прочитать комментарий DimSun75 viewtopic.php?f=4&t=3769&start=10#p47112

[DimSun75]

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 0);
} else {
 getURL('http://192.168.2.10/gpio?st=1&pin=12', 1);
} 

Я извеняюсь но не является ли ошибкой в обоих запросах gpio?st=1 и не надо ли в нижнем заменить на gpio?st=0

Да, там ошибка, второй getURL должен быть:

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

getURL('http://192.168.2.10/gpio?st=0&pin=12', 0);
 

Ну или как я писал

[vitar78]

Можно и так. Только во втором getURL st=0 и в конце не 1, а 0. Чтобы каждый раз не переписывать метод refresh, добавляем в класс свойства IP и pin. Метод refresh такой:

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

$status=$this->getProperty("status");
if ($status) {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=1&pin='.$this -> getProperty("pin"), 0);
} else {
 getURL('http://'.$this -> getProperty("IP").'/gpio?st=0&pin='.$this -> getProperty("pin"), 0);
}

Теперь в каждом новом объекте нужно прописать адрес в IP и номер gpio в pin.

Уху. Чой-то как обычно, я написал много и сумбурно. Попробую коротко.

Вся наша чудо магия только в методе refresh, который привязываем к изменению свойства статус. Стандартные метода не трогаем.
Управление производим через изменение статуса в нужное состояние.

P.S. Метод switch полезен в скриптах помигай лампочкой.

а откуда брать свойство pin, esp его не передаёт, или указывать вручную в свойствах?

[DimSun75]

а откуда брать свойство pin, esp его не передаёт, или указывать вручную в свойствах?

В ручную конечно.

[vitar78]

а откуда брать свойство pin, esp его не передаёт, или указывать вручную в свойствах?

В ручную конечно.

а для чего тогда указывать в свойствах если можно прописать сразу в коде ?, или есть какой то практический смысл в этом

[DimSun75]

а откуда брать свойство pin, esp его не передаёт, или указывать вручную в свойствах?

В ручную конечно.

а для чего тогда указывать в свойствах если можно прописать сразу в коде ?, или есть какой то практический смысл в этом

Если один такой объект, то смысла нет. Если несколько, то тогда для каждого объекта нужно править код. В моем случае, создаете новый объект и просто прописываете свойства