Fkt. mit Rekursion - Parameterrückgabe

[magic_halli]

Hi,

ein Funktion liefert mir als Ergebnis einen Verzeichnispfad zurück. Der Verzeichnispfad, den die Funktion ermittelt, ist erstmal korrekt, nur bekomme ich den einfach nicht korrekt zurückgeliefert, wenn ich die Funktion im Hauptprogramm aufrufe. Hier haut nochwas nicht ganz hin in Bezug auf das return-Statement in der Funktion, glaube ich?! Hab bereits zig Varianten versucht, aber es geht einfach nicht:

public void main(){

   // sPath ist hier ein bestimmtes Verzeichnis
   String sPath = curSession.GetCurrentDirectory();
   String ergPath = getDirName(sPath, model);

}

public String getDirName(String sPath, Model model){

		String ergPath = sPath;
		try{
			String sSachnummer = model.GetParam("SACHNUMMER").GetValue().GetStringValue();
			
			File file = new File(ergPath);
			File[] fileList = file.listFiles();
			for ( int i=0;i<fileList.length;i++ ){
				String sDirName = fileList[i].getName();
				if( fileList[i].isDirectory() && sDirName.equals(sSachnummer) ){
					ergPath = sPath + sDirName;
				}
				else if( fileList[i].isDirectory() && sSachnummer.indexOf(sDirName) != -1 ){
					ergPath = sPath + sDirName + "/";
					getDirName(ergPath, model);
				}
			}
			
		}catch(Exception err){
			JOptionPane.showMessageDialog(null, "getFileName(): " + err);
		}
		return ergPath;

	}

Die Variable 'ergPath' in der main() hat immer wieder den Wert von 'sPath' der main(), obwohl 'ergPath' in der Funktion getDirName(...) den korrekten Wert aufweist.
Hier klappt anscheinend die Rückgabe des Wertes in die main() nicht?!

Was ist hier falsch?

Danke und Gruß

 

[SlaterB]

Zeile 12: ergPath = sPath

damit fängt es also an,
ergPath wird genau in Zeile 21 oder 24 geändert,
setzte dort ein Logging mit System.out.println("Treffer");
wenn diese Ausgabe erscheint, dann wurde ergPath geändert, sonst nicht,
kannst außerdem noch am Ende der Methode den Inhalt von ergPath prüfen


> obwohl 'ergPath' in der Funktion getDirName(...) den korrekten Wert aufweist
wie kommst du darauf?


der rekursive Aufruf von getDirName(ergPath, model); in Zeile 25 ist übrigens noch fraglich,
dessen Rückgabewert verwendest du nicht,

 

[Marco13]

Hab's jetzt nicht komplett nachvollzogen, aber dass in Zeile 25 nur
getDirName(ergPath, model);
aufgerufen wird, ohne den Rückgabewert zu beachten, ist vermutlich schonmal falsch. Der Wert, den man da zurückbkommt, müßte noch mit dem bisherigen ergPath verwuerstet werden ... sowas wie
ergPath += getDirName(ergPath, model);
oder so - mach' dir am besten ein paar Debug-Ausgaben, um zu erkennen, WAS da WIE zusammengesetzt werden müßte.

EDIT: Hm - SlaterB ist ja heut' wieder flott

 

[SlaterB]

die letzten Tage des Forums, noch mal ordentlich zuschlagen

 

[xysawq]

import java.io.File;

public class Main
{
	public static void main(String[] args)
	{
		String sPath = "D://temp";
		String sSachnummer = "1234";
		String ergPath = getDirName(sPath, sSachnummer);
		
		System.out.println(ergPath);
	}

	public static String getDirName(String sPath, String sSachnummer)
	{
		String ergPath = sPath;
		try
		{		         
			File file = new File(ergPath);
			File[] fileList = file.listFiles();
			for (int i=0;i<fileList.length;i++)
			{
				String sDirName = fileList[i].getName();
				if( fileList[i].isDirectory() && sDirName.equals(sSachnummer) )
				{
					ergPath = sPath + "/" + sDirName;
				}
				else if( fileList[i].isDirectory() && sSachnummer.indexOf(sDirName) != -1 )
				{
					ergPath = sPath + sDirName + "/";
					getDirName(ergPath, sSachnummer);
				}
			}
		}
		catch(Exception err)
		{
			err.printStackTrace();
		}
		
		return ergPath;
	}
}

Habs mal damit durchgetestet... im else if Teil ist aber der Sinn für mich weg, weil der direkt in den ersten Ordner den er findet reingeht, sollte er nicht mit der Sachnummer übereinstimmen.

 

[magic_halli]

...mh, ich will eigentlich "nur" folgendes machen:
Ein Verzeichnis inkl. Unterordner soll durchsucht werden und als Ergebnis der Verzeichnispfad zurückgeliefert werden, wo der Hauptordner oder Unterordner gleich der Sachnummer ist.
Bsp: Sachnummer='0815-2-1' bei folgender Ordnerstruktur:

c:\temp
  |--0815
       |--0815-1
       |--0815-2
          |--0815-2-1
          |--0815-2-2

In 'ergPath' sollte am Schluß "c:\temp\0815\0815-2\0815-2-1" stehen! Das ist soweit eigentlich mein Vorhaben...

 

[xysawq]

Versuchs mal so:

import java.io.File;

public class Main
{
	public static void main(String[] args)
	{
		String sPath = "D:/temp";
		String sSachnummer = "1234-5";
		String ergPath = getDirName(sPath, sSachnummer);
		
		System.out.println(ergPath);
	}

	public static String getDirName(String sPath, String sSachnummer)
	{
		String ergPath = null;
		boolean found = false;
		try
		{		         
			File file = new File(sPath);
			File[] fileList = file.listFiles();
			for (int i=0;i<fileList.length && found == false;i++)
			{
				String sDirName = fileList[i].getName();
				if( fileList[i].isDirectory() && sSachnummer.startsWith(sDirName) )
				{
					String tempPath = sPath + "/" + sDirName;
					if(sSachnummer.equals(sDirName))
					{
						ergPath = tempPath;
					}
					else
					{
						ergPath = getDirName(tempPath, sSachnummer);
					}
					found = true;
				}
			}
		}
		catch(Exception err)
		{
			err.printStackTrace();
		}
		
		return ergPath;
	}
}

Das geht bei mir ^^.

 

[xysawq]

Habe zu meinem Beispiel die Ordner temp\1234 und darin die Ordner 1234-1 bis 1234-8 erstellt und er hat ohne Probleme "D:/temp/1234/1234-5" zurück gegeben.

EDIT: Wenn du übrigens Backslashes (nich / sondern \) haben willst musst du statt / -> \\ eingeben... das geht auch ^^.