Statement consistently returning true

Question

Statement consistently returning true

I need assistance in verifying the accuracy of an input and moving on to the next word if it's incorrect.


"Apples",
"Bananas",
"Pears",
"Door",
"Towel",
"Computer",
];

var x = myArray[Math.floor(Math.random()*myArray.length)];

function clearbox(){
if (x = myInput.value){
    var x = myArray[Math.floor(Math.random();*myArray.length)]);
    document.getElementById('myInput').value = ''
    document.getElementById("word").innerHTML = x
} else {
    document.getElementById('myInput').value = ''   ;
}
};

It seems that the if statement is not working properly, could someone kindly assist me with this issue?

javascript arrays

Answer 1

Answer №1

Your conditional statement if (x = myInput.value) needs to use the comparison operator (==) instead of the assignment operator (=). Using an assignment operator will assign the value of myInput.value to x, causing it to always evaluate as true since any value greater than 0 is considered true. To correct this, modify your function like so:

function clearbox(){
    if (x == myInput.value){
        var x = myArray[Math.floor(Math.random()*myArray.length)]);
        document.getElementById('myInput').value = ''
        document.getElementById("word").innerHTML = x
    } else {
        document.getElementById('myInput').value = ''   ;
    }

Answer 2

Your conditional statement if (x = myInput.value) needs to use the comparison operator (==) instead of the assignment operator (=). Using an assignment operator will assign the value of myInput.value to x, causing it to always evaluate as true since any value greater than 0 is considered true. To correct this, modify your function like so:

function clearbox(){
    if (x == myInput.value){
        var x = myArray[Math.floor(Math.random()*myArray.length)]);
        document.getElementById('myInput').value = ''
        document.getElementById("word").innerHTML = x
    } else {
        document.getElementById('myInput').value = ''   ;
    }

Answer 3

Answer №2

It is recommended to use == instead of = as it will always return true (you can also consider using === for type checking):

if (x == myInput.value)

There seems to be a syntax error in this line (please remove the extra ;):

var x = myArray[Math.floor(Math.random()*myArray.length)]);

Answer 4

It is recommended to use == instead of = as it will always return true (you can also consider using === for type checking):

if (x == myInput.value)

There seems to be a syntax error in this line (please remove the extra ;):

var x = myArray[Math.floor(Math.random()*myArray.length)]);

Statement consistently returning true

Answer №1

Answer №2

Similar questions

Presented is the Shield UI JavaScript sparklines chart

Extract data from a multidimensional array using a one-dimensional array as a filter

Dynamic array of pointer variables in C++

The selectedIndex attribute is failing to function properly for the mat-tab-nav-bar tabs in Angular Material

Utilizing the HTML5 Pattern Attribute in Conjunction with Websockets

Tips for successfully transferring a concealed value within a form

What could be causing the issue with uglify not functioning properly with AngularJS content?

What's the best way to ensure an endless supply of copied elements with identical classes to the clipboard?

Tap on the child to reveal their parent

Flickering effects in THREE.js using Frame Buffer Objects

Prevent scrolling in AngularJS model popups

Evolution of table size

It is impossible for me to invoke a method within a function

Controlling multiple bx-sliders with a single pager using only JavaScript

Error in Node application: Cannot search for 'x' in 'y' using the 'in' operator with Express and Nunjucks

Invoking an AJAX function that is not inside the document.ready function

Manipulate array of observable data structure in Angular using RxJs filter function

Having difficulty retrieving information from a factory in AngularJS

Issue with MeteorJS: The template event fails to trigger when within a popover

Angular ng-repeat table displaying items in alternating columns