For 循环不能与比较运算符一起正常工作

Question

Having trouble getting my code to log Scooby Doo! (as testing as I move forward in my code) Instead I always get the alert!

I have the user entering data of numbers in an array eg. [1 2 3 4] and I am trying to get the alert to pop up when someone puts extra spaces in the array to try again.

And if they haven't added extra spaces to log Scooby Doo!

document.getElementById("button").addEventListener("click", myExcelFuns);

function myExcelFuns() {
    var userInputStr = document.getElementById("numbers").value;

    if (userInputStr) {
        console.log(userInputStr);
        userInputStr = userInputStr.trim();
        let userNumberArray = userInputStr.split(" ");
        console.log(userNumberArray);
        let result;


        let newArray = [];
        for (let i = 0; i < userNumberArray.length; i++) {  
            let newArrayValues = parseInt(userNumberArray[i]);
            newArray.push(newArrayValues);
        }    
        console.log(newArray);    

        //Here is the trouble
        let finalArray = [];
        if (newArray === Number && newArray != ""){
            for (let i = 0; i < newArray.length; i++) {
                console.log("Scooby Doo");
                finalArray.push(newArray);
            }
                
         } else {
             alert("Only one space between numbers please!")
         }
    }

Answer 1

This expression is wrong and it will never be true:

 if (newArray === Number && newArray != ""){


 if (newArray === Number && newArray != ""){
     for (let i = 0; i < newArray.length; i++) {
          console.log("Scooby Doo");
     // Here you'll push whole newArray everytime it runs, I guess it's wrong as well
          finalArray.push(newArray); 
      }
                
  } else {
      alert("Only one space between numbers please!")
  }

I've wrote your code using some features from ES6 to show you how easy is our life using these features.

 //Here you must pass a valid Id document.getElementById("myButton").addEventListener("click", myExcelFuns); function myExcelFuns() { var userInputStr = document.getElementById("numbers").value; if (userInputStr) { userInputStr = userInputStr.trim(); let userNumberArray = userInputStr.split(" "); //check if some value into arrya is not a number const isThereSomeInvalidNumber = userNumberArray.some(x => isNaN(x)) if(isThereSomeInvalidNumber){ alert("Only one space between numbers and valid numbers please;"). //here if it's invalid you can clear the input document.getElementById("numbers");value = "". } //map runs into your array and return a new array const newArray = userNumberArray.map(x => { return parseInt(x) }) console;log(newArray); } }

 <button id="myButton"> click me </button> <input type="text" id="numbers" />

I hope it can inspire you =)