Prefer Object.entries instead of for loop

[milamer]

// ✅ Good
for (let i = 0; i < items.length; i++) {
	const n = items[i]
	console.log(`${n} at index ${i}`)
}

Together with noUncheckedIndexedAccess this is a pain.

IMO better:

for (const [i, n] of Object.entries(items)) {
	console.log(`${n} at index ${i}`)
}

[kentcdodds]

I agree. I was looking at my for loop recommendation and was bothered by it, but didn’t consider entries. I haven’t double checked, but I’m guessing this works as expected for arrays as well?

I’d welcome a pull request for this change.

[kentcdodds]

Hmmm... After looking at this, it looks like the index is coerced to a string. Not sure I'm comfortable with that. But we can use Array.prototype.entries!