bad too long logic