That seems to be correct use of next. However, $do_I_pick_up_request == 'Y' is actually testing whether $do_I_pick_up_request equals zero, so you probably want eq.