should you check for negative values?