wahrscheinlich ein kleines Problem

[PeterK]

Hallo,

ich habe eine Übung die ToolBox heißt. Dabei ist eine passive Klasse zu schreiben, die mit Hilfe eine ausführbaren, bestimmte Werte auf Ihre Richtigkeit prüft. Mein Problem ist, dass in der Methode signum ein int Wert zurückgeliefert werden soll, über den Aktualparameter jedoch ein ein double wert geliefert wird. Natürlich bringt mir der Compiler anschließend die Fehlermeldung ...


Toolbox.java:56: missing return statement
}//end signum
^
So weit so gut, ich weiß nur nicht wie ich es hinbekomme, dass er mir die Werte 0, 1 und -1 zurückliefert. Natürlich könnte ich diese Werte auch über die System.out.println ausgeben, dass würde aber nicht der Aufgabenstellung meines Professors entsprechen und fällt somit auch als Lösung aus.

Ich wäre euch für eine kurze Antwort sehr dankbar. Anbei meine Quellcodes.


public class ToolBox
{

final double i = -10e-6;
final double j = 10e-6;



public boolean istNull(double dwert)
{
if ((dwert>=i)&&(dwert<=j))
{
return true;
}//end if
else
{
return false;
}//end else


}//end istNull



public boolean istGueltig(int iwert)
{
if ((iwert>9)&&(iwert<100)&&(iwert%4==0))
{
return true;
}//end if

else
{
return false;
}

}//end istGueltig

public int signum(double dwert)
{
if (dwert>0.0)
{
System.out.println("
}

if(dwert==0.0)
{
return 0;
}

if(dwert<0.0)
{
return -1;
}
}//end signum

public boolean istHexZiffer(char cwert)
{
if((cwert>='0')&&(cwert<='9'))
{
return true;
}

else if((cwert>= 'a')&&(cwert<='f'))
{
return true;
}
if((cwert>='A')&&(cwert<='F'))
{
return true;
}
else
{
return false;
}


}//end istHexZiffer




}//end class


//////////////////////////////////////////////////////////////////////////////////////////////////


public class ToolBoxTest
{

public static void main (String [] args)
{
int iwert =88;
double dwert =0.000002;
ToolBox tool_ref = new ToolBox();

if (tool_ref.istNull(dwert))
System.out.println("Wer:" + dwert + "ist innerhalb der Toleranz");

else
System.out.println("Wer:" + dwert + "ist außerhalb der Toleranz");

if (tool_ref.istGueltig(iwert))
System.out.println(iwert + "ist ein gueltiger Wert!");

else
System.out.println(iwert + "ist ein ungueltiger Wert!");



System.out.println(tool_ref.istHexZiffer('F'));
System.out.println(tool_ref.istHexZiffer('g'));
System.out.println(tool_ref.istHexZiffer('H'));
System.out.println(tool_ref.istHexZiffer('5'));



}//end main


}//end ToolBoxTest

 

[darekkay]

benutze bitte die Java-Tags hier im Forum...

Zu deinem Problem: deine Methode muss immer etwas zurückgeben. Das mach die Signum Methode aber nicht. Du fragst alles ab ("gilt das, dann return bla"). Und was, wenn das alles nicht zutrifft?
Lösung: zwischen dem letzten if und der schließenden Klammer das hier einfügen: return 1; (optional mit "else" davor, aber ist in dem fall nicht notwendig)

 

[Der Müde Joe]

In dieser Methode:

public int signum(double dwert)
{
if (dwert>0.0)
{
System.out.println("");
}

if(dwert==0.0)
{
return 0;
}

if(dwert<0.0)
{
return -1;
}
}//end signum

Auch wenn es denn anschein macht (der Compiler weiss nicht das alle Fälle abgegolten sind ;-)

EDIT:
könntest einfach
if (grösser null)
else if (kleiner null)
else

dann klappts auch mit dem compiler..

 

[PeterK]

benutze bitte die Java-Tags hier im Forum...

Zu deinem Problem: deine Methode muss immer etwas zurückgeben. Das mach die Signum Methode aber nicht. Du fragst alles ab ("gilt das, dann return bla"). Und was, wenn das alles nicht zutrifft?
Lösung: zwischen dem letzten if und der schließenden Klammer das hier einfügen: return 1; (optional mit "else" davor, aber ist in dem fall nicht notwendig)

Sorry, wegen des Codes. Bin noch relativ neu hier und wusste nicht das es diese Möglichkeit gibt. Werde es aber definitiv beim nächsten mal so machen.

Bezüglich der Antwort komme ich aber trotzdem nicht ganz klar. Mir ist ja bewusst, dass ich immer auf jede Abfrage eine Antwort bringen muss, aber ich dachte das wäre gegeben, da ich ja immer die Abfrage mache und durch return einen Intwert zurückgebe. Es gibt ja nur die Möglichkeit dwert>0.0, dwert<0.0 und dwert==0. Verstehe nicht warum ich das nicht mit den drei If-Anweisungen lösen kann. Wahrscheinlich stehe ich auch voll auf dem Schlauch, aber bitte versucht mir das noch mal näher zu bringen.

Danke

 

[andiv]

Der müde Joe hat das eigentlich schon gesagt, der Compiler kann nicht erkennen dass du alle Möglichkeiten abgedeckt hast. Für den Compiler sieht das aus wie:

public int signum(double wert) {
    if(bedingung1) {
        return 1;
    }
    if(bedingung2) {
        return 0;
    }
    if(bedingung3) {
        return -1;
    }
    // Und was wird hier zurückgegeben?
}

 

[Der Müde Joe]

Wie schon erwähnt: Für dich macht es den Anschein, dass alle Fälle abgegolten sind.
klar: grösser null, kleiner null, oder gleich null.
Der Compiler sieht aber nur
Wenn so..dann das
Wenn so..dann das
Wenn so..dann das.
...und was sonst? --> kein return?

Drum

if (x < 0 ) {

} else if (x > 0) {

} else {

}

Jetzt sieht der compiler
wenn kleiner ..so
wenn grösser ..so
sonst...immer das

 

[PeterK]

Wie schon erwähnt: Für dich macht es den Anschein, dass alle Fälle abgegolten sind.
klar: grösser null, kleiner null, oder gleich null.
Der Compiler sieht aber nur
Wenn so..dann das
Wenn so..dann das
Wenn so..dann das.
...und was sonst? --> kein return?

Drum

if (x < 0 ) {

} else if (x > 0) {

} else {

}

Jetzt sieht der compiler
wenn kleiner ..so
wenn grösser ..so
sonst...immer das

Ah ok, jetzt bin ich auch dabei. Hab hier so ein Java Buch das dieses Problem nicht beschreibt, aber als Lösungsweg eine Option mit drei if-Anweisungen hintereinander aufzeigt. Mit anderen Worten muss ich...

if (dwert>0.0)
		{
		return  1;
		}
			
				if(dwert<0.0) 
				{
				return -1;
				}	
				else 
				{		
				return 0;	
				}

... machen um das Problem zu beseitigen.

 

[Der Müde Joe]

Das wäre die 2te Möglichkeit neben dem if.. else if.. else
Es kommt immer zu einem Ende. Spätestens im else..
Der Compiler ist zufrieden ;-)

EDIT:
da gäbs natürlich noch viele Varianten..

if (x < 0) {
} else {
 if( x > 0) {

 } else{
   // == 0
 }
}
//oder
if (x < 0) {
}
if(x > 0) {
}
return .. //== 0
//oder...