Возвращаемое значение (вполне) базовой функции "undefined"?

Question

Я изо всех сил пытаюсь вернуть одну из моих функций JavaScript. Пожалуйста, соблюдайте следующий код:

function GW2API_getEventInfo(p_eventid) { 
    console.log("Getting EventInfo for event " + p_eventid); 
    $.each(arrEvents, function(i, eventItem) { 
     $.each(eventItem, function(j, eventInfo) { 
      if (eventInfo.event_id == p_eventid) { 
       console.log(GW2API_getEventName(p_eventid) + " - " + eventInfo.state); 
       return { 
        'name': GW2API_getEventName(p_eventid), 
        'state': eventInfo.state 
       }; 
      } 
     }); 
    }); 
} 

Теперь я хотел бы вернуть объект с двумя свойствами: «имя» и «состояние». Однако, независимо от того, что я добавляю под console.log, моя функция ничего не возвращает (хотя данные найдены, поэтому консоль регистрирует).

currEvent = GW2API_getEventInfo(GW2API_events_ShadowBehemoth[i]); alert(currEvent) -> дает "неопределенными"

Answer 1

Это должно - вы ничего не возвращаются.

Вы возвращаете что-то внутри анонимной функции второго $.each, но вы ничего не возвращаете в функции GW2API_getEventInfo.

Может быть, это то, что вы имели в виду:

function GW2API_getEventInfo(p_eventid) { 
    var result = null; 
    console.log("Getting EventInfo for event " + p_eventid); 
    $.each(arrEvents, function(i, eventItem) { 
     $.each(eventItem, function(j, eventInfo) { 
      if (eventInfo.event_id == p_eventid) { 
       console.log(GW2API_getEventName(p_eventid) + " - " + eventInfo.state); 
       result = { 
        'name': GW2API_getEventName(p_eventid), 
        'state': eventInfo.state 
       }; 
       return; //Out of the inner $.each 
      } 
     }); 
     if (result !== null) { 
      return; //Out of the outer $.each 
     } 
    }); 
    return result; //Return the value in the GW2API_getEventInfo function 
} 

Answer 2

Использование .each() означает, что вы создаете внутреннюю функцию закрытия. Как бы то ни было, вы возвращаетесь из функции закрытия, но не возвращаетесь из своей основной функции.

Возможно, вам лучше не использовать jQuery .each() здесь и просто использовать стандартный цикл Javascript for().

function GW2API_getEventInfo(p_eventid) { 
    for(var i=0; i<=arrEvents.length; i++) { 
     var eventItem = arrEvents[i]; 
     for(var j=0; j<=eventItem.length; j++) { 
      var eventInfo = eventItem[j]; 
      if (eventInfo.event_id == p_eventid) { 
       return { 
        'name': GW2API_getEventName(p_eventid), 
        'state': eventInfo.state 
       }; 
      } 
     }); 
    }); 
} 

Это позволяет избежать использования функций закрытия, что позволяет избежать путаницы над возвращаемыми значениями.

Вероятно, он будет работать быстрее, чем версия jQuery.

(заметьте, я не знаю структуру данных здесь, я предположил, что это arrEvents и eventItem являются массивы Если они объекты, вам нужно будет использовать цикл в for(..in..) стиль вместо этого, но принцип. то же самое)