a me pare corretto, magari metterei un break per non fare cose inutili, tipo:

alert(i + " si sovrappone a " + j)
break;