У вас 4 раза используется segm_data[0], и нигде не используется просто segm_data
может тогда лучше сделать так:
var segm_data = data.Flight[i].Segment[0];
аналогично с flight_start_time и flight_end_time, к ним везде применяется time2minutes()
var flight_start_time = time2minutes(segm_data.Departure.attributes.Time);
А на какие маленькие функции вы бы хотели разбить этот кусок?