Ayuda con Código Sencillo

[LaLakers94]

Hola, estoy empezando con Java y he empezado con actividades muy sencillitas, al menos me lo parecen hasta que me doy cuenta de que no sé hacerlas. Las intento probar mediante consola y no funcionan, no sé en que puedo fallar. Si me pudiesen echar una mano, lo agradecería de corazón.

Son 2 actividades:

1) Un método calcularNumeroArmonico tal que, dado un número entero n, devuelva el sumatorio 1 + 1/2 + 1/3 + ... + 1/n. Este es mi código:

Código (javascript) [Seleccionar] Expandir

public static Double numeroArmonico (Integer n){
Double numero = 0.0;
for (int i = 1; i <= n; i++){
numero += (1/i);
}
return numero; }

2)Un método que, dado un número entero n, devuelva la suma de todos sus dígitos. Por ejemplo, 12345 devolvería 15. Este es mi código:

Código (javascript) [Seleccionar] Expandir

public static Integer sumaDigitos (Integer n){
Integer suma = 0;
String numero = n.toString();

for (int i = 0; i < numero.length(); i++) {
suma += Integer.valueOf(numero.charAt(i)); }

return suma; }

En el primero pruebo con n=3 y me devuelve 1.0
En el segundo, con 12345, me devuelve 255  

¿Qué error he cometido?

[egyware]

El primero es que haces esto

Código (java) [Seleccionar] Expandir

int divisor = 10;
int numero = 5/divisor;

Según nuestros conocimientos de matemática, el resultado debería ser 0.5. Sin embargo el resultado es 0. Pero dirán "es que estamos almacenando el numero en un entero" Y ES CIERTO entonces hacemos las modificaciones necesarias.

Código (java) [Seleccionar] Expandir

int divisor = 10;
float numero = 5/divisor;

Sin embargo "numero" sigue siendo 0.
Lo que sucede realmente es que el compilador ASUME que estás haciendo una división entera debido que el divisor es entero. Para evitar que esto ocurra el divisor debe ser float o doble (puedes hacer un cast a float/double o declarar divisor como float/doble.)

Lo segundo NO TENGO IDEA PORQUE NO TE FUNCIONA, pero podrías intentar imprimir en pantalla que es lo que te devuelve "numero.charAt(i)" y luego  Integer.valueOf(numero.charAt(i)) para confirmar si está haciendo lo que debe.

Bueno Saludos! espero haberte ayudado.

[LaLakers94]

Hola, muchas gracias por las respuestas.

He podido solucionar el primero y en el segundo he mostrado por pantalla lo que comentas.

Código (javascript) [Seleccionar] Expandir


Integer n1 = 12345;
String numero = n1.toString();

System.out.println(numero.charAt(1));
System.out.println(Integer.valueOf(numero.charAt(1))); //Devuelve 50 - Ni idea de por qué.

La línea 4 devuelve 2. (Parece que funciona).
La línea 5 devuelve 50 (Ni idea de por qué).
Como ''numero.charAt(i)'' devuelve el dígito correspondiente al índice i, he reorganizado el código eliminando el ''Integer.ValueOf()'' y me sigue dando mal, de hecho, es el mismo resultado erróneo (255).

[Mitsu]

Hola. Al parecer, la conversión del char primitivo a int está devolviendo su valor ascii (creo).

Al parecer hay que utilizar el método de la clase Character getNumericValue, que devuelve el valor numérico (Integer) de ese número extraido en tipo char:

Código (=java) [Seleccionar] Expandir


suma += Integer.valueOf(Character.getNumericValue(numero.charAt(i)));


O puedes primero transformar ese char a String y luego convertirlo a Int:

Código (=java) [Seleccionar] Expandir


suma += Integer.valueOf(String.valueOf(numero.charAt(i)));


El código quedaría de éste método quedaría así:

Código (=java) [Seleccionar] Expandir


public static Integer sumaDigitos (int n){
    Integer suma = 0;
    String numero = String.valueOf(n);
     
      for (int i = 0; i < numero.length(); i++) {
        suma += Integer.valueOf(Character.getNumericValue(numero.charAt(i)));
      } 

    return suma;
  }


[LaLakers94]

Muchísimas gracias a ambos, ahora entiendo. Ya había optado por hacer otro método que se me había ocurrido (Ir dividiendo el entero entre 10 sucesivamente e ir sumando los restos) pero aplicaré este para no olvidarlo.

Gracias de nuevo.

[Mitsu]

Igual te dejo el code por si te sirve:

Código (=java) [Seleccionar] Expandir


import java.util.Scanner;
import java.text.DecimalFormat;

public class OperacionesMatematicas {
 
  public static Double numeroArmonico (int n){
    double numero = 1.0;
    for (int i = 2; i <= n; i++){
      numero += (double) (1.0/i);
    }
   
    return numero;
  }
 
  public static Integer sumaDigitos (int n){
    int suma = 0;
    String numero = String.valueOf(n);
     
      for (int i = 0; i < numero.length(); i++) {
        suma += Integer.valueOf(Character.getNumericValue(numero.charAt(i)));
      } 

    return suma;
  }
 
  public static void main(String... args) {
   
    OperacionesMatematicas op1 = new OperacionesMatematicas();
    Scanner into = new Scanner(System.in);
    Object resultado = 0;
    int numero = 0;
    int choice;
   
    do {
    System.out.println("¿Que operacion desea realizar?");
    System.out.println("[+] 1 - Numero Armonico de un numero");
    System.out.println("[+] 2 - Suma de Digitos de un numero");
    choice = into.nextInt();
    if(choice != 1 && choice != 2) {
      System.out.println("Opcion no disponible. Intente nuevamente.");
    }
    } while (choice != 1 && choice != 2);

   
    if (choice == 1) {
      System.out.println("Ingrese el numero.");
      numero = into.nextInt();
      DecimalFormat df = new DecimalFormat("#,#,##0.00"); // para formatear un real (2 decimales)
      System.out.println("Numero armonico de " + numero + ":\t" + df.format(op1.numeroArmonico(numero)));
    }
    else if (choice == 2) {
      System.out.println("Ingrese el numero.");
      numero = into.nextInt();
      System.out.println("Suma de los digitos de " + numero + ":\t" + op1.sumaDigitos(numero));
    }

    // System.out.print(Character.getNumericValue('%')); // devuelve -1 si le pasamos un char que no represente un numero
   
  }
 
}




Un saludo.



PD: No uses los Wrapper de los primitivos si no es necesario. Puedes usarlos con Listas, o para utilizar metodos que estan en los Wrappers. Para lo sencillo, los primitivos van bien.