А зачем Вам поля REC_ID, да еще с одинаковыми именами для разных табл? Это я так понимаю ассоциативные сущности. Суррогатные ключи ведь могут быть оправданы только для соединений или если нет других идентификаторов.
Ну, или для единообразия, хотя в ассоциативных сущностях, если те больше ни для чего не предназначены они все равно лишние.
Вообще, стоит подумать о том, стоит ли давать одинаковые имена столбцов в разных таблах. Все-таки там разные данные, и это может усложнить написание запросов, например, естественное соединение будет сложней применять.
На предмет нарушения нормальных форм выше первой можно подозревать тока таблы с числом столбцов, превышающих 2. Ну тада надо выписывать зависимости атрибутов.
Лучше объявить все ключи. Так как счас нарисовано, например, могут быть магазины с одинаковым name даже в одном городе. И что еще хуже они могут повторяться, хотя и с разными ID.
Тоже про товары - можно много раз повторять один товар с одной ценой, но с разными ID. Тем более это относится к ассоциативным сущностям: какой смысл в повторе пар сурроготав, но с разными REC_ID?
Т.е. раз уж Вы беспокоитесь о нормальных формах, то следуе озаботиться тем полностью ли она характерихзует функциональные зависмости ПО.
И вообще - Нет ли потерь информации после такой декомпозиции, что Вы проделали, и является ли число отношений минимальным.