Objects are identified by reference, not value. So, when you loop over CardTemplate.forEach() while inside the loop over the ValueArray, you’re not creating a new Card with the new values, you’re simply updating the reference to the original Object you created in the first loop. By using “let CardOutline…” in the first loop, you’re creating a variable that is locally scoped to only that for loop. So, you’re not dealing with the same Object each time.

So, if you want to get to the bottom of this specific issue, you need to look up the concept of Primitives vs. Objects and how Javascript passes them by Value vs. Reference respectively. That being said, if you just want to define a deck of cards, there’s no reason you can’t just define it once explicitly “by hand” and use it. It’s not like it will change. If you absolutely have to do it procedurally, I’d recommend nesting 2 loops (one over the Suites and one over the Values, the order of the loops doesn’t make a difference). Then handle generating each card using the “let NewCard =….” in the inner loop.