Wouldn't be better cache this as an attribute?