ermittlung des minimus zweier eingegebener werte

[turbo]

Hallo! Ich hab mal wieder ein newbie-problem. und zwar gibt mir folgendes progrämmchen zur ermittlung des minimums zweier eingegebener werte kein ergebnis aus, sondern nur einen "stackoverflowerror". das programm hängt also in einer endlosschleife fest. zur berechnung soll die methode "static int min (int a, int b)" genutzt werden. leider find ich den fehler nicht.

import java.io.*;				
public class test{
				
 static int min (int a, int b) {
 return min (a,b);
 } 
 
 public static void main (String[] args) throws IOException{
 int a;								
 int b;
  
 BufferedReader din = new BufferedReader(			
 		       new InputStreamReader (System.in));	
  
  System.out.println("Bitte a eingeben: ");
  a = Integer.parseInt(din.readLine());
  System.out.println("Bitte b eingeben: ");
  b = Integer.parseInt(din.readLine());	
			   
  System.out.println(min (a,b));
 }
}

kann jemand das problem erkennen?

vielen dank mal wieder im vorraus!!

 

[Der Müde Joe]

du rufst in deiner min wieder min auf..und die wiederum min etc...

nimm doch einfach:
Math.min(x,y);

 

[turbo]

danke für die antwort!

ich habs allerdings anders gemacht, da ich eine eigene methode schreiben muss:

static int min (int a, int b) {
 	if (a < b){
 	 return (a);
 	}
 	else {
 	 return (b);
 	}
 }

diese methode funktioniert wunderbar, obwohl das return statement in die if anweisung eingebettet ist.

wenn ich die methode nun aber erweitern möchte, um von 3 eingegebenen zahlen das minimum zu ermitteln, wird das eingebettete return- statement nicht erkannt:

static int min (int a, int b, int d) {
  if (a < b){
    if (a < d){
   	  return (a);
    }
  }
    else {
     if (b < a){
      if (b < d){
   	   return (b);
      }
    }
    else{
     return (d);
    }
    }
   }

hier wird mir immer gesagt, dass das return statement fehlt, obwohl es hier lediglich in zwei statt in eine if-Anweisung eingebettet ist. gibt es dafür einen anderen weg? oder hab ich einfach mal wieder nen fehler eingebaut, den ich nich seh?

 

[SlaterB]

static int min(int a, int b, int d)
    {
        if (a < b)
        {
            if (a < d)
            {
                return (a);
            }
//          else
//          {
//             was hier?
//          }

        }
        else
        {
            if (b < a)
            {
                if (b < d)
                {
                    return (b);
                }
                // und hier auch im else Fall? die Bedingugen a< b und b<a machen eh keinen Sinn
            }
            else
            {
                return (d);
            }
        }
    }

 

[turbo]

sorry SlaterB, aus deinem post werd ich nich so recht schlau

 

[SlaterB]

ich habe da zwei else-Fälle markiert, an denen der Compiler nicht weiß, was er zurückgeben muss,
das return muss für alle möglichen Durchläufe definiert sein

 

[Saxony]

Hiho,

wie wärs mit folgender Lösung:

public static int min(int a, int b, int c) {
		
		return (a<=b)?((a<=c)?a:c):((b<=c)?b:c);
	}

bye Saxony

 

[Der Müde Joe]

oder gleich:

public static int  min(int a, int b, int c){
return min( min(a,b), c );
}

zuerst a und b vergleichen, dann noch mit c