This is a good idea much better than the current system based on RNG,
The only flaw is that since there are many enchantments, it would create a situation where every time you want to enchant something you find yourself faced with a very long list of options, and even if I prefer this system to the current system based on RNG, this system is not an intuitive and ergonomic experience for the player.
I also wrote a topic where I propose solutions to the problems of the enchantment system, Take a look if u are interested: The RNG Problem, Enchantment System Vs Gem System (Refined)