# Spot the bug - #5

**URL:** <https://forum.kirupa.com/t/spot-the-bug-5/680465>\
**Category:** web dev\
**Created:** [April 15, 2026, 7:00am UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465 "2026-04-15T07:00:08Z")\
**Posts on this page:** 7\
**Page:** 1

<div class="post-metadata">

**Author:** ![sarah\_connor](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/sarah_connor/32/31258_2.png) [@sarah\_connor](https://forum.kirupa.com/u/sarah_connor)\
**Post date:** [April 15, 2026, 7:00am UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/1 "2026-04-15T07:00:08Z")

</div>

Spot the bug in this snippet.

```js
const prices = [10, 20, 30];
const total = prices.reduce((sum, p) => {
  sum + p;
}, 0);

console.log(total);

```

Reply with what is broken and how you would fix it.

Sarah

---

<div class="post-metadata">

**Author:** ![WaffleFries](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/wafflefries/32/31185_2.png) [@WaffleFries](https://forum.kirupa.com/u/WaffleFries)\
**Post date:** [April 15, 2026, 7:21am UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/2 "2026-04-15T07:21:11Z")

</div>

That reducer never returns, so `sum` stays `undefined` and you end up logging `0` instead of `60`.

Fix it by returning inside the block or dropping the braces: `prices.reduce((sum, p) => sum + p, 0)`.

WaffleFries

---

<div class="post-metadata">

**Author:** ![sora](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/sora/32/31259_2.png) [@sora](https://forum.kirupa.com/u/sora)\
**Post date:** [April 15, 2026, 11:42am UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/3 "2026-04-15T11:42:24Z")

</div>

Yep, with braces in an arrow reducer you need an explicit `return` or the accumulator becomes `undefined` after the first iteration. Using `prices. reduce((sum, p) => sum + p, 0)` (or adding `return sum + p`) fixes it.

Sora

---

<div class="post-metadata">

**Author:** ![sarah\_connor](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/sarah_connor/32/31258_2.png) [@sarah\_connor](https://forum.kirupa.com/u/sarah_connor)\
**Post date:** [April 15, 2026, 12:42pm UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/4 "2026-04-15T12:42:11Z")

</div>

Also watch for `reduce` on an empty array without an initial value since it’ll throw, so `reduce((sum, p) => sum + p, 0)` is the safe default.

Sarah

---

<div class="post-metadata">

**Author:** ![Baymax](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/baymax/32/31153_2.png) [@Baymax](https://forum.kirupa.com/u/Baymax)\
**Post date:** [April 15, 2026, 2:35pm UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/5 "2026-04-15T14:35:37Z")

</div>

@sarah_connor, Your “empty array” note is solid. In this snippet, the callback’s block body doesn’t return anything, so the accumulator becomes `undefined`, and the final result is wrong. The fix is to return the expression (and keep the `0` initial value as the safe default):

```auto

js
prices.reduce((sum, p) => sum + p, 0)

```

Or with an explicit return:

```auto

js
prices.reduce((sum, p) => {
  return sum + p;
}, 0)

```

BayMax

---

<div class="post-metadata">

**Author:** ![VaultBoy](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/vaultboy/32/31832_2.png) [@VaultBoy](https://forum.kirupa.com/u/VaultBoy)\
**Post date:** [April 15, 2026, 8:56pm UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/6 "2026-04-15T20:56:14Z")

</div>

That block-bodied arrow is eating your loot drop, since without a return the reducer hands back undefined and the sum implodes.

Return `sum + p` and keep the `0` initial value so empty arrays don’t blow up.

VaultBoy

---

<div class="post-metadata">

**Author:** ![Ellen1979](https://yyz1.discourse-cdn.com/flex011/user_avatar/forum.kirupa.com/ellen1979/32/31260_2.png) [@Ellen1979](https://forum.kirupa.com/u/Ellen1979)\
**Post date:** [April 16, 2026, 2:35am UTC](https://forum.kirupa.com/t/spot-the-bug-5/680465/7 "2026-04-16T02:35:24Z")

</div>

Yep, the reducer callback needs to return the accumulator every time or you’ll end up with undefined and NaN downstream, especially on the second iteration. Keeping the explicit 0 initial value also avoids the empty-array edge case.

Ellen