1

Quiero retornar el valor del console, pero no retorna nada.

Este es mi código:

function findOutlier(integers) {
  let odd = integers[0];

  integers.forEach(function(current) {

    if (odd % 2 == 0) {
      if (current % 2 > 0) {
        console.log(current);// ESTE VALOR
        return current;
      }
    } else {
      if (current % 2 == 0) {
        return current;
      }
    }
  });
}

console.log(findOutlier([2, 6, 8, 10, 3]));

Mi ejercicio es:

You are given an array (which will have a length of at least 3, but could be very large) containing integers. The array is either entirely comprised of odd integers or entirely comprised of even integers except for a single integer N. Write a method that takes the array as an argument and returns this "outlier" N.

Examples [2, 4, 0, 100, 4, 11, 2602, 36] Should return: 11 (the only odd number)

[160, 3, 1719, 19, 11, 13, -21] Should return: 160 (the only even number)

Trato de obtener estos resultados:

Test.assertEquals(findOutlier([0, 1, 2]), 1)
Test.assertEquals(findOutlier([1, 2, 3]), 2)
Test.assertEquals(findOutlier([2,6,8,10,3]), 3)
Test.assertEquals(findOutlier([0,0,3,0,0]), 3)
Test.assertEquals(findOutlier([1,1,0,1,1]), 0)
7
  • ¿Buscas obtener solo el primer resultado que cumpla la condición o todos?
    – Marcos
    Commented el 11 ene. 2018 a las 15:55
  • @Marcos exactamente!!!
    – hubman
    Commented el 11 ene. 2018 a las 15:56
  • ¿Exactamente cual de las 2?
    – Marcos
    Commented el 11 ene. 2018 a las 15:57
  • @Marcos el return esta en los dos casos, el algoritmo esta bien, lo que no entiendo es porque no retorna?
    – hubman
    Commented el 11 ene. 2018 a las 15:59
  • 1
    @egallardo, odd, es igual al primer elemento del arreglo.
    – Marcos
    Commented el 11 ene. 2018 a las 16:15

2 respuestas 2

3

El método forEach, sirve para:

ejecutar la función indicada una vez por cada elemento del array.
Valor devuelto: undefined.

Solución:

Para obtener solo el primer resultado que cumple la condición podrías usar find

Ejemplo:

function findOutlier(integers) {
  let odd = integers[0];

  return integers.find(function(current) {

    if (odd % 2 == 0) {
      if (current % 2 > 0) {
        return current;
      }
    } else {
      if (current % 2 == 0) {
        return current;
      }
    }
  }) || 0; // Si no encontramos ninguno
}

console.log(findOutlier([2, 6, 8, 10, 3]), 3);
console.log(findOutlier([0, 1, 2]), 1)
console.log(findOutlier([1, 2, 3]), 2)
console.log(findOutlier([2,6,8,10,3]), 3)
console.log(findOutlier([0,0,3,0,0]), 3)
console.log(findOutlier([1,1,0,1,1]), 0);


Update

Como ha comentado @egallardo, tu método no funciona para todos los casos.
Esta podrías ser una mejor solución:

function findOutlier(integers) {
  // Buscamos todos los numeros pares
  let even = integers.filter(a => a % 2 === 0);
  // Si solo encontramos 1, lo devolvemos, sino buscamos el impar
  return even.length === 1 ? even[0] : integers.find(a => a % 2);
}

console.log(findOutlier([2, 6, 8, 10, 3]), 3);
console.log(findOutlier([0, 1, 2]), 1)
console.log(findOutlier([1, 2, 3]), 2)
console.log(findOutlier([2, 6, 8, 10, 3]), 3)
console.log(findOutlier([0, 0, 3, 0, 0]), 3)
console.log(findOutlier([1, 1, 0, 1, 1]), 0);
console.log(findOutlier([9,2,8,6,4]), 9); // Caso comentado

5
  • porque debo poner doble return, en la funcion find y en las condiciones if
    – hubman
    Commented el 11 ene. 2018 a las 16:07
  • @hubman, el primero (eg: return integers.find(...), es para indicar el valor retornado por la función findOutlier. El segundo es para indicarle a la funcion find, cual es el valor correcto y que debe dejar de buscar. PD: El segundo return estaba, pero en una de las ediciones lo borraste
    – Marcos
    Commented el 11 ene. 2018 a las 16:10
  • si estaba pero lo ponia por costumbre, ahora ya aprendi el porque, muchas gracias!!!
    – hubman
    Commented el 11 ene. 2018 a las 16:15
  • @hubman Ten en cuenta que esta función no soluciona el problema. No va a encontrar el outlier en este caso: findOutlier([9,2,8,6,4]);
    – egallardo
    Commented el 11 ene. 2018 a las 16:41
  • @hubman, he actualizado mi respuesta
    – Marcos
    Commented el 11 ene. 2018 a las 18:57
1

Esto es más corto y funciona en todos los casos.

var findOutlier = (arr, f = 0) => {
 arr.forEach(c => c%2 === 0 ? ++f : f);
 return f > 1 ? arr.find(n => n%2 !== 0) : arr.find(n => n%2 === 0);
};



console.log(findOutlier([160, 3, 1719, 19, 11, 13, -21]));
console.log(findOutlier([9,2,8,6,4]));
console.log(findOutlier([2, 4, 0, 100, 4, 11, 2602, 36]));

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.