JSON.parse() w JavaScript

Question

Witam. Napisałem prostą aplikację, która losuje liczbę, a użytkownik musi ją odgadnąć. Dodałem również funkcję, która wyświetla ostatnie wyniki. Chciałem, aby te wyniki były przechowywane w localStorage. Problem pojawia się, gdy chcę je sparsować poprzez JSON.parse() do Array. Kod znajduje się poniżej.

$(() => {
  const prompt = $('.prompt');
  const answer = $('.answer');
  const prev = $('.prev');
  let isTrue = false;
  let prevAnswers = [];
  if (localStorage.getItem('prevAnswers')) {
    let prev = localStorage.getItem('prevAnswers');
    prevAnswers = JSON.parse(prev, null);
  }
  let number = parseInt(Math.random()*100);
  
  setInterval(() => {
    const value = answer.val();

    if(isNaN(value)) {
      prompt.text(`${value} isn't a number!`);
    } else {
      if(value != '') {
        if(value < number) {
          prompt.text(`${value} is smaller than number.`);
        } else if(value > number) {
          prompt.text(`${value} is greater than number.`);
        } else if(value == number){
          if (isTrue == false) {
            isTrue = true;
            prompt.text(`${value} is equal to number! Bravo!`);
            setTimeout(() => {
              prompt.text('In 5 seconds I will draw the next number and clear the answer input. In that time I will show the last number below. Have a nice game!');
            }, 2000);

            setTimeout(() => {
              prevAnswers.push(value);
              answer.val('');
              console.log(prevAnswers);
              number = parseInt(Math.random() * 100);

              prev.html('<h2>Previous answers:</h2>');
              for (let i = 0; i < prevAnswers.length; i++) {
                if(i == prevAnswers.length-1) {
                  prev.html(`${prev.html()}${prevAnswers[i]}.`);
                } else {
                  prev.html(`${prev.html()}${prevAnswers[i]}, `);
                }
                
              }
              isTrue = false;
            }, 5000);
          }
        }
      } else {
        prompt.text(`You didn't type a answer. Please type number in input!`);
      }
    }
  }, 100);
});

Problemem jest fakt, że konsola wywala mi błąd związany z 2 argumentem funkcji JSON.parse(), który nie jest mi potrzebny.

Answer 1

a co chcesz osiagnac tym nullem jako drugi argument? daj po prostu JSON.parse( data );

Comments

W przypadku braku null'a dzieje się to samo:

Dodam, że w JSON.parse(prev); nie ma żadnego przecinka ani znaku, który mógłby ten błąd powodować.

---

a napewno masz tam poprawnego jsona? daj sobie to parsowanie w try catch i w bloku catch wyrzuc na console co siedzi w tym localStorage... bez chocby najprpstszego zdebugowania można tylko zgadywac.

---

Konsola pokazuje, że w localStorage siedzi napis "52,24".

---

no wiec nie jest to json, po co chces to parsowac? Metoda string.prototype.replace zmien przecinek na kropke i nastepnie np. parseFloat aby dostac typ number.

---

Dziękuje bardzo. Użyłem metody split(',') do utworzenia z napisu tablicy i wszystko działa.

---

można i split, wszystko zależy co dalej chcesz z tym zrobić, stosując moją metodę od razu dostajesz liczbę:

parseFloat('52,24'.replace(/,/, '.')); // 52.24

 

---

Właśnie problem był tego typu, że wartości 52 i 24 były oddzielnymi liczbami, które użytkownik odgadł w poprzedniej turze i musiałem zachować je obydwie.

---

no to szczerze mówiąc przecinek nie jest dobrym separatorem... widzisz, ja myślałem, że chodzi o liczę float i pewnie wiele osób tak pomyśli... kod trzeba zawsze pisać z myślą o innych programistach, którzy mogą na nim pracować.

Ja bym tu dał więc po prostu tablicę:

const numbers = [52, 54];
const json = JSON.stringify(numbers);
JSON.parse(json); // [52, 54]

i masz wtedy tablicę, która od razu przechowuje typy number bez żadnych dodatkowych kombinacji. Według mnie jest to dużo czytelniejsze rozwiązanie.

---

Zrobiłem tak samo jak Pan i wszystko teraz działa. Dziękuję za pomoc