Fehler beim Ausführen

[SilMil]

Hallo zusammen

Bin ziemlich neu am programmieren und wollte unten aufgeführtes Programm ausführen.
Nun kann ich bei der Ausgabe "Geschlecht" nichts ausfüllten. Wenn ich die beiden Linien fürs Geschlecht aber VOR der Abfrage fürs Alter schreibe, klappts - woran liegt das und wie kann ich das umgehen?

import java.util.Scanner;

public class Begruessung{

public static void main (String [] args) {

String Name, Vorname, Geschlecht;
int Alter;

Scanner sc = new Scanner (System.in);

System.out.print("Name: " );
Name = sc.nextLine();

System.out.print("\nVorname: ");
Vorname = sc.nextLine();

System.out.print("\nAlter: ");
Alter = sc.nextInt();

System.out.print("\nGeschlecht: ");
Geschlecht = sc.nextLine();

System.out.println();

if (Alter>= 18){

{if (Geschlecht == "m"){
System.out.println("Guten Tag Herr " + Name);
}else
System.out.println("Guten Tag Frau " + Name);
}
}
else
System.out.print("Hallo "+ Vorname);

 
}
}

 

[Tarrew]

System.out.print("\nAlter: ");
        Alter = sc.nextInt();
        sc.nextLine();

So sollte es passen. Das sc.nextInt() liest die Zahl ein aber nicht den Zeilenumbruch. Deswegen wurde beim nächsten nextLine() Aufruf nur der Zeilenumbruch gelesen und auf keine Eingabe gewartet.

Variablennamen solltest du übrigens klein schreiben.
Außerdem gibt es Code-Tags für Quellcode, dann kann man es auch besser lesen

 

[PoGoMoTo]

Hi SilMil,

schön mal wieder einen neuen Anfänger zu sehen^^ dachte schon ich wusel hier alleine herum

ich habe mich mal an dein Problem gesetzt. Das mit dem Wechsel zwischen nextLine() und nextInt() scheint wohl ein bekanntes Problem zu sein, und irgendwo habe ich auch mal den Grund gelesen...nur inzwischen schon wieder Vergessen

Ich löse das meist durch einen zweiten Scanner.

Außerdem überprüfst du das Geschlecht mit den == Operatoren, das geht nicht bei Strings. Für Strings wird der [String].equals("...") Befehl benutzt.

Ich hoffe ich konnte das Verständlich Schreiben.


Hier mal die korrigierte Version:

import java.util.Scanner;

public class Begruessung {
  public static void main(String[] args) {
  
    String name, vorname, geschlecht;
    int alter;
  
    Scanner sc = new Scanner(System.in);  // Scanner für String
    Scanner si = new Scanner(System.in);  // Scanner für Integer
  
    System.out.print("Name: ");
    name = sc.nextLine();
  
    System.out.print("\nVorname: ");
    vorname = sc.nextLine();
  
    System.out.print("\nAlter: ");   // der wechsel zwischen nextLine() und
    alter = si.nextInt();            // nextInt() macht ab und an Probleme
                                     // löse das oft durch einen zweiten Scanner.
  
    System.out.print("\nGeschlecht: ");
    geschlecht = sc.nextLine();
  
    if (alter >= 18) {
    
      if (geschlecht.equals("m")) {                   // hier ist noch ein fehler
        System.out.println("Guten Tag Herr " + name); // um einen String zu vergleichen     
      } else {                                        // musst du .equals("...") verwenden.
        System.out.println("Guten Tag Frau " + name);
      }
    } else {
      System.out.println("Hallo " + vorname);
    }
  }
}

 

[thecain]

Der Wechsel macht nicht "ab und an Probleme", sondern nextInt() geht einfach nicht in newLine...

Also entweder noch ein sc.nextLine() vor dem nextInt() oder immer nextLine und ein Integer.parseInt(sc.nextLine()) und eine NumberFormatException handeln.

Ich würde den zweiten Weg bevorzugen. Dann brauchst du auch keinen zweiten Scanner....

 

[Tarrew]

Man sollte ohnehin niemals zwei Scanner für den selben InputStream benutzen:

https://www.securecoding.cert.org/c...wrappers+on+a+single+byte+or+character+stream

 

[SilMil]

danke für eure Antworten, hat mir doch sehr geholfen