System.out.println metoden?

Prøv sådan her:

  public void removeItem(Item i)
    {
        System.out.println(Item.value(i) + " has been removed from " + getName().toString());
        items.remove(i);
    }

//>Rune

Jeg bør måske sige at mit Item kommer fra en ArrayList, så hvis jeg kan hente navnet på itemmet derigennem, er det fint.... men items.get(i) virker ikke :(

Det er ikke getName funktionen der er et problem... det er Item.value(i)
Resten virker fint...

Jeg forstår bare ikke, for nu fik jeg endelig min (i) ind og stadig retunere den, den sidst indsatte...

Her er Item klassen:

public class Item {

    private static String value;

    public Item(String value) {
        this.value = value;
    }

    public static String value(Item i) {
        return value;
    }
   
    public String toString()
    {
        return value;
    }
}

Prøv med:

public void removeItem(Item i)
    {
        System.out.println(items.remove(i) + " has been removed from " + getName());
    }

Eller:

public void removeItem(Item i)
    {
        items.remove(i);
        System.out.println(i + " has been removed from " + getName());
    }

ystem.out.println(i + " has been removed from " + getName());

Jeg forstår nu heller ikke helt din klasse. Mener du ikke:

public class Item {

    private String value;

    public Item(String value) {
        this.value = value;
    }

    public String value(Item i) {
        return value;
    }
 
    public String toString()
    {
        return value;
    }
}

og
    public String value(Item i) {
skal jo bare være
    public String value() {

Fejlen ligger faktisk i når jeg opretter et nyt Item, så får alle mine items den sidst indtastede værdi... og det fatter jeg slet ikke en pind af...

Det vil sige fejlen ligger her et sted:

public class Item {

    private static String value;
   
    public Item(String value) {
        this.value = value;
    }

    public static String value(Item i) {
        return value;
    }
   
    public String toString()
    {
        return value;
    }
}

Ja, Erik men så kan jeg vel ikke tilføje en parameter når jeg kalder metoden fra en anden klasse... Desuden ser det ikke ud som om det gør nogen forskel i Item klassen...

Jeg har jo behov for at kunne retunere et bestemt object fra Item...

Jeg synes du skal prøve at forstå mit forslag :)

Din Item klasse indeholder kun 1 af hvad det nu er.

Prøv og ændre:

private static String value;

til:

private String value;

som Erik foreslår.

Erik --> Det er lidt lige meget for alle mine Items objecter får samme navn(String)... Så lige meget hvordan jeg retunere dem skriver den jo selvfølgelig det samme...

Og jeg har prøvet at ændre på public String value()... det ændrer ikke ved at alle items får samme navn... underligt nok.

Tak Arne V! Fantastisk svar... og det var løsningen...
Jeg kunne fandme ikke se skoven for bare træer, jeg har fået en dårlig vane med at lave mine metoder static...det skal jeg satme lade være med.

Nu var det faktisk eriks forslag - jeg var bare lidt mere eksplicit
med hensyn til hvad forskellen var.

Det er ret bigtigt om du bruger static eller ej der.

static = en værdi som er fælles for alle objekter af typen
ikke static = en værdi for hver objekt af typen

Det er ikke nogen kritik af dig Arne, men jeg kan bare så godt lide at
prøve at få spørgeren til selv at "se lyset". Det sker en gang imellem, at
man kan "se" resultatet af de små grå cellers arbejde i den anden ende.

Men nu er jeg også miljøskadet som handelsskolelærer ;))

Smider du et svar Arne? :-)

Det kan jeg godt, men er det ikke snarere Erik der burde have point ?

BTW:

Hvorfor kan jeg ikke gøre sådan her fra en anden klasse:


pub.addItem(beer);

Hvor pub er et tilfældigt rum...
Den siger at metoden er non-static, men hvis jeg gør addItem til static tilføjer den jo alle mine items i alle mine rum... hmm

Hvad er "pub" - vi kan ikke se det her

import java.util.*;

public class TestDriver
{
    public static void test()
    { 
        Room pub = new Room("pub");
        Room disco = new Room("disco");
       
        pub.connectTo(Direction.SOUTH, disco);
        disco.connectTo(Direction.NORTH, pub);
       
        Item beer = new Item("beer");
        Item softdrink = new Item("softdrink");
        Item wallet = new Item("wallet");
        Item girl = new Item("girl");
        Item budweiser = new Item("budweiser");
       
        pub.addItem(beer);
       
       
        Visitor kim, line;
        kim = new Visitor("Kim", pub);

        kim.move(Direction.NORTH);
        kim.pickUp(kim.location().chooseItem());
    }
}

Hvis pub er et objekt så burde det være helt OK at addItem er
non static.

Hvad er fejlen præcis og hvordan ser metoden ud ?

og Room klassen er lige oven over...

Problemet er som sagt at jeg ikke må bruge static her, fordi ikke alle mine items skal under alle Room... hvis i forstår...

public void addItem(Item i)
    {
      items.add(i);
      System.out.println(i + " has been added to " + (String) getName());
    }

Hvis addItem og items begge er static så deler alle Room's items.

Hvis addItem og items begge er ikke-static så har hver Room sine items.

Du er sikker på det er addItem der giver problemer??

Sjovt nok, så virker det nu... jeg har stadig skrevet:
        pub.addItem(beer);

ingenting er ændret så vidt jeg kan se...

Godt, at du fik dette problem løst ;)

Kigger du også lige her?:
http://www.eksperten.dk/bruger.phtml?sort=%40&order=asc&catid=0&q=&navn=skurggman&option=22