Well, I can't really see this being fast. You're getting the items from a place where, as far as I know, is not stored in memory, then loading the data from an array, which isn't exactly efficient.
There must be something I don't know, because I never even considered using SQL for an item system for this reason. Please correct me if I'm wrong
.