Стиль кода жо.. плохой кароче. В принципе выше уже написали, я немного повторюсь по пунктам.
- Пишите camelCase стилем - это модно и красиво :) (а так же рекомендуется стандартами psr-1, что гораздо важнее).
- Фигурные скобки ставьте по psr-1 - код становится опрятнее.
- Не жалейте букв - пишите длинные понятные наименования методов и свойств.
- В объектах модели не нужно(читай нельзя) использовать куски представления(читай html и иже с ним). Модель(в данном случае товар - модель) хранит и обрабатывает чистые данные, отображение - не ее задача.
- Получение данных кодом и передача в объект - ваще кошмар. Объект сам должен знать как инициализировать себя, ему достаточно передать данные для инициализации(например уникальный id), запрос в базу в идеале должен происходить в отдельном объекте(например bd), который вызывается из модели, результат возвращаться в объект модели. Этими данными модель и инициализируется.