Javascript object literal: почему я не могу это сделать?

Question

У меня есть следующий (упрощенный) литерал объекта. Метод значков использует закрытие, чтобы скрыть переменную значков, которую я хотел бы иметь в качестве ассоциативного массива для последующих поисков.

var MapListings = { 
    icons: function() { 
     var allIcons = [] ; 

     return { 
      add: function (iconType, iconImage) { 
       var icon = new GIcon(MapListings.baseIcon); 
       icon.image = iconImage; 
       allIcons[iconType] = icon; // fails, but this is what I want 
       // allIcons.push(icon); // works, but this is not what I want 
      }, 
      get: function() { 
       return allIcons; 
      } 
     }; 

    }() 
} 

добавить детали к иконам объекта следующим образом:

MapListings.icons.add("c7", "/images/maps/blue.png"); 
MapListings.icons.add("c8", "/images/maps/red.png"); 

Следующая не работает:

allIcons[iconType] = icon; 

Но это делает:

allIcons.push(icon); 

За пределами закрытия работает стиль ассоциативного массива отлично, возможно, есть конфликт с jQuery? Ошибка, которую я получаю в firebug a не определена выглядит из библиотеки. Я бы хотел поддерживать стиль ассоциативного массива.

Любые идеи?

Update

Похоже, что этот конфликт происходит из карт Google. Странно, не уверен в этом.

Dumbass Update

Часть моего литерал объекта, возвращающего база GIcon() объект не возвращается объект вообще. Таким образом, объект не имел правильных свойств.

baseIcon: function() { 
    var base = new GIcon(); 
    base.shadow = '/images/maps/shadow.png'; 
    base.iconSize = new GSize(12, 20); 
    base.shadowSize = new GSize(22, 20); 
    base.iconAnchor = new GPoint(6, 20); 
    base.infoWindowAnchor = new GPoint(5, 1); 
    return base; 
} 

И MapListings.baseIcon не совпадает с MapListings.baseIcon()! D'о

Answer 1

если вы хотите таблицу поиска, просто сделать var allIcons = {}

EDIT: Хотя технически она должна работать в любом случае, так как массив является объектом. Вы уверены, что этого больше нет?

EDIT # 2: Не можете ли вы просто сделать allIcons как свойство MapListings?

EDIT # 3: Я думаю, что он работает, но, возможно, вы не обращаетесь к нему правильно? Это или он не создает объект с Google каким-то образом, или ошибка вы вывесили происходит в другом месте, а не здесь

function GIcon(){}; 
var MapListings = { 
    icons: function() { 
     var allIcons = [] ; 

     return { 
      add: function (iconType, iconImage) { 
       var icon = new GIcon(MapListings.baseIcon); 
       icon.image = iconImage; 
       allIcons[iconType] = icon; // fails, but this is what I want 
       // allIcons.push(icon); // works, but this is not what I want 
       window.x = allIcons 
      }, 
      get: function() { 
       return allIcons; 
      } 
     }; 

    }() 
}; 

MapListings.icons.add("c7", "/images/maps/blue.png"); 
MapListings.icons.add("c8", "/images/maps/red.png"); 

alert(MapListings.icons.get()['c8']['image']) 

Вы не должны петли с помощью .Length, но вместо того, чтобы получить быстрый доступ к c7 или c8.

x = MapListings.icons.get(); 
for (var prop in x) { 
    if (x.hasOwnProperty(prop)) { 
     alert(x[prop]['image']) 
    } 
} 

Answer 2

allIcons[iconType] = icon; терпит неудачу, потому что allIcons является массив, не является объектом. Попробуйте инициализировать allIcons вместо {}. Это позволит вам размещать элементы в коллекции по ключу.

Answer 3

Итак, одно можно сделать, чтобы исправить это - изменить способ обращения к массиву.Так как внешний по отношению к методу надстройки вы это делаете:

MapListings.icons["c7"] 

Вы также можете просто использовать это, чтобы добавить в массив внутри добавления функции:

add: function (iconType, iconImage) { 
    MapListings.icons[iconType] = iconImage; 
},