Как пробежаться по свойствам объекта в js?

Question

[mamaanarhiya]

есть объект с объектами вида:

{operation: {status: "success", text: "Approved"} , anotherOperation: {status: "success", text: "Approved"}, op: {status: "error", text: "Declined, because..."}}

Нужно получить "text" из объектов, у которых в "status" прописано "error", как это сделать?

Comments

[Леонид Князев]

А, в чём проблема?
Перебираете ключи объекта и у каждого ключа проверяете чему равно поле status. Если равно "error", то у этого же ключа берете значение поля text

[Kovalsky]

Вы шутите что ли? Последнюю неделю на тостере каждый день на тостере задают вопрос "как итерировать по свойствам объекта". Уже даже репортить смысла нет, модераторы вероятно устали от этого и уже даже не удаляют.

"в чём проблема?" - вероятно в том, что ТСу лень даже самому лезть в гугл. Но в некотором смысле он прав, т.к. энтузиастыкармадрочеры на тостере настолько активны, что с пеной у рта рвутся написать решение для любой проблемы, даже если очевидно что ТС просто хочет попользоваться ими как бесплатной рабочей силой.

Answer 1

[Антон Спирин]

const obj = {
  operation: {
    status: "success", 
    text: "Approved"
  } ,
  anotherOperation: {
    status: "success", 
    text: "Approved"
  },
  op: {
    status: "error", 
    text: "Declined, because..."
  }
};

const textsArray = Object.values(obj).filter(el => el.status === 'error').map(el => el.text); 

// результат: ["Declined, because..."]

Answer 2

[Morfeey]

for (var key in YourObject) {
   var obj = YourObject[key];
   if (obj.status === "error") {
       console.log(obj.text);
   }
}

Comments

[Антон Спирин]

1. Переменные в js в верхнем регистре называть не принято.
2. for in не гарантирует соблюдение очередности при итерации.
3. Использовать правильно и безопасно for in надо так:

for (var key in yourObj) {
  if(Object.prototype.hasOwnProperty.call(yourObj, key) ) {
    var obj = yourObject[key];
     if (obj.status === "error") {
        console.log(obj.text);
     }
  } 
}

[Morfeey]

Антон Спирин, спасибо, учту)

[Антон Спирин]

Morfeey, Проверка hasOwnProperty исключит наследуемые перечисляемые свойства. А вызов ее из протоипа объекта исключит ее возможное переопределение на экземпляре, в случае которого приложение может поймать исключение и завершиться ошибкой.
Все это конечно маловероятно, но в промышленных проектах с кучей разработчиков лучше застраховаться от всего. По крайней мере, знать полезно.

[Kovalsky]

Антон Спирин, вы шутите что ли? Такой уровень паранойи скорее пугает, чем внушает уважение. Ну раз вы зовёте Object.prototype.hasOwnProperty из-за того что боитесь переопределения у экземпляра, то почему бы вам вообще в самом начале сценария не сохранять исходную функцию Object.prototype.hasOwnProperty в константу и потом вызывать сохранённую версию? А то ведь переопределить могут, рофл.

"в промышленных проектах с кучей разработчиков лучше застраховаться от всего" - всё немного не так. Гляньте как организована разработка в Mozilla, там к коду есть цельная куча требований; весь код написан так что перестраховываться не от чего, ибо любые извращения, включая вызов hasOwnProperty из прототипа, не пройдут еще на стадии ревью. Если вам кажется, что это не так, то просто представьте себе, что вы начальствуете над отделом разработки в 100 человек, пишете требования к коду, выделяете сотрудников под осуществлению ревью, и как-то раз они вам доносят, что какой-то в целом полезный, но всё-таки мудак держит отдельный индекс под исходные версии встроенных функций.

[Антон Спирин]

Никита Полевой, это не моя паранойя, а промышленные стандарты. Почитайте в документации и обсуждениях на github ESlint почему к этому пришли.
Вот сохранять в константу нет никакого смысла.

Мне не надо смотреть в документацию Mozila, так как я работаю в больших промышленных проектах с высокими требованиями к качеству и безотказности кода.
А это всего лишь один из стандартов гарантирующих безопасность и безотказность. Спорить с этим глупо.
А обсуждения почитайте на досуге, там много примеров.

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

[Kovalsky]

Антон Спирин, "это не моя паранойя, а промышленные стандарты" - ну конечно, а Mozilla это так, лавчонка подмосковная, их source code submission policy совсем не является промышленным стандартом, я понимаю.

[Антон Спирин]

Никита Полевой, вы что сейчас доказать пытаетесь? Спорите с действительностью?
"source code submission policy" - нет такого ни гайдлайна, ни стандарта. Или вы имели ввиду требования для разработчиков аддонов? https://developer.mozilla.org/en-US/Add-ons/Source...

Вот правило https://eslint.org/docs/rules/no-prototype-builtins .
ESLint сейчас стандарт в промышленной JS разработке. Prettier скоро им станет.
Попробуйте устроиться в маломальски серьезную компанию использующую ESLint и precommit проверки в проектах. Скорей всего, если это правило нарочно не отключили, что вряд ли, то вы даже коммит сделать не сможете.

[Kovalsky]

Антон Спирин, простите меня, грешного, что я не помню наизусть названия всех документов Mozilla. Позже найду. Нет, не требования для разработчиков аддонов, а официальный документ от Mozilla в котором написано как именно разделён труд в Mozilla, о чем должен заботиться разработчик, а что не его забота. В данный момент я не помню как он называется и не могу найти, до сих пор ищу и не вижу его.

Лол, я прочитал что в вашем правиле написано, это гениально. То есть вместо того, чтобы ввести правило не создавать объекты с помощью Object.create(null), они решили "подстелить соломки" - ввести правило вызывать встроенные функции только сделав сальто мортале. Вы правда считаете, что подобное может иметь право на жизнь в "серьезной" компании? Я представления не имею кто это придумал, но звучит как "у нас тут некоторые разработчики срут прямо на пол, вы пожалуйста смотрите в оба чтобы не наступить. А запретить мы им не можем, они ссылаются на конвенцию о защите прав человека и основных свобод"

[Антон Спирин]

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

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

Зайдите на сайт https://eslint.org найдите плашку Who's using ESLint и нажмите. Узнаете, что это стандарт чуть ли не для всего передового фронтенда, а с ними и для многих других компаний.
Можете посмотреть на Prettier https://prettier.io/en/users/, он также набирает популярность.
Сам рекомендую к использованию оба инструмента.

Вот ESLint файл из первого попавшегося репозитория Mozilla https://github.com/mozilla/thimble.mozilla.org/blo...
Они тут так же используют Prettier. Не удивлюсь если узнаю, что он используется во всех их более менее новых JS проектах как стандарт.

[userS4]

Антон Спирин,
Здравствуйте! Вижу, вы хорошо разбираетесь в вопросе. Не могли бы Вы привести примеры ситуаций, когда использование  HasOwnProperty небезопасно? Уже долго пытаюсь разобраться с этим правилом, нагуглил этот топик, но пока так и не понял.