if-else

[Happysheep99999]

Good evening,
i´m at the beginning of learning java and i have a question to a code i wrote which isn´t working like it should.
Following you´ll see it:

package Kapitel_4;
import java.util.Scanner;
public class Schrauben
{

public static void main(String[] args)
{
//Scanner
Scanner scan = new Scanner(System.in);
System.out.println("Bitte geben Sie den Durchmesser der Schraube ein.");
int durchmesser = scan.nextInt();
System.out.println("Bitte geben Sie die Länge der Schraube ein.");
int länge = scan.nextInt();
scan.close();
//Variablen boolean
boolean durchmesser1 = (0 < durchmesser < 3);
boolean durchmesser2 = (4 < durchmesser < 6);
boolean durchmesser3 = (7 < durchmesser < 20);

boolean länge1 = (0 < länge < 20);
boolean länge2 = (21 < länge < 30);
boolean länge3 = (31 < länge < 50);
//Verzweigung
if (durchmesser1 && länge1)
{
System.out.println("Typ 1");
}
else if (durchmesser2 && länge2)
{
System.out.println("Typ 2");
}
else if (durchmesser3 && länge3)
{
System.out.println("Typ 3");
}
else
{
System.out.println("Unbekannter Schrauebntyp");
}
}

}

Syntax error on token "<", << expected
Syntax error on token "<", << expected
Syntax error on token "<", << expected
Syntax error on token "<", << expected
Syntax error on token "<", << expected ...

... it says and means the things i highlighted.

I don´t understand why i need a << and a < isn´t working.

And if i write << the whole program isn´t working and the only thing you see on the console is "Typ 1".

i don´t understand.

please help me.

good night



post scriptum (p.s.):
I´m sorry that the program is in german. I´m from Germany. Maybe if you want you can answer in german.

 

[KonradN]

Da dies ein deutschsprachiges Forum ist, sollten wir auch Deutsch nutzen …

Als ersten Punkt: Code bitte immer in Code Tags posten. Dazu einfach den Knopf </> links oberhalb des Eingabebereiches nutzen.

Dann ist eine Operation wie a < b < c eine Zusammensetzung von Operationen, die dann von links nach rechts ausgewertet würden. Also erst a<b was dann true oder false wäre und dann eben dieser Boolean Wert < c was dann keinen Sinn mehr ergeben würde.

Daher sind das zwei Vergleiche und du willst wissen, ob beides wahr ist. Also ein a<b und b<c

Edit: Typo angepasst.

 

[KonradN]

Ach ja - was ich jetzt noch als offenen Punkt sehe: < vs. <<:

< ist ein Vergleichsoperator. Dieser vergleicht den linken mit dem rechten Wert um dann true oder false zurück zu geben.
a < b prüft also, ob a kleine b ist um dann entweder true (a ist kleiner) oder false (a ist nicht kleiner) zurück zu geben.

<< ist ein Shift Operator. Zahlen werden ja nur als sogenannte Bits gespeichert. 8 Bist sind dabei ein Byte und die Datentypen haben eine gewisse Grösse. Ein int hat so 4 Bytes, also 32 Bits. Davon ist ein Bit das Vorzeichen und die übrigen Bits geben dann den Wert an.
Mit dem Befehl a << b wird nun der Wert von a genommen und die Bits um b Stellen nach links verschoben.

Also wenn Du 1 << 2 hast, dann ist die 1 ja ein Wert ...0001. Nun werden die Bits um zwei Stellen nach links verschoben. Damit erhältst Du ...0100. 1 << 2 ergibt also wieder eine Zahl und zwar die 4.

Das erklärt dann auch den Hinweis. Wenn Du bei a < b < c aus einem < ein << machst, dann hast Du einen Vergleich von zwei Zahlen. Das funktioniert dann für den Compiler aber hat natürlich nichts mehr mit dem gewünschten Vergleich zu tun.

 

[White_Fox]

Also ein a<b und b<c

In Quellcode ausgedrückt:

boolean foo = a > b && b < c;

Und irgendwo wurde mal der Hinweis geben, Umlaute in Quellcode zu vermeiden.

