Ese código no es correcto para lo que quieres. Por ejemplo comparas un item del array con el array resultado completo, ni le veo el sentido ni es correcto, luego haces el return dentro del bucle, no es lógico en tu problema.
Te doy un par de soluciones.
La más elemental, y a mi juicio más sencilla de entender
function duplicar (array){
for (let i = 0; i < array.length; i++) {
array[i] = array[i]*2 ;
}
return array;
}
document.getElementById("demo").innerHTML = duplicar ([1,3,4,5]);
Aunque en realidad el return está de más, porque el array en el argumento se pasa por referencia (la función recibe la dirección del array), no por valor.
function duplicar (array){
for (let i = 0; i < array.length; i++) {
array[i] = array[i]*2 ;
}
}
let lista = [1,3,4,5];
duplicar(lista);
document.getElementById("demo").innerHTML = lista;
Otra usando el método map del objeto array
function duplicar (lista){
return lista.map((v)=>v*2);
}
lista = [1,3,4,5];
document.getElementById("demo").innerHTML = duplicar(lista);
Realmente esta última sería más sencilla aún:
lista = [1,3,4,5];
document.getElementById("demo").innerHTML = lista.map(function(v){return >v*2;});
Te pongo la función del map descrita con => y con function, para que veas dos formas de los mismo. Ambas soluciones son iguales.
El método map crea otro array a partir de cada elemento del array original. El argmento de map es una función que devuelve cada elemento del array original ya transformado. Esta función recorre el array y puede llevar tres argumentos: valor del elemento actual del array, índice actual y array completo. En este caso como solo se necesita el valor solo uso el primero.