Avatar billede mlunde Nybegynder
14. oktober 2008 - 11:27 Der er 6 kommentarer og
1 løsning

Lille hjælp til loop

Hejsa.

Jeg har et program hvor man skal indtaste antal minutter, og så vises det, hvor i antal timer og resterende minutter. Dette fungerer også fint nok.
Når man indtaster et negativt tal, så skal den vise en fejlmeddelelse, hvilket den også gør. Men ved indtastning af et negativt tal beregner den også og viser svaret selvom det er negativt og det skal den ikke gøre.

Er der nogle der kan hjælpe med dette?

import java.util.Scanner;


public class AS62 {
    /**
    * author: Michael Lunde
    */   
  public static void main (String[] args) {

      int totalMinutes;
      int Hours;
      int remainMinutes;

     

      boolean waitingForInput = true;

     
      Scanner sc = new Scanner (System.in);
      do {
          System.out.println("Please enter an integer value of minutes:");
         
        if (sc.hasNextInt())  {
       
           
           
          totalMinutes = sc.nextInt ();
         
          Hours = (int)(totalMinutes/60);
          remainMinutes = (int)(totalMinutes % 60);
         
         
         
          if (totalMinutes >=0) 
              waitingForInput = false;
       
          System.out.printf ("The ammount of hours are %d", Hours);
          System.out.printf(" and the remaining ammount of minutes are %d\n", remainMinutes);
         
          if (totalMinutes<0)
            System.out.println("Please enter an integer equal or greater than zero.");         
           
      }
      else {
          sc.nextDouble();
          System.out.println("Invalid amount...please re-enter!");
      }
           
      }
      while (waitingForInput);
     
      } 
  }
Avatar billede erikjacobsen Ekspert
14. oktober 2008 - 11:45 #1
Du skal måske rette

  if (totalMinutes >=0)
              waitingForInput = false;
     
          System.out.printf ("The ammount of hours are %d", Hours);
          System.out.printf(" and the remaining ammount of minutes are %d\n", remainMinutes);

til

  if (totalMinutes >=0)  {
              waitingForInput = false;
     
          System.out.printf ("The ammount of hours are %d", Hours);
          System.out.printf(" and the remaining ammount of minutes are %d\n", remainMinutes);

    }
Avatar billede mlunde Nybegynder
14. oktober 2008 - 12:11 #2
Perfekt. Mange tak
Avatar billede erikjacobsen Ekspert
14. oktober 2008 - 12:23 #3
Og egentlig er det mere logisk at have beregningen

      Hours = (int)(totalMinutes/60);
          remainMinutes = (int)(totalMinutes % 60);

inden i if-sætningen, da de kun skal beregnes hvis tallet ikke er negativt.
Avatar billede arne_v Ekspert
14. oktober 2008 - 15:46 #4
Og saa er cast ikke noedvendigt !
Avatar billede erikjacobsen Ekspert
14. oktober 2008 - 16:57 #5
Og "ammount" staves "amount".
Avatar billede blackoak Nybegynder
24. oktober 2008 - 10:25 #6
Umiddelbart så jeg det samme som erikjacobsen, og det med at flytte beregningen ind i if sætningen er en god ide.

import java.util.Scanner;

public class MinutTest {

    public static void main(String[] args) {

        int totalMinutes, hours, remainMinutes;

        boolean waitingForInput = true;

        Scanner sc = new Scanner(System.in);
        do {
            System.out.println("Please enter an integer value of minutes:");

            if (sc.hasNextInt()) {

                totalMinutes = sc.nextInt();

                if (totalMinutes >= 0) {
                    waitingForInput = false;

                    hours = (int) (totalMinutes / 60);
                    remainMinutes = (int) (totalMinutes % 60);

                    System.out.printf("The amount of hours are %d", hours);
                    System.out.printf(" and the remaining amount of minutes are %d\n", remainMinutes);
                }
                if (totalMinutes < 0)
                    System.out.println("Please enter an integer equal or greater than zero.");
            } else {
                System.out.println("Invalid amount...please re-enter!");
            }
        } while (waitingForInput);
    }
}
Avatar billede mlunde Nybegynder
25. juli 2010 - 07:20 #7
Tråd slut
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Kurser inden for grundlæggende programmering

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester