[RFC]: Add `@stdlib/utils/every-in-by`

[kgryte]

Description

This RFC proposes adding a utility to test whether every "own" and inherited property of a provided object satisfies a predicate function.

This is the for-in equivalent of @stdlib/utils/every-by.

Package: @stdlib/utils/every-in-by
Alias: everyInBy

Related Issues

None.

Questions

No.

Other

No.

Checklist

• I have read and understood the Code of Conduct.
• Searched for existing issues and pull requests.
• The issue name begins with RFC:.

[saisrikardumpeti]

would you explain this more i am not able to understand this?

[kgryte]

By example,

function predicate( value, key, obj ) {
	return ( value > 0 );
}

var o = {
	'a': 1,
	'b': 2,
	'c': 3,
	'd': 4
};

var bool = everyInBy( o, predicate );
// returns true

o.a = -2;
bool = everyInBy( o, predicate );
// returns false

Pay attention to the OP. Namely, it should also work for inherited properties.

[saisrikardumpeti]

i have created this test function will this work or should i make changes?

function predicate( value, key, obj ) {
	return ( value > 0 );
}

function everyByIn(obj, predicate, thisArg) {
  var out

  if ( !isObject( obj ) ) {
		throw new TypeError( format( 'invalid argument. First argument must be a object. Value: `%s`.', obj ) );
	}
	if ( !isFunction( predicate ) ) {
		throw new TypeError( format( 'invalid argument. Second argument must be a function. Value: `%s`.', predicate ) );
	}

  for (const key in obj) {
    if (obj.hasOwnProperty(key)) {
      out = predicate.call(thisArg, obj[key], key, obj)
      if (!out) {
        return false
      }
    } 
  }
  return true
}

[kgryte]

You don't want the hasOwnProperty check, as this function should also check inherited properties, as mentioned in the OP.

[kgryte]

You can also inline the predicate.call expression into the conditional. No need for a temporary variable in this case.

[kgryte]

Otherwise, yes, what you have proposed is a good start. Feel free to open a PR with the implementation and associated package files.

@Planeshifter Should we go ahead and rename the package to @stdlib/object/every-in-by?

[saisrikardumpeti]

cool i am excited on this one i will make changes regarding to it. then comment it before making a PR.