Problema con metodi

[eagle87]

Per imparare a utilizzare i metodi in Java ho provato a fare un esercizio: una volta estratti n numeri casuali stampo quello massimo. Il problema è che il risultato del numero massimo è sbagliato.
Qualcuno mi saprebbe spiegare quale sia l'errore?
Grazie!

Questo è il codice:

package RipassoCasa;
import java.util.Scanner;


public class Funzioni {
    public static void main(String[] args) {
        Scanner tastiera = new Scanner(System.in);
        int n=0;
        int max=0;
        
        do                                                   //controllo che il numero inserito sia positivo
        {
            System.out.print("Quanti lanci? ");
        n = tastiera.nextInt();
            
            if(n<=0)
                System.out.println("Solo valori positivi");
            else
                break;
        }while(true);
        
        for (int i=0;i<n;i++)     
         
    System.out.println(numerocasuale()); //stampa dei numeri casuali richiamando il metodo
        
        for (int j=0;j<n;j++)
        
        {
        int numero=numerocasuale();       //assegno a una variabile il numero casuale
         if (numero>max)                           //calcolo massimo
             max=numero;
         
        }
        System.out.println("Massimo" +max);   //stampa massimo
        
    }
        
        
        public static int numerocasuale()                 //dichiarazione metodo
        {
             return (int)(1+ Math.random()*1000);
                  }    
}

[francesco.muia]

Il problema è che richiami 2 volte il metodo numeroCasuale, una per stamparlo e l'altro invece per calcolarti il max, ora il proble è che ogni invocazione di quel metodo ti viene generato un numero diverso quindi quello che stampi non è quello per il quale calcoli il max

[RoTeam]

Li generi due volte, una per stampare e una per confrontarli mentre devi farlo una volta sola

public static void main(String[] args) {
        Scanner tastiera = new Scanner(System.in);
        int n, max = 0;

        do
        {
            System.out.print("Quanti lanci? ");
            n = tastiera.nextInt();

            if(n<=0)
                System.out.println("Solo valori positivi");
            else
                break;
        }while(true);

        for (int j=0;j<n;j++) {
            int numero = numerocasuale();
            System.out.println(numero);
            if (numero>max)
                max = numero;
        }
        System.out.println("Massimo " + max); //stampa massimo
    }

    public static int numerocasuale() {
        return (int)(1+ Math.random()*1000);
    }

[Alex'87]

E comunque while(true) è bruttissimo da vedere

[eagle87]

[QUOTE=RoTeam;25276839]Li generi due volte, una per stampare e una per confrontarli mentre devi farlo una volta sola

Grazie mille!!

[eagle87]

E comunque while(true) è bruttissimo da vedere

Non può essere usato per un semplice controllo dei numeri inseriti?

[eagle87]

Il problema è che richiami 2 volte il metodo numeroCasuale, una per stamparlo e l'altro invece per calcolarti il max, ora il proble è che ogni invocazione di quel metodo ti viene generato un numero diverso quindi quello che stampi non è quello per il quale calcoli il max

Grazie