Finding objects within objects

Question

I'm trying to search inside an object array to see if any object contains the object I'm looking for and assign it to a variable.

This is the interface that I'm using. Basically I have an array of countries, each of which has it's own array of cities.

import { ICity } from "./city";

export interface ICountry {
    name: string,
    capital: string,
    language: string,
    population: number,
    density: number,
    area: number,
    majorCities: ICity[]
}

The object I'm looking for is the city parameter of this function, but it always returns undefined. What is the best way to find country which a certain city belongs to?

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

is ICity an object? If so you can't use includes unless you are looking for strict reference equality. You will need to look for a matching property of city either in a find(), findIndex() call, or if you just want to establish if it exists in the array you can use some(). ie. if (cn.majorCities.some(c => c.name === city.name)) {... — pilchard
– pilchard, Commented Jan 9, 2022 at 13:43
Does this answer your question? How to determine if Javascript array contains an object with an attribute that equals a given value? — pilchard
– pilchard, Commented Jan 9, 2022 at 13:52

Mahdi Zarei · Accepted Answer · 2022-01-09 13:24:04Z

1

The best way (in my opinion) is to store country id in city so you can find it easier.

But in this case you can do it like below:

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

answered Jan 9, 2022 at 13:24

Mahdi Zarei

7,7538 gold badges36 silver badges67 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

pilchard Over a year ago

This is no different than the OP's current code and assumes city is a string

Mahdi Zarei Over a year ago

it is a lot different because arrays don't support includes

pilchard Over a year ago

Array.prototype.includes()

Mahdi Zarei Over a year ago

You just teach me something. thanks man.

obscure · Accepted Answer · 2022-01-09 13:53:54Z

1

Your ICity type object is not just a simple string I assume, so a check like:

if (cn.majorCities.includes(city))

would just return true if one of majorCities elements is the actual instance referenced via the city variable.

As your ICity interface surely consists of something like a name property e.g.

interface ICity {
name: string
}

you should check for such a string-type property.

if (cn.majorCities.some((el) => {
        return el.name == city.name

    })) {
    // do something
}

edited Jan 9, 2022 at 13:53

answered Jan 9, 2022 at 13:48

obscure

13k2 gold badges19 silver badges40 bronze badges

Comments

Ahmad · Accepted Answer · 2022-01-09 14:16:46Z

1

You should do like below

const countries = [{
    name: 'Iran',
    cities: ['Shiraz', 'Tehran']
  },
  {
    name: 'Germay',
    cities: ['Berlin']
  }
]

const findCity = (city) => {
  countries.forEach(country => {
    if (country.cities.includes(city))
      console.log(city, 'has founded!')

  })
}
findCity('Shiraz')

answered Jan 9, 2022 at 14:16

Ahmad

9042 gold badges9 silver badges15 bronze badges

Collectives™ on Stack Overflow

Finding objects within objects

3 Answers 3

4 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related