Ionic中for循环内contacts.find()仅执行最后一个元素的问题

看起来你踩了异步操作在同步循环里的经典坑！问题出在this.contacts.find()是异步Promise操作，而你的for循环是同步执行的——循环会一口气跑完所有迭代，此时所有的find请求都还在等待结果，等这些Promise终于resolve的时候，循环里的变量已经指向了最后一个元素，所以看起来只有最后一次查询生效了。另外你用了var i，这个变量的作用域是整个函数，所有回调都会共享同一个i的最终值，这也加重了问题。

下面给你几个可行的解决方案：

1. 先修复变量作用域，改用forEach循环 #

forEach的每个回调都有独立的作用域，能完美绑定当前迭代的联系人数据，避免变量共享问题：

somefunc() {
  data.forEach(item => {
    // 创建当前迭代对应的联系人对象
    let contact: Contact = this.contacts.create();
    contact.name = new ContactName(null, item.lastName, item.firstName, item.middleName);
    contact.photos = [new ContactField("base64", item.photoBase64, true)];
    contact.phoneNumbers = [new ContactField(item.phoneNumber, "mobile", true)];

    // 注意：这里的筛选条件要对应当前联系人，别用固定的"Sam"！
    this.contacts.find(
      ["displayName", "phoneNumbers", "emails"],
      { 
        multiple: true, 
        filter: `${item.firstName} ${item.lastName}`, // 用当前联系人的姓名筛选
        hasPhoneNumber: true // 要检查手机号的话，这里设为true更准确
      }
    ).then((contactsArray) => {
      if (contactsArray.length === 0) {
        // 通讯录里没有这个联系人，保存新的
        contact.save().then(saved => {
          console.log(`成功保存联系人：${saved.displayName}`);
        }).catch(saveErr => {
          console.error(`保存联系人失败：`, saveErr);
        });
      } else {
        // 联系人已存在，更新现有数据
        const existing = contactsArray[0];
        existing.phoneNumbers = contact.phoneNumbers;
        existing.photos = contact.photos;
        existing.save().then(updated => {
          console.log(`成功更新联系人：${updated.displayName}`);
        }).catch(updateErr => {
          console.error(`更新联系人失败：`, updateErr);
        });
      }
    }).catch(findErr => {
      console.error(`查询联系人失败：`, findErr);
    });
  });
}

2. 用async/await+for...of实现串行处理（推荐） #

如果你需要保证联系人的处理顺序（比如必须前一个处理完再处理下一个），async/await配合for...of是最清晰的写法，还能避免回调地狱：

async somefunc() {
  for (const item of data) {
    try {
      // 创建当前联系人对象
      let contact: Contact = this.contacts.create();
      contact.name = new ContactName(null, item.lastName, item.firstName, item.middleName);
      contact.photos = [new ContactField("base64", item.photoBase64, true)];
      contact.phoneNumbers = [new ContactField(item.phoneNumber, "mobile", true)];

      // 等待查询结果返回
      const contactsArray = await this.contacts.find(
        ["displayName", "phoneNumbers", "emails"],
        { 
          multiple: true, 
          filter: `${item.firstName} ${item.lastName}`,
          hasPhoneNumber: true
        }
      );

      if (contactsArray.length === 0) {
        const savedContact = await contact.save();
        console.log(`联系人已保存：${savedContact.displayName}`);
      } else {
        const existingContact = contactsArray[0];
        existingContact.phoneNumbers = contact.phoneNumbers;
        existingContact.photos = contact.photos;
        const updatedContact = await existingContact.save();
        console.log(`联系人已更新：${updatedContact.displayName}`);
      }
    } catch (error) {
      console.error(`处理联系人${item.firstName}失败：`, error);
    }
  }
}

几个关键提醒 #

• 别用固定筛选条件：你原来代码里filter写死了"Sam"，这肯定不对，要根据当前要创建的联系人信息来筛选，才能准确判断是否已存在。
• 错误处理不能少：每个Promise操作都要加catch，不然出现错误会变成未处理的Promise rejection。
• var vs let：永远在循环里用let或const，别用var，var的作用域是函数级的，很容易导致变量共享问题。

内容的提问来源于stack exchange，提问作者beginner