Color Map

[pisco]

Hey Leute,

ich bin gerade dabei mir rekursion beizubringen und spiel mich mit diesem code herum:

	public HashMap<String, Integer> countColors() {
		
		if(null != this.content){
			HashMap<String, Integer> map = this.content.countColours();
			if(map.containsKey(this.color)){
				int oldChar = map.get(this.color);
				map.put(this.color, oldChar+1);
			} else {
				map.put(this.color, 1);
			}
			return map;
			
		} else {
			HashMap<String, Integer> map = new HashMap<String, Integer>;
			map.put(this.color, 1 );
			return map;
		}
		
		
	}

ich bekomme aber immer nur null. Eigentlich sollte ich die color der Box herauskriegen...

warum krieg ich immer null?

Ich bedank mich schonmal für eure Antwort ;D

gruss

 

[SlaterB]

wo bekommst du null, als Rückgabewert der Methode?, das ist zu bezweifeln,
da kommt entweder die neu angelegte Map (die nicht gespeichert wird), die vorhandene Map
oder ein NullPointerException wenn die vorhandene Map null ist (edit: ok, letzteres ist die Rekursion, also eher auch nicht möglich)

was an Color alles gespeichert wird tut anscheinend nicht zur Sache oder geht es darum?
es wird aber letztlich im geposteten Code gar nichts abgefragt

 

[pisco]

import java.util.HashMap;
public class Box {
	
	private String color;	
	private Box content;
	
	public Box(String color) {
		this.color = color;
	}
	
	public void insert(Box box) {	
		this.content = box;
	}
	
	public String getMostFrequentColor() {
		

		return null;	
		
	}
	
	public HashMap<String, Integer> countColors() {
		
		if(null != this.content){
			HashMap<String,Integer> map =   
			this.content.countColors(); 
			
			if(map.containsKey(this.color))
			{
				int a = map.get(this.color);
				map.put(this.color, a + 1);
			}
			else
			{
				map.put(this.color, 1);
			}
			return map;
			
		}
		else
		{
			HashMap<String,Integer> map = new HashMap<String,Integer>();
			map.put(this.color, 1);
			return map;
		}
}

hier ist der restliche code... das die map null zurückgibt ,denk ich eher nicht... habs sie ja erzeigt und gespeicher oder irre ich mich da?

gruss und danke nochmals...

 

[SlaterB]

nochmal: was ist die Frage, an welcher Stelle bekommst du null, was erwartest du stattdessen?

 

[pisco]

also erwarten tu ich das ich:

zBsp.: red zurück bekomme aus der main, welches in der main so übergeben wird:

Box whiteBox1 = new Box("white");
Box redBox1 = new Box("red");
Box greenBox1 = new Box("green");
Box redBox2 = new Box("red");

....

an welcher Stelle ich null bekomm is mir komplett unklar...

gruss

 

[chalkbag]

Kannst du mir dein Main Programm geben, dann verstehe ich vielleicht was du machen willst und spring schnell mit dem Debugger drüber.

Grüße

 

[SlaterB]

was haben die Aufrufe
Box whiteBox1 = new Box("white");
Box redBox1 = new Box("red");
Box greenBox1 = new Box("green");
Box redBox2 = new Box("red");

auf einmal mit der Map-Methode zu tun?
dass du nicht weißt, wo du null bekommst, steht im Widerspruch zu deiner ursprünglichen Frage 'warum krieg ich immer null?',
denn die Voraussetzung für diese Frage ist, erstmal FESTZUSTELLEN, dass ein Ding namens 'null' irgendwo auftaucht

ich verstehe ich dich wohl einfach überall falsch,
damit es mal vorangeht hier ein Testprogramm:

public class Test
{
    public static void main(String[] args)
    {
        Box whiteBox1 = new Box("white");
        Box redBox1 = new Box("red");
        Box redBox2 = new Box("red");
        redBox1.insert(whiteBox1);
        redBox2.insert(redBox1);

        HashMap<String, Integer> m = redBox2.countColors();
        System.out.println("map: ");
        for (String key : m.keySet())
        {
            System.out.println(key + " -> " + m.get(key));
        }
    }
}

class Box
{
    private String color;
    private Box content;

    public Box(String color)
    {
        this.color = color;
    }

    public void insert(Box box)
    {
        this.content = box;
    }

    public String getMostFrequentColor()
    {
        return null;
    }

    public HashMap<String, Integer> countColors()
    {

        if (null != this.content)
        {
            HashMap<String, Integer> map = this.content.countColors();

            if (map.containsKey(this.color))
            {
                int a = map.get(this.color);
                map.put(this.color, a + 1);
            }
            else
            {
                map.put(this.color, 1);
            }
            return map;

        }
        else
        {
            HashMap<String, Integer> map = new HashMap<String, Integer>();
            map.put(this.color, 1);
            return map;
        }
    }
}

Ausgabe:

map: 
red -> 2
white -> 1

sieht doch alles recht funktional aus

 

[pisco]

gut... danke erstmal für deine Antwort:

also ich denke damit wir nicht ganz aneinander vorbeireden post ich auch mal meine main:

public class BoxTester {
	
	public static void main(String[] args) {
	
		/*
		 * Fall 1
		 * "red" ist die am häufigsten vorkommende Farbe
		 */
		{
			Box whiteBox1 = new Box("white");
			Box redBox1 = new Box("red");
			Box redBox2 = new Box("red");
			Box greenBox1 = new Box("green");
			
			
			greenBox1.insert(redBox2);
			redBox1.insert(greenBox1);
			whiteBox1.insert(redBox1);
			
			System.out.println(whiteBox1.getMostFrequentColor());
		}
		
	}
	
}

null hab ich aus dem Grund bekommen ,da ichs compiliert und ausgeführt habe... nur weis ich nicht wo es im code steht das ich null krieg...

 

[xehpuk]

[JAVA=15] public String getMostFrequentColor() {


return null; // <-- sieht stark danach aus, als würde hier null zurückgegeben werden

}[/code]

 

[pisco]

ahhhhhhhh danke...

gruss ;D