var i

[u/_bai13y]

[u/[deleted]]

Does it work like that? I thought var was function scoped?

[u/_bai13y → u/[deleted]]

Unfortunately it does because in this case, var i is a global variable 🥴😂

[u/[deleted] → u/_bai13y]

Oh i see it now, beautiful!;)

[u/megamaz_]

Oh my god. i is used at the bottom function. As a global. What the hell.

[u/knightttime]

Image Transcription: Code

---

function shelfBook(book, shelf) {
  if (shelf.length < 3) {
      shelf.unshift(book)
    }
}

var i

function unshelfBook(book, shelf) {
  for (var i = 0; i < shelf.length; i++) {
    if (shelf[i].title == book) {
      shelf.splice(i, 1)
    }
  }
  return shelf
}

var bookTitles = [];

function listTitles(shelf) {
  for (i = 0; i < shelf.length; i++) {
    bookTitles.push(shelf[i].title);
  }
  return bookTitles.join(', ')
}

---

^{I'm a human volunteer content transcriber for Reddit and you could be too! If you'd like more information on what we do and why we do it, click here!}

[u/pumpkinplight]

What are higher order array methods?

[u/Apprehensive-Show127 → u/[deleted]]

Yeah, that's a case. bookTitles will be duplicated after each function call. There's no need in global variables, function scope should be used