Почему не срабатывает коллбек в fs.readFile?

Question

[wufapexef]

Почему не срабатывает err что файл не существует (file не существует), а выбивает другую ошибку, связанную с кодом else

SyntaxError: Unexpected token u in JSON at position 0
    at JSON.parse (<anonymous>)

fs.readFile(file, 'utf8', (err, snapshot) => {
    if (err) {
      axios.get(url)
        .then((res) => {
          let firstSnapshot = { data: '', date: '' }
          let $ = cheerio.load(res.data)
          firstSnapshot.data = $(selector).html().replace(/\s/g, '')
          firstSnapshot.date = new Date(Date.now())
          fs.writeFile(file, JSON.stringify(firstSnapshot, null, 2), (err) => {
            return console.log('First snapshot file created!')
          })
        })
    }
    snapshot = JSON.parse(snapshot)
    console.log(snapshot.date)
  })

Как вообще лучше всего осуществить такую задачу: реквайрить или читать файл, но если он не существует, то создавать его и потом опять же читать?

Answer 1

[Виталий]

Потому что после проверки ошибки несуществующий файл пытаются распарсить:
snapshot = JSON.parse(snapshot)

Comments

[wufapexef]

Это понятно. Но почему он лезет в другое условие если срабатывать должно if (err)?

[Виталий]

wufapexef, какое другое условие? Я вижу только одно. Кажется, кто-то забыл обернуть парсинг в блок else :)
Получается, что код выполняет блок if(err), а затем продолжает выполнение.

[wufapexef]

Как мне вернуть данные из этого написанного файла?

[Виталий]

wufapexef, не понял вопрос

[wufapexef]

Я оговорился. На момент чтения файла не существет. Срабатывает err, Нода идет создавать файл. . Так почему же тогда она не ругает на то что snapshot не существует? Его ведь нету, раз JSON.parse(snapshot) происходит ДО создания файла.

[wufapexef]

Я так полагаю мне надо как-то ждать пока файл создастя.

[Виталий]

Конечно, изучи по-лучше принципы асинхронного программирования. Сперва выполняется

if (err)
      axios.get(url).then(...)

затем

snapshot = JSON.parse(snapshot)
console.log(snapshot.date)

после чего вызывалось бы
(res) => {...}
и в самую последнюю очередь
(err) => {...}

[wufapexef]

Вообщем я промисицировал fs с помощью Bluebird и с readFileAsync все заработало.