Как сократить и правильно написать данный код?

Question

[JTester]

Здравствуйте! Помогите пожалуйста сократить и правильно написать данный код!

$a = 1;
$b = 0;
if ($b === 0 && $a % 7 == 0) {
 // 
}
if ($b === 1 && $a % 3 == 0) {
 // 
}
if ($b === 2 && $a % 2 == 0) {
 // 
}
$a++;

В каждой проверки у меня выполняется одинаковые функции, просто меняются аргументы и все. В каждой функции я меняю $b
Правильный ли данный код?))))
$a - это время, он у меня выполняется в таймере

Comments

[Виталий Артемьев]

Можешь показать весь код?

[JTester]

Виталий Артемьев, ну у меня в каждой проверки вызывается функции set(63, 73); просто меняются в каждой проверки аргументы. Функцию set менять нельзя :)
Хочу оставить именно такой. В первой проверки я меняю переменную $b меняю на 1, во второй проверки меняю на 2 и все.

[rPman]

больше всего мне не понятно про $a, если это счетчик таймера (что бы это ни значило) то значит факт выполнения условия не гарантируется, мало того, как только хотя бы какое то условие не выполнится, b$ не увеличится и значит последующие проверки так же не будут выполнены

я правильно понимаю весь указанный код где то еще в цикл завернут и выполняется многократно? но в любом случае все равно бессмыслица

[JTester]

rPman, нет, цикла нету

[rPman]

JTester, про $a нужно больше информации, совершенно непонятно причем тут $a%7 и таймер

[JTester]

rPman, В общем, я хочу реализовать такую штуку. Кликаю на кнопку, она красится красной, потом через 7 сек она красится в жёлтый, потом через 3 сек она красится в зелёный и через 2 сек она снова становится такой, какой и была. Хочу на JS сделать, пример кидал по пхп

[rPman]

на лицо фатальное непонимание как это все работает
php - это приложение на бакэнде (сервер), его задача в общем случае отработать максимально быстро и выдать ответ (микросекунды) никаких sleep не допускается (да есть технологии где это допускается но они устарели и сейчас не о них)
javascript это приложение в браузере (на клиенте), оно работает асинхронно, при возникновении каких либо событий запускается соответствующий скрипт, отрабатывает и все, браузер ждет новых событий

события это - действие пользователя (кликнул например), ответ от сервера (загрузился скрипт/документ/сообщение от websocket) или таймер

соответственно чтобы реализовать твою задачу у тебя должно быть:
код обработки onclick кнопки - запускает таймер7 секунд смены цвета (функция setTimer), код смены цвета запускает следующий таймер на 3 секунды в другой цвет и так далее

будет многоэтажная конструкция но простая для понимания

[JTester]

rPman, кхм... А не проще запустить 1 таймер на 1 секунду и делать проверки как у меня?

[Кирилл Несмеянов]

> никаких sleep не допускается (да есть технологии где это допускается но они устарели и сейчас не о них)

rPman, есть подозрение, что эвентлуп никогда не устареет. А sleep (точнее usleep) - единственный способ разгрузить while(true) для делегирования задач шедулеру ОС (ну т.е. чтобы не грузить процессор на 146% и не создавать из процесса зомби). Или у вас есть какие-то другие варианты это сделать?

[rPman]

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

JTester, да, почему нет но код в вопросе все равно будет неправильный, нужно не остаток от деления брать а тупо сравнивать со значением времени (если >=) и отмашки у тебя будут 7, 7+3 и 7+3+2

[Кирилл Несмеянов]

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

Т.е. все аргументы, что я написал выше - вы просто проигнорировали?

Давайте тогда ссылочки покидаю:
1) React PHP
2) Amphp
3) Revolt PHP

А теперь, после того как ознакомились с кодом - можно ещё раз попробовать прочитать что я написал =)

Answer 1

[Ищю в поисковиках]

Ну, из того описания, что приложили вы - сокращать уже нечего.

Answer 2

[AlexRas]

function c($a, $b, $numbers) {
    foreach ($numbers as $key => $value) {
        if ($b === $key && $a % $value == 0) {
        // 
        }
    }
}

$a = 1;
$b = 0;
c($a, $b, [7, 3, 2]);
$a++;

Особо не сократил, но более правильный вариант, если у вас будет больше подобных if конструкций. А так сокращать там нечего.