Question

I am trying to return false after a conditional statement fail.

I have

$('#btn').click(function() {
    $('.title').each(function() {
        if (id == $(this).attr('id')) {
            alert('The name already exists.')
            return false; //I hope my codes would stop here if condition is true
        }
    })
    // my codes still call the doSomething function even if the conditional         
    //statement is true
    doSomething();
})​

I want do call the doSomething function ONLY if id != $(this).attr('id).

The codes below gave me what I want but it seems ugly.

$('#btn').click(function() {
    var nameExist = false
    $('.title').each(function() {
        if (id == $(this).attr('id')) {
            alert('The name already exists.')
            nameExist = true;
            return false; //I hope my codes would stop here if condition is true
        }
    })
    if (!nameExist) {
        doSomething();
    }
})​

Anyone has a better approach for this? Thanks a lot!

Answer 1

You don’t need to iterate thrue the elements, you can get it by the id and class like, #myId.myClass.

$('#btn').click(function() {
    if($('#' + id + '.title').length) {
        alert('The name already exists.');
    } else {
        doSomething();
    }
});