Правильно ли я создал класс?
Бегло посмотрел код и что бросилось в глаза - это названия методов, пишите в стиле camelCase, а не almost_Camel_Case
insertCSS (можно и Css как вам удобнее)
appendContent
etc.Меня особенно волнует правильно ли я наполняю переменные для return-на - лучше минимизировать return , чтоб он возвращал какой-нибудь примитивный тип(boolean, array, int etc.), но не HTML текст(прочтите про MVC).
И зачем префиксы my*? без не лучше?set_h1- это не айс, а что если потом захотите h1 поменять на div#title? абстрагируйтесь, пишите setTitle или как-то по другому.
В остальном - учитесь и практикуйтесь, смотрите код на github и сравнивайте, а то ваш вопрос какой-то некорректный что ли
Стиль кода жо.. плохой кароче. В принципе выше уже написали, я немного повторюсь по пунктам.
- Пишите camelCase стилем - это модно и красиво :) (а так же рекомендуется стандартами psr-1, что гораздо важнее).
- Фигурные скобки ставьте по psr-1 - код становится опрятнее.
- Не жалейте букв - пишите длинные понятные наименования методов и свойств.
- В объектах модели не нужно(читай нельзя) использовать куски представления(читай html и иже с ним). Модель(в данном случае товар - модель) хранит и обрабатывает чистые данные, отображение - не ее задача.
- Получение данных кодом и передача в объект - ваще кошмар. Объект сам должен знать как инициализировать себя, ему достаточно передать данные для инициализации(например уникальный id), запрос в базу в идеале должен происходить в отдельном объекте(например bd), который вызывается из модели, результат возвращаться в объект модели. Этими данными модель и инициализируется.