如何在ICountry对象数组中查找包含指定ICity对象的所属国家？

我需要在ICountry类型的对象数组中检索，找到包含指定ICity类型对象的所属国家并赋值给变量。目前定义了如下接口：

import { ICity } from "./city";
export interface ICountry {
  name: string,
  capital: string,
  language: string,
  population: number,
  density: number,
  area: number,
  majorCities: ICity[]
}

我编写了remove函数，传入的city参数即为要查找的ICity对象，但执行后country变量始终返回undefined。当前代码如下：

remove(city: ICity): void {
  var country;
  this.countries.forEach(cn => {
    if (cn.majorCities.includes(city)) {
      country = cn;
      console.log(cn);
    }
  });
  console.log(country);
}

请问实现“查找指定城市所属国家”需求的最佳方法是什么？

先给你分析下当前代码出问题的核心原因：Array.prototype.includes()是基于**严格相等（===）**判断元素是否存在的。如果传入的city参数和数组里的ICity对象不是同一个内存引用（比如是两个属性完全相同但独立创建的对象），includes()就会返回false，自然找不到对应的国家。

下面给你几种实用的实现方式，按推荐程度排序：

1. 使用Array.prototype.find()（最推荐） #

find()方法会返回数组中第一个满足测试条件的元素，找不到则返回undefined，完全匹配你的需求，代码也更简洁高效：

remove(city: ICity): void {
  // 假设你的ICity有唯一标识属性（比如id），用它来匹配业务上相同的城市
  const country = this.countries.find(cn => 
    cn.majorCities.some(c => c.id === city.id)
  );
  
  console.log(country);
  // 后续可根据需求处理找到的country，比如赋值或其他逻辑
}

这里用some()替代includes()，通过对象的唯一业务标识（比如id、cityCode）判断是否为目标城市——这是处理对象数组匹配的常规操作，彻底避开了引用相等的限制。

2. 修复原有的forEach写法 #

如果你坚持想用forEach，核心还是要解决对象匹配的问题，同样通过唯一标识来判断：

remove(city: ICity): void {
  let country;
  this.countries.forEach(cn => {
    // 检查当前国家的城市数组中是否存在目标城市
    const hasTargetCity = cn.majorCities.some(c => c.id === city.id);
    if (hasTargetCity) {
      country = cn;
      // 找到后直接return跳出当前迭代，不用继续遍历剩余元素
      return;
    }
  });
  console.log(country);
}

注意这里用let替代了var（var存在变量提升问题，容易引发意外bug），找到匹配项后及时终止迭代也能提升性能。

关键注意事项 #

• 永远不要依赖对象引用相等来判断两个业务上相同的对象，一定要用唯一业务标识（比如id）作为匹配依据。
• find()方法天生适合“查找第一个匹配项”的场景，相比forEach可读性更好，性能也更优（找到匹配项后立即停止遍历）。

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