itfuzzy Вы намекаете на то что нужно смотреть в сторону EAV?LSV ВСЕГДА намекает на EAV. Карфаген должен быть разрушен (с)

Кот Матроскин Если у Вас есть четкая зависимость Категория-> Группа, то поле ID_group не нужно в Product, поскольку оно полностью определяется полем ID_category. Или топикстартер применяет универсальный справочник

Под двумя группами вы называете ID_group и ID_category, намекая что где-то существует ID_category2, ID_category3 и т.д? Не вижу ничего криминального.

itfuzzy правильна ли такая структураВопрос не имеет смысла, ибо информации для ответа недостаточно, в результате каждый из отвечающих ДОДУМЫВАЕТ детали и выдает оптимальное для своей ПРИДУМАННОЙ ситуации решение.
Более продуктивным будет набор возможных решений со списком достоинств/недостатков каждого, который вы можете применить к своей ситуации.

itfuzzy перенес поле expiry_date в табл характеристик т.к некоторые группы не имеют срока годности Занесите в общую таблицу предка все поля по которым идет поиск, то бишь, даже если товар данной группы (кстати подумайте над другим названием) срока годности не имеет, но поиск по сроку годности в системе есть (в базовой таблице будет null для таких товаров).
Это не рекомендация - это информация к размышлению.

itfuzzy не будет ли наличия NULL "плохим тоном"Суббота для человека, а не человек для субботы (с)
Любые правила нарушать МОЖНО, но надо четко понимать что тебе за это будет, поэтому новичкам (не понимающим) их нарушать запрещают.
Соглашусь со Злым бобром - любая готовая реализация будет лучше самопала, но это не означает что самопал надо перестать ваять - его (самопал) просто не надо пускать в продакшн.

LSV зы: А ЕАВ бы хорошо подошел. :)Стабильность - признак мастерства