Já foi dito a maior parte nas outras respostas. Deixo uma outra resposta com algumas coisas ainda não ditas.
Nota que quando usas for (id = 0; id
estás a declarar essa variável globalmente e podes estar a sobrescrever valores nas outras variáveis que possam ter o mesmo nome. Junta sempre var
em for (var i = ...
.
No teu código estás a criar um objeto com index numerário. Nesse caso vais ter algo como
{
0: 'string',
1: 'outra string',
etc...
e sendo assim parece-me mais semântico (ie mais correto) mapear a string inicial. Deixo também uma sugestão com regex, que dispensa o .indexOf
e uppercase
.
Nesse caso o código que queres poderia ser assim:
$scope.verificaVogal = function(nome) {
var vogais = ['A', 'E', 'I', 'O', 'U'];
return nome.split('').map(function(letra) {
return vogais.indexOf(angular.uppercase(letra) != -1);
});
}
ou assim:
$scope.verificaVogal = function(nome) {
return nome.match(/[aeiou]/i);
}
Nota: se usares a primeira variante, eu prefiro assim:
(function(){
var vogais = ['a', 'e', 'i', 'o', 'u'];
$scope.verificaVogal = function(nome) {
return nome.split('').map(function(letra) {
return vogais.indexOf(letra.toLowerCase()) != -1);
});
}
})();
para diminuir sempre que possível código não nativo e para não declarar constantes dentro da função.