maybe not, maybe that is just not thinking what your PCs need.
but why would you give bunch of magic items no one will use?
just give a sack of gold/gems then. it's the same thing.
sure, now and then you can give item that has sense for NPC that is set against the party, but not throw those around.
1. You ran not making fun for players, as selling stuff is not fun 90% of the time.
2. You are wasting your time as you are finding items that are useless to the party.
it's a lose-lose situation.
And I have witnessed that, when the DMs will for D&D drained from his face.
we got 4 custom made magic items, and that took work to design them,
4 of us looked at them and decided that we will "vendor" them and buy a bunch of +1 generic items because those actually made out characters better(atleast from our perspective).
effort was made for those items and we treated them as truck load of scrap metal to be sold.