Noch etwas, daß man nie früh genug lernen kann und auf die kein Tutorial eingeht: Schreib deinen Quellcode so, daß man ihn als Anleitung lesen kann. Dazu gehört, daß man z.B. auch englisch schreibt und auf deutsche Bezeichner verzichtet (es gibt hier zwar für bestimmte Spezialfälle Verfechter, die das anders sehen, aber die tun hier herstmal nichts zur Sache). Es klingt ja auch auf Deutsch unfassbar scheiße, wenn jemand in ein und demselben Satz mehrmals von Deutsch nach Englisch wechselt.

Wenn du Quellcode als menschenlesbaren Fließtext formulierst hast du zu allererst einmal den Vorteil, daß du dir Codekommentare sparen kannst.

• Weniger Schreibarbeit.
• Codekommentare müssen bei Änderungen nicht nachgezogen werden, fehlerhafte Kommentare führen dich nicht in die Irre.

Wenn du das Gefühl hast, du hast eine Menge Code geschrieben die durch ihre schiere Größe unübersichtlich wird: Methoden sind nicht nur für mehrfach aufgerufene Codestellen sinnvoll, sondern auch um "Sinnabschnitte" zu bilden. Wenn du im Grunde ein größeres Problem hast daß aber mit mehreren Handgriffen gelöst werden muß – pack es in eine Methode und gib ihr einen griffigen Namen. Du wirst dir damit in Zukunft die Fehlersuche oft stark vereinfachen.

Ich erlaube mir mal, deinen Code entsprechend umzuschreiben:

package Kapitel_4;

import java.util.Scanner;

public class Schrauben{
    private int scewDiameter;
    private int screwLength;
    private boolean screwIsSmallInDiameter;
    private boolean screwIsMiddleInDiameter;
    private boolean screwIsBigInDiameter;
    private boolean screwIsSmallInLength;
    private boolean screwIsSmaMiddleInLength;
    private boolean screwIsBigInLength;

   public static void main(String[] args){
        getScrewDiameterFromUser();
        categorizeScrew();
        printCategorizationResult();
    }
   
    private void getScrewSizeFromUser(){
        var userInputScanner = new Scanner(System);
        System.out.println("Bitte geben Sie den Durchmesser der Schraube ein.");
        screwDiameter = scan.nextInt();
        System.out.println("Bitte geben Sie die Länge der Schraube ein.");
        screwLength = scan.nextInt();
    }
   
    private void categorizeScrew(){
        screwIsSmallInDiameter = 0 < scewDiameter && screwDiameter < 3;
        screwIsMiddleInDiameter = 4 < scewDiameter && screwDiameter < 6;
        screwIsBigInDiameter = 7 < scewDiameter && screwDiameter < 20;
       
        screwIsSmallInLength = 0 < screwLength && screwLength < 20;
        screwIsSmaMiddleInLength = 21 < screwLength && screwLength < 30;
        screwIsBigInLength = 31 < screwLength && screwLength < 50;
    }
   
    private void printCategorizationResult(){
        if (screwIsSmallInDiameter && screwIsSmallInLength){
            System.out.println("Typ 1");
        }else if (screwIsMiddleInDiameter && screwIsSmaMiddleInLength){
            System.out.println("Typ 2");
        }else if (screwIsBigInDiameter && screwIsBigInLength){
            System.out.println("Typ 3");
        }
        else{
            System.out.println("Unbekannter Schrauebntyp");
        }
    }
}

Ich weiß, man könnte jetzt hergehen und sagen daß das am Anfang ja noch gar nicht so wichtig sei, ich sehe das aber etwas anders. Viele lernen es erst spät, und je später desto schmerzhafter wird die Lektion (weil dann der unbrauchbare Quellcode meist umso größer ist). Und nicht wenige lernen es nie. Dann doch lieber gleich von Anfang an richtig beibringen.

Ach ja...der Übung halber für dich habe ich noch einige Programmierfehler dringelassen, die aber auch schon vorher da waren. Die darfst du jetzt aber selber finden. Tipp: Du wirst sie auf jeden Fall finden, wenn du mal alle Schraubengrößen durchgehst. Und damit meine ich nicht sowas wie Schrauben mit M2,5-Gewinde oder so.

 

