конструктор в любом языке предназначен по идее для инициализации каких-либо данных данных, если данных для инициализации много то и пиши туда много кода, если нет то нет.
в пределах группы для радио кнопок должны быть уникальные имена jshop_attr_id[1], jshop_attr_id[2] и т.д. а у вас все одинаковые, логично что выбирается только последнее.
создать тег img (сделать скрытым display:none) в его аттрибут src добавить ссылку на картинку, повесить событие load, после загрузки картинки выполниться функция прописанная в load, там выставить свойство display: в inline.