[Select] Simplify handleChange in SelectInput

[ulrichstark]

This line

material-ui/packages/mui-material/src/Select/SelectInput.js

Line 247 in 34f2253

const index = childrenArray.map((child) => child.props.value).indexOf(event.target.value);

could be simplified to const child = childrenArray.find((child) => child.props.value === event.target.value); to avoid:

• creation of the temporary array as result of map()
• second iteration through the array
• the following line:
  

  material-ui/packages/mui-material/src/Select/SelectInput.js

  Line 253 in 34f2253

  const child = childrenArray[index];

I can do the change.
Is it worth submitting a PR?

[michaldudak]

Nice catch! Feel free to submit a PR.

[ulrichstark]

Submitted @michaldudak