I like the general idea of not hiding affixes, but I think it would only be good if affixes are made way more frequent. With their current frequency, I think most of the time the effect would be that we would never find affixed weapons, because we would silently discard them without even trying the weapon. There's no way in hell I'm going to actually try 50 spears (not try as in wield and unwield, but actually taking the time to kill stuff with them) to see if one of them is of lightning, vampiric or something like that.
An intermediate solution is what some games like Brogue or Pixel Dungeon do: they tell you that the weapon has some special property (called runes in Brogue and enchantments in Pixel Dungeon IIRC), but they don't tell you what it is until you have ID'd it. So we could see things like "an enchanted mace (+1, 1d6)", hinting the player that they might want to keep that mace because it's somewhat special, and then actually IDing it would reveal whether it's of fire, brutal or whatever.
Optional twist: you need Pe (or Metallurgy/Appraising could help) to tell enchanted from non-enchanted weapons.