betingelser i en if-sætning

prøv:

if ((kortnr.length() > 1) && (kortnr.length() < 8) && (kortnr.length()  > 8)) {
        JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
            }

if (kortnr.length() != 8) {
        JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
            }

Men du mener nok
if (kortnr.length() != 8) {

men kortnr.length() kan jo ikke både være mindre end 8 og større end 8!!

henni80>

Din version er ikke syntax mæssigt korrekt.

Det er:
  (variabel sammenligning værdi) && (variabel sammenligning værdi)

r9>

if ((kortnr.length() > 1) && (kortnr.length() < 8) && (kortnr.length()  > 8)) {

er syntax mæssig korrekt men vil aldrig være sand.

netop

Hvis det endelig skulle laves som en dobbelt sammenligning ville det være:

if ((kortnr.length() < 8) || (kortnr.length() > 8)) {

ok - kigger lige på det

henni80 har nok bare byttet rundt på AND og OR

Jeg smider lige lidt kode for at forklare hvad jeg gerne vil.

-----------------
while (kortnrInput.length() == 8) {
            if (kortnrInput.length() == 8) {
                elever.add(kortnrInput);
                kortnrInput =
                    JOptionPane.showInputDialog("Indtast elevens årskortnr");
            }

            if (kortnrInput.length() > 1 && < 8 && > 8) {
                JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
            }

            if (kortnrInput.length() ==1) {
                Collections.sort(elever);
                JOptionPane.showMessageDialog(
                    null,
                    "Følgende elever er bestået: \n" + elever);
            }
-------------

Det jeg gerne vil er:
1. når der er indtastet et kortnr på 8 karakter så skal man kunne indtaste endnu et
2. hvis man indtaster 1 karakter skal en resultat vises
3. hvis man taster 2-7 karakter eller over 8 skal det give en fejlmeddelse.

Derfor tror jeg ikke at jeg bare kan bruge !=8 for så dur 1 jo pludselig ikke...

og så er det disse sætninger det drejer sig om:

if (kortnrInput.length() > 1 && < 8 && > 8) {
                JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
            }

har skrevet således i stedet:

if ((kortnrInput.length() > 1) && (kortnrInput.length() < 8) || (kortnrInput.length() > 8)) {
            JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
            }

- lige endnu et spg. når jeg indtaster et ugyldigt kortnr så får jeg en fejlmeddelse hvilket er 10/4

men hvis man så skal have mulighed for at indtaste nogle flere kortnr, hvad skal jeg så gøre?

if ((kortnrInput.length() > 1) && ((kortnrInput.length() < 8) || (kortnrInput.length() > 8))) {

...hvis du vil gøre det sådan

Hvad med:

boolean more = true;
while(more) {
    kortnrInput = JOptionPane.showInputDialog("Indtast elevens årskortnr");
    int l = kortnrInput.length();
    if(l == 1) {
      Collections.sort(elever);
      JOptionPane.showMessageDialog(null, "Følgende elever er bestået: \n" + elever);
      more = false;
    } else if(l == 8) {
        elever.add(kortnrInput);
    } else {
        JOptionPane.showMessageDialog(null, "Ugyldigt årskortnr");
    }
}

Jeg gættede på at 1 betød exit.

Tak for hjælpen