String comparison in Javascript not working

Question

for(i=0;i<contacts.length;i++){
    if((contacts[i].lname.toUpperCase().charAt(0))===(lastName.toUpperCase()))
        window.alert("already exists");
}

Am I doing correct? It is not working even if lastName already exists! Is there any other way to compare Strings in Javascript?

.charAt(0)? are you comparing first character with lastName? — Zakaria Acharki
– Zakaria Acharki, Commented Mar 4, 2016 at 17:11
You're comparing the first character lname to the entire lastName string. Remove the charAt call. — Chris Tavares
– Chris Tavares, Commented Mar 4, 2016 at 17:12

G3z · Accepted Answer · 2016-03-04 17:17:41Z

3

as pointed out i think it should be

for(i=0;i<contacts.length;i++){
    if( contacts[i].lname.toUpperCase() === lastName.toUpperCase() )
        window.alert("already exists");
}

answered Mar 4, 2016 at 17:17

G3z

6556 silver badges20 bronze badges

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

Comments

James Thorpe · Accepted Answer · 2016-03-04 17:19:11Z

0

It should be like:

for(i=0;i<contacts.length;i++){
            if((contacts[i].lname.toUpperCase())===(lastName.toUpperCase()))
                window.alert("Last name already exists");
            else
            contacts.push(person);  
        }

edited Mar 4, 2016 at 17:19

James Thorpe

32.3k6 gold badges76 silver badges95 bronze badges

answered Mar 4, 2016 at 17:18

Droidr

631 gold badge2 silver badges13 bronze badges

Collectives™ on Stack Overflow

String comparison in Javascript not working

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