Rekursion Problem

[HakBak]

Hallo,
es gibt soviele Probleme mit der Rekursion und nu hab ich auch eins. Ich finde den Fehler aber nicht.

Hier mal der Code

public String createSitemapItems(String replace, COP cop){
		if (cop != null) {
			List<BE> beList = cop.getBE();
			
			if (beList != null && beList.size() > 0) {
				for (int j = 0; j < beList.size(); j++) {
					BE curBE = beList.get(j);
					replace = replace + "<div>BE"+ curBE.getId().toString()+"</div>\n";
					
					if (curBE.getType() != null) {
						if (curBE.getType().equalsIgnoreCase("InternalLink") || 
								curBE.getType().equalsIgnoreCase("MyMicrosites")) {
							if (curBE.getCOP() != null) {
								replace = replace + "<div>COP" + curBE.getCOP().getId()+"</div>";
								createSitemapItems(replace, curBE.getCOP());
							}
						}
						else {
							
							if (curBE.getCPP() != null) {
								replace = replace + "<div>CPP"+curBE.getCPP().getId()+"</div>";
							}
						}
					}
				}
			}
		}
		return replace;
	}

Das ganze Stammt aus einem Baum, den ich in ein HTML Gebilde umformen will. Ein COP kann hat ein oder mehrere BE, ein BE kann wiederrum ein neues COP haben oder ein CPP, also möchte ich wenn in einem BE ein COP gefunden wurde die Methode nochmal aufrufen, aber in meinem Fall hier wird einfach die Schleifer weiterdurchlaufen. Deshalb schafft es meine Methode dann auch nicht eine Ebene tiefer, was sie aber mindestens machen sollte, da es möglich ist, dass sich die COP-BE beziehungen häufig wiederholen.

Danke schonmal

 

[SlaterB]

in Zeile 16 verwendest du den Rückgabewert von createSitemapItems() nicht

wenn du der Parameter-Variablen replace einen neuen String zuweist,
dann bekommt der Aufrufer das nicht mit,
deswegen hast du ja auch richtigerweise ein return am Ende

wenn du nur so einfach zusammenbaust, dann musst du gar keinen Parameter replace übergeben,
was immer da drin steht interessiert die Unteroperation doch gar nicht, sie fügt nur etwas an,

stattdessen könnte die Unteroperation einen neuen String aufbauen, den der Aufrufer bei Bedarf selber an einen vorhandenen String anfügt

 

[HakBak]

Danke, war doch nur wieder ein kleiner Fehler. Das mit dem "Zusammenbasteln" muss ich mir eh erst nochmal überlegen, aber ich bin jetzt erstmal froh, dass die Rekursion funktioniert, danke!