Я думал может можно укоротить как то

const {selectedOptionsTerrain,  selectedOptionsPopulation} = this.state
if (selectedOptionsTerrain) {
           planets = planets.filter(p => p.terrain.split(', ').includes(selectedOptionsTerrain));
       }
       if (selectedOptionsPopulation) {
           planets = planets.filter(p => p.population > selectedOptionsPopulation)
       }

Итерация первая - деструктуризация

второй if перекрывает первый?

Это два селкта фильтра

если оба тру то работает 2

если один фалс то только один

Я бы тоже взглянул на очень красивый вариант

Вообще, по уму, тебе поменять их местами, если первый отработал, то ретурн. Если нет, тогда запускается второй. В зависимости от величины массива и величины вложенного массива это может сэкономить кучу времени

Только не пугайтесь сильно, это просто другой стиль программирования - функциональный.

Вынесем отсюда логику в функцию:

if (selectedOptionsTerrain) {
  planets = planets.filter(p => p.terrain.split(', ').includes(selectedOptionsTerrain));
}

—->

const findPlanetsByTerrain = (terrain: string) => filter(pipe(
    prop('terrain'),
    split(','),
    includes(terrain)
));

Тоже самое для второго условия:

if (this.state.selectedOptionsPopulation) {
  planets = planets.filter(planet => planet.population > this.state.selectedOptionsPopulation)
}

—->

const findPlanetsWherePopulationMoreThan = (population: number) =>
    filter((planet: Planet) => planet.population > population);

Напишем общую функцию: которая проверяет условия:

const filterPlanets = (terrain: string, population: number) => pipe(
    when(() => isPresent(terrain), findPlanetsByTerrain(terrain)),
    when(() => isPresent(population), findPlanetsWherePopulationMoreThan(population))
);

Полный код с тестом:

const { filter, pipe, prop, split, includes, isNil, negate } = require('lodash/fp');

interface Planet {
    terrain: string;
    population: number;
}

const planets: Planet[] = [{
    terrain: 'ter1,ter2,ter3',
    population: 500000
}, {
    terrain: 'ter3,ter4,ter5',
    population: 1200000
}];

const isPresent = negate(isNil);

const when = (conditionFunc: Function, onTrueFunc: Function) => (data: any) =>
    conditionFunc(data) ? onTrueFunc(data) : data;

const filterPlanets = (terrain: string, population: number) => pipe(
    when(() => isPresent(terrain), findPlanetsByTerrain(terrain)),
    when(() => isPresent(population), findPlanetsWherePopulationMoreThan(population))
);

const findPlanetsByTerrain = (terrain: string) => filter(pipe(
    prop('terrain'),
    split(','),
    includes(terrain)
));

const findPlanetsWherePopulationMoreThan = (population: number) =>
    filter((planet: Planet) => planet.population > population);

const testFilterPlanets = () => {
    console.log('filterPlanets', filterPlanets('ter5', 1000000)(planets));
};

testFilterPlanets();

Можно упрощать еще дальше :) А если подключить RamdaJS, то будет еще легче дышать. В примере выше мне понадобилось написать два хелпера, isPresent и when. К сожалению lodash/fp не включает when()

^ @dezoxel  готовая статья на медиум )

Безумно нравится сама идея ФП, но вот человек просил сократить код, а тут сократить как-то не очень получается )

Спасибо за пример!

А что мы считаем сокращением? Если кол-во символов, я могу сейчас все в одну строку написать, будет коротко)

Я считаю правильным сравнивать вот эти куски кода, какой из них лучше читается.

Императивно с элементами ФП:

if (this.state.selectedOptionsTerrain) {
    planets = planets.filter(planet => planet.terrain.split(', ').includes(this.state.selectedOptionsTerrain));
}

if (this.state.selectedOptionsPopulation) {
    planets = planets.filter(planet => planet.population > this.state.selectedOptionsPopulation)
}

Функционально:

const filterPlanets = (terrain: string, population: number) => pipe(
    when(() => isPresent(terrain), findPlanetsByTerrain(terrain)),
    when(() => isPresent(population), findPlanetsWherePopulationMoreThan(population))
);

чето с ифами лучше, если честно :)

И ними привычнее, но по языку лучше с when

по какому языку лучше с when ?

ааа, все понял.

ну не знаю.

чет такое.