better error message
use constants