Support pullAllWith

[leehj322]

Hi, @raon0211

I would like to implement the pullAllWith method for issue #91.

Would it be okay to implement it in the compat layer?

Suggestion

Option1 (Modify the array in-place using resultIndex)

export function pullAllWith<T>(array: T[], values: T[], comparator: (a: T, b: T) => boolean): T[] {
  let resultIndex = 0;

  for (let i = 0; i < array.length; i++) {
    let shouldRemove = false;
    for (let j = 0; j < values.length; j++) {
      if (comparator(array[i], values[j])) {
        shouldRemove = true;
        break;
      }
    }
    if (!shouldRemove) {
      array[resultIndex++] = array[i];
    }
  }

  array.length = resultIndex; // Resize the array

  return array;
}

Option2 (Create a new filtered array)

export function pullAllWith<T>(array: T[], values: T[], comparator: (a: T, b: T) => boolean): T[] {
  const modifiedArray = array.filter(item => !values.some(value => comparator(item, value)));
  array.length = 0; // Reset array
  array.push(...modifiedArray);

  return array;
}

I expect Option1 to be more performant but slightly harder to read.
I'd love to hear your thoughts on which option would be more appropriate!

[raon0211]

What about we check each function's performance using benchmarks (you can see other benchmarks in our benchmark folder) and try with the more performant one? I think the first one is fine, it is not too complicated.