kritik til min kode

Hov, det ser lidt rodet ud desværre.

Der er vist ikke så meget at påtale. Nydelig kode, nydelig programlogik. :^)

Jeg ville nok selv flytte definitionen af diverse variable ind i det scope de bruges: F.eks. kan "guess" flyttes ind i do-løkken. Det er heller ikke klart om der er nogen speciel grund til at have "max" - den bruges jo kun et sted.

Du kunne msåke overveje om det er Exception du skal catche eller om du bør være lidt mere specifik på hviloken type Exeptions der kunne komme.

Du bør være lidt mere omhyggelig med formateringen: Hvornår skal der være tomme linjer for at hjælpe med læsbarheden? Hvornår skal der være mellemrum før og efter {- og } tegn? Skal der være mellemrum efter komamer? Den slags ting...

Med kun én dag i faget synes jeg faktisk det er flot, men som nielle vil jeg sige, vent med at definere variabler til det er strengt nødvendigt, det gør det nemmere at læse koden

Sådan ville jeg skrue den sammen


import javax.swing.*;
import java.util.Random;
public class GuessGameGui {

    /**
    * @param args
    */
    public static void main(String[] args) {


        int max = 100;
        Random generator = new Random();
        int n = generator.nextInt(max);
        int count = 0;

       

        do {
            String guess = JOptionPane.showInputDialog(null, "Jeg tænker på et tal. Kan du gætte det?", "Gætte leg!", JOptionPane.QUESTION_MESSAGE);
            count++;
           
            if(guess == null) {
                break;
            }

            try{
                int guessInt = Integer.parseInt(s);

                if(guessInt > n) {
                    guess = JOptionPane.showInputDialog(null, "For højt. Prøv igen:", "Forkert", JOptionPane.QUESTION_MESSAGE);
                }
                else if(guessInt < n) {
                    guess = JOptionPane.showInputDialog(null, "For lavt. Prøv igen:", "Forkert", JOptionPane.QUESTION_MESSAGE);
                }
                else{
                    JOptionPane.showMessageDialog(null,"Korrekt!\nAntal forsøg: " + count);
                }
            }
            catch(NumberFormatException nfe){
                JOptionPane.showMessageDialog( nfe.getMessage() ,"Fejl!");
            }


        }while (guessInt != n);
       
        JOptionPane.showMessageDialog(null ,"Farvel!");
    }
}

Tak for hjælpen.
Lig et svar og point vil blive givet :-)

Svar :^)

Takke for point :^)

Jeg kan ikke rigtig lide din try-catch. den er lovlig omfattende. Såpvidt jeg kan se er grunden til den at brugeren kan gå hen og skrive noget andet end et tal. Men hvorfor abortere hele spillet af den grund når du kan nøjes med bare at bede ham indtaste et ordentligt tal ?

eg:
        do {
            bool ikkeEtTal = true;
            try{
                int guessInt = Integer.parseInt(s);
                ikkeEtTal = false;
            } catch( exception e ) {
                guess = JOptionPane.showInputDialog(null, "Du skal skrive et tal. Prøv igen.", JOptionPane.QUESTION_MESSAGE);
            }
        } while ( ikkeEtTal );
og så først bagefter teste om det indtastede tal er korrekt.

mvh JakobA

Ens IDE bør kunne reformatere ens kode til Java coding convention.

Eclipse: højre klik, source, format kan fixe en hel del

Tak for svarene. Jeg retter det straks! :-)