[Happysheep99999]

ja gut, das ist mir jetzt natürlich etwas peinlich, dass ich auf englisch geschrieben habe, aber ist ja auch egal. Auf jeden fall danke fürs antworten. Aber eine frage hätte ich noch: warum screwDiameter und nicht int diameter?

 

[Happysheep99999]

package Kapitel_4;
import java.util.Scanner;
public class Schrauben
{

public static void main(String[] args)
{
//Scanner
Scanner scan = new Scanner(System.in);
System.out.println("Bitte geben Sie den Durchmesser der Schraube ein.");
int diameter = scan.nextInt();
System.out.println("Bitte geben Sie die Länge der Schraube ein.");
int length = scan.nextInt();
scan.close();
//Variablen boolean
boolean diameter1 = (0 < diameter);
boolean diameter11 = (diameter <= 3);
boolean diameter2 = (4 <= diameter);
boolean diameter21 = (diameter <= 6);
boolean diameter3 = (7 < diameter);
boolean diameter31 = (diameter <= 20);

boolean length1 = (0 < length);
boolean length11 = (length <= 20);
boolean length2 = (21 <= length);
boolean length21 = (length <= 30);
boolean length3 = (31 <= length);
boolean length31 = (length <= 50);
//Verzweigung
if (diameter1 && diameter11 && length1 && length11)
{
System.out.println("Typ 1");
}
else if (diameter2 && diameter21 && length1 && length21)
{
System.out.println("Typ 2");
}
else if (diameter3 && diameter31 && length3 && length31)
{
System.out.println("Typ 3");
}
else
{
System.out.println("Unbekannter Schrauebntyp");
}
}

}

ich hab das jetzt so gemacht. geht auch oder? Oder sind da irgendwelchen großen fehler drinne?

 

[White_Fox]

ja gut, das ist mir jetzt natürlich etwas peinlich, dass ich auf englisch geschrieben habe, aber ist ja auch egal. Auf jeden fall danke fürs antworten. Aber eine frage hätte ich noch: warum screwDiameter und nicht int diameter?

Ehrlich gesagt: das habe ich aus dem Bauch heraus geschrieben. screwDiameter in Kombination mit screwLength drückt eine Zugehörigkeit der beiden Variablen aus.
Am Ende des Tages ist sowas immer abhängig vom restlichen Code. In deinem kleinen Beispiel würden diameter und length auch genügen. Wenn das Programm nun allerdigns größer ist: Durchmesser/Größe von was?

Stell dir vor du hättest den Code nicht selber geschrieben, müßtest diesen jetzt aber weiterentwickeln. Dann würden screwDiameter und screwLength weniger Fragen offenlassen. Und ich kann dir sagen: Wenn du ein eigenes Projekt ein Jahr lang nicht mehr angefasst hast und nach so langer Zeit mal wieder daran arbeiten willst, ist das eigentlich ganz genauso als hätte jemand anderes den Code geschrieben. Es ist echt keine Freude, vor seinem eigenen Projekt zu sitzen und nicht mehr zu verstehen, was man damals eigentlich gemacht hat.

Das Gemeine an der ganzen Sache ist: Es gibt eigentlich keine (oder kaum) Regeln, wie richtig guter Quellcode aussieht. Es gibt vielmehr jede Menge Merkmale, wie schlechter Code aussieht. Schlechter Quellcode ist der, an dem du nach einem Jahr Pause sitzt und fluchst, guter Code dagegen macht einfach nur keine Probleme.


Zu deiner zweiten Version: Bitte übe das mit den Codetags nochmal. Ansonsten hast du herausgefunden auf was ich dich aufmerksam machen wollte, sehr schön.

 

[mihe7]

warum screwDiameter und nicht int diameter?

There are only two hard things in Computer Science: cache invalidation and naming things.

-- Phil Karlton

 

[Oneixee5]

There are only two hard things in Computer Science: cache invalidation and naming things.

-- Phil Karlton

The Curmudgeon's Home Page

 

[meientau]

There are only two hard things in computer science: cache invalidation, naming things, and off-by-one errors.