Refactor or split this lesson about js-functions-ins-and-outs HOT 7 OPEN

Branch? File name?

from js-functions-ins-and-outs.

I'm guessing this: https://github.com/ga-wdi-boston/js-functions-ins-and-outs/blame/master/README.md#L84

from js-functions-ins-and-outs.

https://github.com/ga-wdi-boston/js-functions-ins-and-outs/blame/master/README.md#L98

This moves it from ins to both ins and outs. Avoid having them write functions that replace things in the objects passed.

from js-functions-ins-and-outs.

Like this?

const arrayTransform = function (values, predicate, mutator) {
  // if the predicate is true, mutate the value, otherwise don't mutate it
  let result = [];

  for (let i = 0; i < values.length; i++) {
    let value = values[i];

    if (predicate(value)) {
      result.push(mutator(value));
    } else {
      result.push(value);
    }
  }

  return result;
};

from js-functions-ins-and-outs.

Yes, or like this, :-).

let result = new Array(values.length);

for (let i = 0; i < values.length; i++) {
  result[i] = predicate(values[i]) ? mutator(values[i]) : values[i];
}

from js-functions-ins-and-outs.

Moved in readme, but not closing since I think this lesson needs to be refactored/split/something.

from js-functions-ins-and-outs.

Is it ok to continue this over in #12? If so, we can close this. Or, we can keep this open and close #12?

from js-functions-ins-and-outs.