Regex doesn't return true in a loop

Question

I wrote simple regex to check few words and then put them into an new array. Everything works fine, when I check values manually, but when I put it into loop, I receive false. My question is - why? I think there's something wrong in my if statement, but for me it's ok... Here's my code:

var elements = ["sztuka", "sztuki", "sztuk", "sztukateria", "sztukmistrz", "sztuczka"];
var correctElements = [];
var reg = /^sztuk[ia]$/ig;

function checkElements(reg, elements) {
    for (var i=0; i < elements.length; i++) {
        if (reg.test(elements[i] == true)) {
            correctElements.push(elements[i]);
         } else {
             return false;
         }
    }

    console.log(correctElements);
}

When I check it manualy I receive true:

var reg = /^sztuk[ia]$/ig;
console.log(reg.test(elements[0]));

I would be very grateful if you could help me with that and explain why it's happening.

The actual issue here is the reg.test() closing round bracket. It should go after ] and not after true: if (reg.test(elements[i]) == true) { — Dmitry Egorov
– Dmitry Egorov, Commented Mar 22, 2017 at 14:51
This sounds very much like a duplicate of Why RegExp with global flag in Javascript give wrong results, but there are so many other issues here... — Bergi
– Bergi, Commented Mar 22, 2017 at 23:51

jdmdevdotnet · Accepted Answer · 2017-03-22 14:47:45Z

1

reg.test(elements[i] == true) need to be reg.test(elements[i]) == true)

answered Mar 22, 2017 at 14:47

jdmdevdotnet

1

Sign up to request clarification or add additional context in comments.

Comments

VHS · Accepted Answer · 2017-03-22 14:50:32Z

0

You shouldn't return false until you iterate through all the items. Check the following working snippet.

var elements = ["sztuka", "sztuki", "sztuk", "sztukateria", "sztukmistrz", "sztuczka"];
var correctElements = [];
var reg = /^sztuk[ia]$/ig;
checkElements(reg,elements);

function checkElements(reg, elements) {
    console.log("hi");
    for (var i=0; i < elements.length; i++) {
        if (reg.test(elements[i]) == true) {
            correctElements.push(elements[i]);
        }
    }

    console.log(correctElements);
}

answered Mar 22, 2017 at 14:50

VHS

10.2k3 gold badges25 silver badges48 bronze badges

Collectives™ on Stack Overflow

Regex doesn't return true in a loop

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related