Hallo, woran liegt der Fehler ?

[kaper28]

import javax.swing.JOptionPane;


public class java03_122 {


	public static void main(String[] args) {
		
		
	int jahr;
	boolean sjahr;

	jahr=Integer.parseInt(JOptionPane.showInputDialog("Bitte geben Sie eine Jahreszahl ein : "));
	boolean a=((jahr%4 ==0)&& (jahr%100 ==0));

	if (jahr%4 != 0)
	  
		sjahr=false;


	/*
	if ((jahr%4 ==0)&& (jahr%100 !=0))
	{
		sjahr=true;
	}	
	
	
	
    if ((a==true)&& (jahr%400==0))
		
    {
    	sjahr=true;
    }
  
  */
    if (sjahr = true)
    
    	System.out.println(jahr + " ist ein Schaltjahr");
    	 
    else
    
    		 System.out.println(jahr + " ist kein Schaltjahr"); 
    
    
	
	
	
	
	/*
	if (jahr%4 ==0);
	  
		if((jahr%4 ==0)&& (jahr%100 !=0));
	    
			if  ((a ==true) && (jahr%400 == 0))
			
	        	 System.out.println(jahr + " ist ein Schaltjahr");
		
	        	 else
	        	 
	        		 System.out.println(jahr + " ist kein Schaltjahr");		
	        	 
	*/
	}
}

if (jahr%4 != 0)

sjahr=false;

woran liegt der fehler ? Warum gibt mir die if abfrage nicht am ende das richtige resultat .? Die if abfrage lautet :

Eine zahl ist keine Schlatzahl wenn sie nicht durch 4 geteilt werden kann .

 

[Gelöschtes Mitglied 5909]

ich weiß nimmer genau wie die rechnung ging, aber hab noch folgendes auf der verstaubten platte gefunden

	public boolean isLeapYear(int year ) {
		return (year % 4 == 0) && (year % 400 == 0 || year % 100 != 0);		
	}

edit:

if(); mit semikolon ist böse btw und macht nix, des weiteren solltest du gerade als anfänger die geschweiften klammern immer setzten

 

[kaper28]

ich weiß nimmer genau wie die rechnung ging, aber hab noch folgendes auf der verstaubten platte gefunden

	public boolean isLeapYear(int year ) {
		return (year % 4 == 0) && (year % 400 == 0 || year % 100 != 0);		
	}

edit:

if(); mit semikolon ist böse btw und macht nix, des weiteren solltest du gerade als anfänger die geschweiften klammern immer setzten

Hallo rail , das habe ich verstanden .Wenn ich das jahr 1974 eingebe muß es normale heißen es ist kein Schlatjahr weil es nicht glatt durch 4 getteilt werden kann, aber tut es nicht ..

if (jahr%4 == 0)

sjahr=true;

warum funktioniert das nicht ? Wenn die Variable (jahr) glatt durch 4 geteilt wird dann ist sjahr true , es nimmt den wert true an.

if (sjahr = true)

System.out.println(jahr + " ist ein Schaltjahr");

else

System.out.println(jahr + " ist kein Schaltjahr");

und hier muß es die ausgabe

1974 ist kein Schlatjahr heißen weil 1974 nicht glatt durch 4 geteilt werden kann .

 

[Beni]

if (sjahr = true)

= ist eine Zuweisung
== ist ein Vergleich

Hier schreibst du also "setze sjahr auf true"... danach ist sjahr auf true :wink:

Es reicht wenn du schreibst

if (sjahr )

(sjahr ist schon ein Wahrheitswert, der wird durch ein "== true" nicht wahrer)

 

[kaper28]

import javax.swing.JOptionPane;


public class java03_122 {


	public static void main(String[] args) {
		
		
	int jahr;
	boolean sjahr;

	jahr=Integer.parseInt(JOptionPane.showInputDialog("Bitte geben Sie eine Jahreszahl ein : "));
	boolean a=((jahr%4 ==0)&& (jahr%100 ==0));
	
	if (jahr%4 == 0)
	  
		sjahr=true;


	
	//if ((jahr%4 ==0)&& (jahr%100 !=0))
//	{
	//	sjahr=true;
//	}	
	
	
	
    //if ((a==true)&& (jahr%400==0))
		
    //{
    //	sjahr=true;
   // }
  
 
    if (sjahr)
    
    	System.out.println(jahr + " ist ein Schaltjahr");
    	 
    else
    
    		 System.out.println(jahr + " ist kein Schaltjahr"); 
    
     
	
	
	
	
	
	
	        	 
	
	}
}

Hallo Beni ich habe gemacht nur der compiler gibt wieder fehler , the local variable sjahr has not been initilatied. Obwohl es oben schon deklariert ist

 

[Beni]

Deklariert ist nicht initialisiert.

Wenn du kein Schaltjahr hast, wird sjahr nie etwas zugewiesen, und kann deshalb auch nicht verwendet werden.

Entweder machst du noch irgendwo ein "sjahr = false", oder du könntest auch direkt "boolean sjahr = ((jahr%4) == 0);" schreiben.

 

[kaper28]

Hallo Beni, ich glaube ich habe es geschaft mit deiner hilfe .... Dieser Code Funktionert und rechnet die Jahre richtig habe ich eben getestet.

import javax.swing.JOptionPane;


public class java03_122 {


	public static void main(String[] args) {
		
		
	int jahr;
	boolean sjahr=true;

	jahr=Integer.parseInt(JOptionPane.showInputDialog("Bitte geben Sie eine Jahreszahl ein : "));
	boolean a=((jahr%4 ==0)&& (jahr%100 == 0));
	
	if (jahr%4 > 0 )
	  
		sjahr=false;

	
	if ((jahr%4 == 0)&& (jahr%100 > 0))
	
		sjahr=true;
		
	 if ((a==true)&& (jahr%400 > 0))
		
      	sjahr=false;

  

	 if (sjahr)
		    
	    	System.out.println(jahr + " ist ein Schaltjahr");
	    	 
	    else
	    
	     System.out.println(jahr + " ist kein Schaltjahr"); 
   
    
     
	
	
	        	 
	
	}
}