问题描述
|
我知道如何从数组中删除重复项,但是我想做的是防止某个项目已经存在时首先将其添加到数组中。我正在以循环方式从xml提要中提取数据,我认为搜索该值索引是可行的,但是无论如何,索引始终为-1。这是我的代码:
var yearArr:Array = new Array();
for (var i=0;i<numCovers;i++){
var coverRef = xmlObj.cover[i];
var coverClip:MovieClip = new MovieClip();
coverClip.year = coverRef.@year;
if (yearArr.indexOf(coverClip.year === -1)){
yearArr.push (coverClip.year);
}
}
也许我误解了indexOf函数,但我认为如果数组中不存在值,则应该返回-1。我究竟做错了什么?
解决方法
这是我想出的解决方案:
var yearArr:Array = new Array();
for (var i=0;i<numCovers;i++){
var coverRef = xmlObj.cover[i];
var coverClip:MovieClip = new MovieClip();
coverYear = coverRef.@year;
addCoverYear(coverYear);
}
function addCoverYear(coverYear:int):void {
if (yearArr.indexOf(coverYear) == -1){
yearArr.push(coverYear);
}
}
, 您可以通过将所有内容传递给字典来减少数组,字典将自动删除冗余。然后将字典作为新数组传递回来。
//Reduce Array
private function reduceArray(array:Array):Array
{
var dictionary:Dictionary = new Dictionary();
for each (var element:String in array)
dictionary[element] = true;
var result:Array = new Array();
for (var key:String in dictionary)
result.push(key);
dictionary = null;
return result;
}
, 您的代码几乎可以了。问题在于E4X属性“ 3”不是文字字符串(我现在不确定,但我相信它是“ 4”对象)。这就是indexOf
调用将继续返回-1的原因,因为它正在查找该对象的副本,而不是字符串。一旦您将E4X放到只能放置字符串的地方,它将立即将其转换为字符串,但是直到那时为止,它都是另外一回事。
如果您以这种方式重写代码,则该代码应立即可用:
var yearArr:Array = new Array();
for each (var coverRef : XML in xmlObj.cover){
var year : String = coverRef.@year; // force the property to be a string
if (yearArr.indexOf(year) < 0){
yearArr.push (year);
}
}
您还可以对代码进行其他一些优化。没有使用new MovieClip()
部分,不是所有变量都经过强类型输入,通过使用for each
循环,您可以更清楚地说明要循环通过的对象。
, 这是您可以做的,例如,如果您有一个字符串数组。
var ItemList:Array = new Array();
for each(var Item:String in UseYourXMLFeed)
{
if(ItemList.indexOf(Item) == -1)
{
ItemList.push(Item);
}
}
编辑:
无论如何,您真正的答案在Sam的评论中。