Где перезаписывается переменная?

Question

[Orbb]

Привет.

Есть такой код:

case 'order':
        $orderObj = new orderHandler($first_name, $last_name, $phone, $adress, $email, $message, $order);
        $msg = "Имя: {$orderObj->getFirstName()} <br>";
        $msg .= "Телефон: {$orderObj->getPhone()} <br>";
        $msg .= "Заказ: {$orderObj->getOrder(false)} <br>";
        $msg .= "Адрес: {$orderObj->getAdress()} <br>";
        break;
}

$arOrder = [
    'order' => $orderObj->getOrder(false),
];

И есть метод:

public function getOrder($toString){
            $order_string = $this->order;
        if($toString){
            $ar = array();
            foreach($order_string as $item){
                if($item == ''){
                    unset($item);
                }
                else{
                    $ar[] = $item;
                }
            }
            $order_string = implode(',', $ar);
            return $order_string;
        }else{
            return $this->order;
        }

    }

Суть вот в чем: если я один раз вызываю метод $orderObj->getOrder(false), то все работает как надо - на выходе имею либо почищенный массив, либо строчку, сколь угодно раз.
Но стоит вызвать где-нибудь еще, так начинается катавасия - то нотис вылезет, что пытаюсь массив в строку перевести, то наоборот.

Собственно, вывода в любом случае нужно будет 2 - одно для сообщения, а другое для отправки в панель обработки заказов, ткните меня носом в что я делаю не так

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

Comments

[padlyuck]

По приведенному куску невозможно определить где у вас проблема, кроме той, что вы не знаете о функции array_filter

[Orbb]

padlyuck: мне показалось этого кода будет достаточно, ну да ладно, проблему я решил позже. Действительно не знал, но почему для чистки массива лучше применять array_filter?

[padlyuck]

Руслан как минимум потому что меньше писанины, плюс сразу понятно что происходит, плюс array_filter это встроенная функция и как бонус ей вторым параметром можно передать callback для фильтрации

[Orbb]

padlyuck: спасибо за ответ. Пока действительно плохо знаю встроенные функции, делаю как умею.

Answer 1

[Максим Тимофеев]

У Вас есть объект orderHandler в нем видимо (вы не привели код всего класса) есть публичная переменная order. При первом вызове она массив, при втором строка. Что там конкретно - хз. Дайте весь код или при каждом обращении создавайте новый инстанс объекта.

Comments

[Orbb]

да, так и оказалось, глаза замылились и долго не мог понять почему перезаписывается)

Answer 2

[ThunderCat]

VAR_DUMP в руки и вперед, в данном участке кода происходит много чего "за кадром", пока не узнаете где поменялось значение тыкать пальцем в разные стороны бесполезно.