Javascript - sprawdzenie czy cały obiekt jest pusty

Question

Witam, mam obiekt tego typu:

obiekt = {
  '1': null,
  '2': 'Tekst', 
  '3': 1, 
  'obj': {
    '1': null, 
    '2': null
  }
};

Chcę przepuścić go przez funkcję, która sprawdza czy dany obiekt jest pusty (=== '' lub === null). Jeżeli przykładowo property obj jest puste, powinno go znullować. W przypadku tego obiektu powinno to zostać tak:

obiekt = {
  '1': null,
  '2': 'Tekst', 
  '3': 1, 
  'obj': null
};

Mam taką funkcję:

nullujPusteObiekty(obj) {
    for (const key in obj) {
      if (obj.hasOwnProperty(key)) {
        if (obj[key] instanceof Object) {
          this.nullujPusteDokumenty(obj[key]);
        }
        if (isEmptyOrNull(obj[key])) {
          console.log(obj[key], 'jest pusty');
        } else {
          continue;
        }
        obj = null;
      }
    }
  }

Problem w tym, że nie wiem w którym miejscu zamieścić zerowanie obiektu (obj = null). Nie mam już pomysłów w jaki sposób to ugryźć

Comments

  nullujPusteDokumenty(obj) {
    const iloscProperty = Object.getOwnPropertyNames(obj).length;
    let licznikPustychElementow = 0;
    for (const key in obj) {
      if (obj && key && obj.hasOwnProperty(key)) {
        if (obj[key] instanceof Object) {
          this.nullujPusteDokumenty(obj[key]);
        } else {
          if (isEmptyOrNull(obj[key])) {
            licznikPustychElementow += 1;
          } else {
            continue;
          }
        }
      }
    }
    if (licznikPustychElementow === iloscProperty) {
      obj = null;
    }
  }

Doszedłem do czegoś takiego. Funkcja prawidłowo określa czy obiekt jest pusty czy nie. Ale mam inny problem - kiedy już funkcja robi ostatnią linijkę (obj=null) to i tak obiekt przekazany pozostaje nienaruszony, jakby był bez referencji.

Answer 1

Proponuję coś takiego:

const obj = {
	'1': null,
	'2': 'Tekst',
	'3': 1,
	'obj': {
		'1': null,
		'2': null
	}
};

function nullObject( obj ) {
	const isEmpty = obj ? Object.keys( obj ).length === 0 : true;
	const isNull = obj ? Object.values( obj ).every( val => val == null ) : true;

	if ( isEmpty || isNull ) {
		return null;
	}

	const entries = Object.entries( obj ).map( ( [ name, value ] ) => {
		if ( value && typeof value === 'object' && !Array.isArray( value ) ) {
			value = nullObject( value );
		}

		return [ name, value ];
	} );

	return Object.fromEntries( entries );
}

console.log( nullObject( obj ) );

Jeśli musimy wspierać też Edge'a, to będzie trzeba zamienić Object.entries( obj ).map + Object.fromEntries na zwykłą pętlę for...of.

Comments

No to wygląda zdecydowanie lepiej:P Jednak z tego co widzę fromEntries jest bardzo świeże, więc próbuję robić to za pomocą for..of. Jednakże nie do końca mi wychodzi zamiana kodu na tego for'a, poprawisz mnie?

export function nullObject(obj) {
  const isEmpty = obj ? Object.keys( obj ).length === 0 : true;
  const isNull = obj ? Object.values( obj ).every( val => val == null ) : true;
  if ( isEmpty || isNull ) {
    return null;
  }
  for (let [name, value] of obj) {
    if ( value && typeof value === 'object' && !Array.isArray( value ) ) {
      value = nullObject( value );
    }
    obj += [name, value];
  }
  return obj;
}

 

---

Hmm, w sumie chyba bardziej elegancko będzie na Object.keys( obj ).forEach:

function nullObject( obj ) {
	const isEmpty = obj ? Object.keys( obj ).length === 0 : true;
	const isNull = obj ? Object.values( obj ).every( val => val == null ) : true;

	if ( isEmpty || isNull ) {
		return null;
	}

	Object.keys( obj ).forEach( name => {
		let value = obj[ name ];

		if ( value && typeof value === 'object' && !Array.isArray( value ) ) {
			value = nullObject( value );
		}

		obj[ name ] = value;
	} );

	return obj;
}

Niemniej jest poważna różnica między moją pierwszą wersją kodu a tą. Nowa modyfikuje bezpośrednio przekazany obiekt, a pierwotna wersja tworzy jego kopię. Żeby nowa również nie modyfikowała obiektu, wypada na początku funkcji dodać jeszcze jedną linijkę: 

obj = Object.assign( {}, obj );

 

---

Dziękuję bardzo za pomoc! :)

Answer 2

https://stackoverflow.com/questions/679915/how-do-i-test-for-an-empty-javascript-object

Answer 3

Object.keys

metoda ta zwraca tablicę kluczy obiektu np.

Object.keys({
  foo: 123,
  bar: 456
}) // == ['foo', 'bar']

więc obiekt jest pusty jeśli

Object.keys(obj).length === 0

 

Answer 4

Dzięki za próbę pomocy ale te sposoby znalazłem i działa to tylko w przypadku dosyć płytkich obiektów, bez wdawania się głębiej, nie mówiąc o tym, że gdy property obiektu = null to i tak length jest wliczany. Ja chciałem coś, co sprawdzi tak głęboko jak się da i wynulluje puste całkiem obiekty. 

Zrobiłem to w ten sposób:

  nullujPusteDokumenty(obj) {
    const iloscProperty = Object.getOwnPropertyNames(obj).length;
    let licznikPustychElementow = 0;
    for (const key in obj) {
      if (obj && key && obj.hasOwnProperty(key)) {
        if (obj[key] instanceof Object && !_.isArray(obj[key])) {
          obj[key] = this.nullujPusteDokumenty(obj[key]);
        } else if (obj[key] instanceof Array) {
          if (obj[key].length === 0) {
            licznikPustychElementow += 1;
          } else {
            continue;
          }
        } else {
          if (isEmptyOrNull(obj[key])) {
            licznikPustychElementow += 1;
          } else {
            continue;
          }
        }
      }
    }
    if (licznikPustychElementow === iloscProperty) {
      return null;
    } else {
      return obj;
    }
  }

Sprawdzam na początku czy badane property obiektu nie jest przypadkiem kolejnym obiektem (jeśli tak to znowu wywołuje tę samą funkcję). Później zabezpieczam jeśli property byłby tablicą (samo sprawdzenie czy jest obiektem nie pomogło - tablica też jest traktowana jako obiekt). 

Jeśli ktoś ma sposób jak to napisać ładniej - chętnie zobaczę, bo wydaje mi się to trochę przekombinowane.

Comments

Jeszcze jeden błąd który znalazłem to continue - te elsy są totalnie niepotrzebne.

 Dla potomnych