Size: a a a

2020 January 15

11

12 123 in OctoberCMS
Привет, подскажите как правильно эта проверка должна выглядеть?
источник

Д

Дмитрий in OctoberCMS
MihailBishkek
походу придётся разбираться, в OneClick не получится ))))
переадресуй ответ запроса на ссылку для скачивания
источник

M

MihailBishkek in OctoberCMS
красивее for использовать наверное было бы
источник

IT

Ivan Trubchaninov in OctoberCMS
12 123
Привет, подскажите как правильно эта проверка должна выглядеть?
тебя интересует красота? или что-то ещё?
источник

ИГ

Иван Горохов in OctoberCMS
12 123
Привет, подскажите как правильно эта проверка должна выглядеть?
ну можно вот так например
$offer1 = (int)$data['offer1'] === 0;
$offer2 = (int)$data['offer2'] === 0;
$offer3 = (int)$data['offer3'] === 0;
$offer4 = (int)$data['offer4'] === 0;
источник

МА

Миржан Асымбаев in OctoberCMS
for var i = 1; i<=4; i++ {
    $offer_active[i] = (int) $data['price' . $i] == 0
}

что то в этом духе
источник

IT

Ivan Trubchaninov in OctoberCMS
Иван Горохов
ну можно вот так например
$offer1 = (int)$data['offer1'] === 0;
$offer2 = (int)$data['offer2'] === 0;
$offer3 = (int)$data['offer3'] === 0;
$offer4 = (int)$data['offer4'] === 0;
мне кажется, или что-то здесь не так)
источник

11

12 123 in OctoberCMS
Ivan Trubchaninov
тебя интересует красота? или что-то ещё?
Я вроде понимаю, что делаю не правильно, но не знаю как лучше сделать. Ладно у меня 4 переменные, а если 50, то будет огромная простыня
источник

11

12 123 in OctoberCMS
Миржан Асымбаев
for var i = 1; i<=4; i++ {
    $offer_active[i] = (int) $data['price' . $i] == 0
}

что то в этом духе
+ Это вроде то что нужно, попробую
источник

A

Axenia in OctoberCMS
qw1127 (0) увеличил карму Миржан (53.9)
источник

МА

Миржан Асымбаев in OctoberCMS
12 123
Я вроде понимаю, что делаю не правильно, но не знаю как лучше сделать. Ладно у меня 4 переменные, а если 50, то будет огромная простыня
массивы
источник

IT

Ivan Trubchaninov in OctoberCMS
12 123
Я вроде понимаю, что делаю не правильно, но не знаю как лучше сделать. Ладно у меня 4 переменные, а если 50, то будет огромная простыня
если условие не меняется, то можно мэпнуть или просто вывести в функцию.
источник

МА

Миржан Асымбаев in OctoberCMS
Миржан Асымбаев
for var i = 1; i<=4; i++ {
    $offer_active[i] = (int) $data['price' . $i] == 0
}

что то в этом духе
ошибка синтаксиса есть. но думаю разберетесь)) давно не писан на пхп уже отвыкаю от синтаксиса
источник

A

Alex in OctoberCMS
12 123
Привет, подскажите как правильно эта проверка должна выглядеть?
сделайте фунцию isOfferActive и вместо того чтобы использовать кучу переменных типа $isActiveOffer1, используйте функцию
if(isActiveOffer($data, 'offer1')) { 
 // @todo: Do some price changes
}
а ещё лучше найдите правильное место, которое подходит по смыслу, для того чтобы сделать эту функцию методом класса (может быть у вас есть модель offer? было бы не плохо поместить туда такой статический метод)
источник

M

MihailBishkek in OctoberCMS
Alex
сделайте фунцию isOfferActive и вместо того чтобы использовать кучу переменных типа $isActiveOffer1, используйте функцию
if(isActiveOffer($data, 'offer1')) { 
 // @todo: Do some price changes
}
а ещё лучше найдите правильное место, которое подходит по смыслу, для того чтобы сделать эту функцию методом класса (может быть у вас есть модель offer? было бы не плохо поместить туда такой статический метод)
о блин ) так пусть вообще атрибут сделать и всё ))))
источник

A

Alex in OctoberCMS
MihailBishkek
о блин ) так пусть вообще атрибут сделать и всё ))))
А у него видимо инстанса там нет(
источник

11

12 123 in OctoberCMS
Кого?)
источник

A

Alex in OctoberCMS
12 123
Кого?)
Если у вас есть модель оффер и ее экземпляр для каждого оффера, то для вас идеальный вариант тот который предложил Михаил
источник

M

MihailBishkek in OctoberCMS
public function getIsActiveAttribute()
{ if ($this->price == 0) { return false } else { return true } }
источник

v

vladimir in OctoberCMS
а где продолжение кода? зачем все эти переменные $offer_active{n} ?
источник