Return aus For-Schleife

[Punk91]

Hey ho,
ich habe hier ein Problem.

boolean vierhundert(Date date1)
	{
		
		for(int i = 1800; i<999999999 ;i=i+400)
		{
		if(i==jahr)
		{return true;}
		else
		{return false;}
		}
	}

Der zeigt mir an, dass die "Return-Anweisung" fehlt. Ich komme einfach nicht weiter...

Habe es auch schon versucht mit einer Variablen zu lösen, aber irgendwie erkennt der keine Variable innerhalb der "For-Schleife" und sagt dann immer Variable nicht Initialisiert.

Ich hoffe ihr könnt mir helfen, danke im Voraus.

 

[Jango]

Hat nichts mit der Schleife zu tun. Deine Methode hat einen Rückgabewert vom Typ boolean. Also musst du auch solch einen Typ zurückgeben.

 

[ARadauer]

der code macht sowieso keinen sinn. du überprüfst 1800 und lieferst true oder false zurück... wozu die schleife?
der compiler ist nicht so schlau, dass es sich sicher ist dass du überhaupt in die schleife gehst, was wäre wenn du gar nicht rein gehst, was gibst du dann zurück?

 

[Punk91]

Habe ich doch, in Zeile 7 und Zeile 9.
"return true;" und "return false;"

Normal sollte das doch eigentlich reichen oder etwa nicht?


Also das ist der ganze Code

public class Date
{
int tag, monat;
static int jahr;

public Date(int day, int month, int year)
{
	tag = day;
	monat = month;
	jahr = year;
}

	public static void main(String[] args)
	{
	boolean a, b, c;
	Date date1 = new Date(1,1,2100);
	a = date1.vier(date1);
	b = date1.hundert(date1);
	
	}
	
	boolean vier(Date date1)
	{
		if(jahr%4==0)
		{return true;}
		else
		{return false;}
	}
	
	boolean hundert(Date date1)
	{
		if(jahr%100==0)
		{return true;}
		else
		{return false;}
	}
	
	boolean vierhundert(Date date1)
	{
		
		for(int i = 1800; i<999999999 ;i=i+400)
		{
		if(i==jahr)
		{return true;}
		else
		{return false;}
		}
	}
	
}

 

[darekkay]

DerCompiler erkennt nicht, dass man niemals aus der Schleife rauskommt. Du müsstest ein return false (oder true) NACH der Schleife hinzufügen.

Die Schleife an sich macht aber auch überhaupt keinen Sinn, denn sie wird nur ein einziges mal durchlaufen

 

[VfL_Freak]

Moin,

boolean vierhundert( Date date1 )
{
    for( int i = 1800; i<999999999; i=i+400 )
    {
        if( i==jahr )
        {
            return true;
        }
        else
        {
            return false;
        }
    }
// HIER fehlt das "return" 
}

