Glavin001/tslint-clean-code
GitHub で見るUse Array.prototype.find() instead of .filter()[0]
Open
#56 opened on 2018年7月11日
good first issuehelp wanted
Repository metrics
- Stars
- (175 stars)
- PR merge metrics
- (30d に merged PR はありません)
説明
See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/find
const items = [
{ id: 123 },
{ id: 456 }
// ...
];
const selectedId = 456;
const selectedItem = items.filter(item => item.id === selectedIds)[0];
Should be:
// ...
const selectedItem = items.find(item => item.id === selectedIds);
Diff:
- const selectedItem = items.filter((item) => item.id === selectedIds)[0];
+ const selectedItem = items.find((item) => item.id === selectedIds);