0

sigo realizando pruebas, esta vez he intentado crear un programa que me localice el numero más grande de una tabla.
El programa está planteado con funciones y la tabla se rellena aleatoriamente, este es el codigo:

package recursividad;

import java.util.*;

public class maxTabla {

    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);

        int max;
        int t[];
        t = new int[6];
        max = max(t);

        System.out.println("------------------------------------------------------");
        System.out.println("Programa que localiza el numero mayor en una tabla");
        System.out.println("------------------------------------------------------");
        for (int i = 0; i < t.length; i++) {
            t[i] = (int) (Math.random() * 100 + 1);

        }
        System.out.println("La tabla es: ");
        for (int i = 0; i < t.length; i++) {
            System.out.println(t[i]);

        }

        System.out.println("El numero mas grande es: " + max);
    }

    public static int max(int t[]) {
        int max;

        max = t[0];
        for (int i = 0; i < t.length; i++) {
            if (t[i] > max) {
                max = t[i];
            }
        }

        return max;

    }
}

El problema es que siempre me dice que el valor máximo es "0"
Gracias de antemano, un saludo.

2 respuestas 2

3

El problema es que ejecutas la función antes de rellenar la tabla. Pon la llamada a la función max=max(t); después de rellenar la tabla con números.

package recursividad;

import java.util.*;

public class maxTabla {

public static void main(String[] args) {
    Scanner sc = new Scanner(System.in);

    int max;
    int t[];
    t = new int[6];
    

    System.out.println("------------------------------------------------------");
    System.out.println("Programa que localiza el numero mayor en una tabla");
    System.out.println("------------------------------------------------------");
    for (int i = 0; i < t.length; i++) {
        t[i] = (int) (Math.random() * 100 + 1);

    }

    max = max(t);
    System.out.println("La tabla es: ");
    for (int i = 0; i < t.length; i++) {
        System.out.println(t[i]);

    }

    System.out.println("El numero mas grande es: " + max);
}

public static int max(int t[]) {
    int max;

    max = t[0];
    for (int i = 0; i < t.length; i++) {
        if (t[i] > max) {
            max = t[i];
        }
    }

    return max;

}
3
  • Para que esto sea una respuesta válida, debes agregarlas modificaciones necesarias. No solo decirlas, al decirlas es más un comentario. Es correcto si, pero vago, si lo completas, tu respuesta será completa.
    – Excorpion
    el 25 nov. 2020 a las 12:17
  • Considero que en este caso no era necesario volver a poner el código porque la solución era bastante fácil y quedaba completamente determinada con el texto. Pero bueno, ya está hecho.
    – Miguel27
    el 25 nov. 2020 a las 12:25
  • Aunque sea fácil y obvio, la respuesta debe ser completa ;) Y gracias por agregarla
    – Excorpion
    el 25 nov. 2020 a las 12:39
1

El problema que tienes es muy simple. Siempre te dice que el valor máximo es 0 porque estás llamando a la función max() antes de tener cualquier numero dentro de la tabla.

max=max(t);
...
for (int i = 0; i < t.length; i++) {
     t[i] = (int) (Math.random() * 100 + 1);
}

Pon esa llamada después de rellanar la tabla:

for (int i = 0; i < t.length; i++) {
    t[i] = (int) (Math.random() * 100 + 1);
}
max = max(t);

Output:

------------------------------------------------------
Programa que localiza el numero mayor en una tabla
------------------------------------------------------
La tabla es: 
8
22
82
56
90
85
El numero mas grande es: 90
2
  • Muchisimas gracias. Revisé el codigo y lo vi. Estoy aprendiendo por mi cuenta y los ejercicios los "invento" entonces no tengo forma ver la correción. Gracias, amigo Mario
    – Jorch13_13
    el 25 nov. 2020 a las 17:17
  • Me alegro que te haya servido :)
    – DevMind
    el 25 nov. 2020 a las 17:21

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.