BTW: was ist jahr ??
BTW-2: zu spät :-(

Gruß
Klaus

 

[faetzminator]

Wenn du bereits 4 und 100 herrlich mit Modulo an Stelle der langsamen Schleife löst, warum denn nicht 400 auch?
[c]return jahr % 400 == 0;[/c]

 

[SlaterB]

ansonsten noch: Date gibts in der API, keine eigene Klasse so nennen

> date1.vier(date1);
das Objekt X an eine Methode von X zu übergeben macht selten Sinn,
der Parameter wird in den Methoden auch gar nicht verwendet

edit:
rein inhaltlich neben verrückter Umsetzung, Prüfung unnötig bis in ferne Zukunft:
von 1800 alle 400 Jahre wäre 1800, 2200, 2600 usw.
das Jahr 2000 war aber gerade eines dieser 400er Jahre..
(oder doch nicht? ich schau nochmal nach edit: doch richtig erinnert)

 

[Punk91]

Ja, aber wenn ich jetzt unten das return setze, dann gibt er ja immer das zurück was ich dort eintrage.

Ich will ja, dass wenn das Jahr (jahr) ein Jahr ist was auf 1800 + 400 liegt sprich 2200, 2600, 3000, etc.

Warum kommt der nie aus der Schleife raus? die Schleife wird doch nur solang durchlaufen wie i kleiner als 999999999 ist.

 

[Plopo]

Edit: Zu spät.

 

[Landei]

Die zusätzliche Bedinung mit 1800 kannst du ganz einfach dazuschreiben:

[c]return jahr >= 1800 && jahr % 400 == 200;[/c]

Dir ist aber schon klar, dass die Jahre für richtige Schaltjahrregel 1600, 2000, 2400 sind? Das wäre

[c]return jahr >= 1600 && jahr % 400 == 0;[/c]

Das Muster [c]if(blubb) return true else return false;[/c] kann einfach durch [c]return blubb;[/c] ersetzt werden, ebenso [c]if(blubb) return false else return true;[/c] durch [c]return ! blubb;[/c].

 

[faetzminator]

Ich will ja, dass wenn das Jahr (jahr) ein Jahr ist was auf 1800 + 400 liegt sprich 2200, 2600, 3000, etc.

Warum kommt der nie aus der Schleife raus? die Schleife wird doch nur solang durchlaufen wie i kleiner als 999999999 ist.

Richtig. D.h., wenn das Datum dem aktuellen [c]i[/c] (in der Schleife) übereinstimmt, soll [c]true[/c] zurückgegeben werden. Andernfalls soll er nach Abarbeitung der Schleife [c]false[/c] zurück geben.

 

[Punk91]

Ja ich seh gerade dass es anstelle von 1800 eigentlich 1600 heißen müsste.

@Landei das verstehe ich gerade nicht

 

[Plopo]

Landei hat dir nur eine vereinfachte schreibweise gepostet.
Es müsste dann Komplett folgend aussehen:

  boolean vierhundert (Date date1)
  {
    return jahr % 400 == 0 && jahr > 1600;
  }

Oder gibt es einen bestimmten Grund, weshalb du nicht den Modulo (%) verwenden möchtest wie zuvor?

 

[Punk91]

Es will einfach nicht funktionieren. Jetzt hab ich es so angepasst dass es Sysntaktisch läuft, aber aus irgendeinem Grund gibt er immer falsche ergebnisse aus...

public class Date
{
int tag, monat;
static int jahr;

public Date(int day, int month, int year)
{
	tag = day;
	monat = month;
	jahr = year;
}

	public static void main(String[] args)
	{
	boolean a, b, c;
	Date date1 = new Date(1,1,2100);
	a = date1.vier(date1);
	b = date1.hundert(date1);
	c = date1.vierhundert(date1);
	
	if(a==true)
	{
		if(b==true)
		{System.out.println("Schaltjahr");}
		else
		{
			if(c==true)
			{System.out.println("Schaltjahr");}
			else
			{System.out.println("Kein schaltjahr");}
		}
	}
	else
	{System.out.println("Kein Schaltjahr");}
	
	}
	
	boolean vier(Date date1)
	{
		if(jahr%4==0)
		{return true;}
		else
		{return false;}
	}
	
	boolean hundert(Date date1)
	{
		if(jahr%100==0)
		{return true;}
		else
		{return false;}
	}
	
	boolean vierhundert(Date date1)
	{
		
		for(int i = 1600; i<999999999 ;i=i+400)
		{
		if(i==jahr)
		{return true;}
		else
		{}
		}
		return false;
	}
	
}

Jap weil es alle 400 Jahre ein schaltjahr gibt, aber wenn ich es mit modulo mache ist 2100 auch ein schaltjahr.

 

[Punk91]

Hab den Fehler gefunden, in Zeile 23 muss man true durch false ersetzen dann gehts.

Vielen Dank für eure hilfe, echt top wie schnell das ging und wie engagiert hier die Leute sind.

 

[Plopo]

Also bei mir gibt er mit "false" zurück bei "2100"

public class Test
{
  public static void main (String[] args)
  {
    System.out.println (vierhundert(2100));
  }

  static boolean vierhundert (int jahr)
  {
    return jahr % 400 == 0 && jahr > 1600;
  }
}

Wiedermal zu lange gebraucht

 

[turtle]

Merkwürdiger Code:shock:

Bei meiner IDE sagt er bei solchen Methoden:

boolean vier(Date date1)
    {
        if(jahr%4==0)
        {return true;}
        else
        {return false;}
    }

The value of the parameter date1 is not used.

Und Recht hat Eclipse. Warum übergibst Du Daten, wenn die Methode damit nichts anfangen will?

Räum erst mal auf!

 

[OTB]

Hab den Fehler gefunden, in Zeile 23 muss man true durch false ersetzen dann gehts.

Vielen Dank für eure hilfe, echt top wie schnell das ging und wie engagiert hier die Leute sind.

Die gleiche Aufgabe musste ich vor kurzem auch lösen.
Mein Ergebnis sah so aus:

public boolean istSchaltjahr(int jahr) {
    // falls jahr ohne rest durch 400 teilbar ist, dann ist das jahr ein Schaltjahr
    if(jahr % 400 == 0) {
        return true;  // die funktion liefert true zurück und wird beendet
    }
    // falls jahr ohne rest durch 100 teilbar ist, dann ist das jahr kein Schaltjahr
    if(jahr % 100 == 0) {
        return false; // die funktion liefert falls zurück und wird beendet
    }
    // falls jahr ohne rest durch 4 teilbar ist, dann ist das jahr ein Schaltjahr
    if(jahr % 4 == 0) {
        return true;  // die funktion liefert true zurück und wird beendet
    }
    // in allen anderen Fällen ist jahr kein Schaltjahr
    return false;
}

 

[ARadauer]

Die gleiche Aufgabe musste ich vor kurzem auch lösen.

Diese Aufgabe musste jeder schon mal lösen. Das ist eine standard Aufgabe im ersten Semenster