Problem bei Übergabe von Werten an eine Methode

[de_maggus]

Hallo zusammen,
ich habe da ein Problem, bei dem ich nicht mehr weiter weiss.

Kurzer Überblick über mein Programm: Ich spiele DSA und will ein Automatisierungstool für den Kampf schreiben. Das erwähne ich, weil sich sonst jemand fragen könnte, warum in meinen Code-Ausschnitten Rassen und so vorkommen

Also, ich habe eine Klasse, in der ich Charaktere erstelle (Daten aus einer anderen GUI-Klasse). In der Charakter-Erstellungsklasse übergebe ich die Daten eine Funktion, die aus einer anderen Klasse Attributmodifikationen einliest.

//diese Methode steht in der Klasse Char
public void getModifikatoren(Char ch)
{
    String tempRasse = ch.getRasse();
    ....
    //Instanz der Klasse Modifikator, in der die Attribute modifiziert werden
    Modifikator modifikator = new Modifikator();
    modifikator.getModifikatoren(tempRasse, tempKultur, tempProf, tLE, .....);
    ....
}

Und hier beim Methodenaufruf ist mein Problem: Wenn ich die Daten so übergebe, werden sie nicht geändert, wenn ich
dagegen konkrete Werte übergebe (z.B. modifikator.getModifikatoren("Mittelländer", "Tulamide", "Krieger", 20, ...) ),
dann macht das Programm, was es soll.

Steh ich irgendwie auf dem Schlauch oder bin ich nur bissel doof? Wieso klappt die Übergabe mit Variablen nicht? Aber wieso klappt es mit konkreten Werten?

Hoffe, jemand hat einen Hinweis für mich.

TIA
Markus


Edit von L-ectron-X: Bitte unbedingt für jeden Thread einen Aussage kräftigen Titel vergeben. Niemand kann mit "Problem" etwas anfangen!

 

[Beni]

Der Methode "getModifikatoren" ist es total egal, woher die Werte kommen, nur stimmen müssen sie.

1. Hast du genau an dieser Stelle den Versuch mit den konkreten Werten gemacht (also genau eine Zeile würde anders aussehen?)
Es gibt ein Fehler der immer wieder auftritt, eine Variable überlagert eine andere (der Effekt ist, dass die äussere Variable nicht verändert wird, obwohl es auf den ersten Blick nicht klar ist):

public class Autsch{
  private Blupp stupid = new Blupp( "ich bin draussen" );

  public void doSomething(){
    Blupp stupid = new Blupp( "ich bin drinnen" );
  }
}

2. Überprüfst du auch, ob die Eingaben korrekt sind? Vielleicht gibts ja irgendwo einen Schreibfehler...
Ich empfehle sowas:

public int getModifikatoren( String rasse, ... ){
  if( rasse.equals( "Thorwal" ) ) return 20;
  else if( rasse.equals ( "Nives" ) return 15;
  else if( ... )
  else throw new IllegalArgumentException( "Sorry, die Rasse ist mir neu: " + rasse );
}

3. Strings werden mit "equals" und nicht "==" verglichen, siehe FAQ.

4. Wieso Kämpfe automatisieren? Abgesehen von der Herausforderung (viel Spass mit den Sonderfertigkeiten :### ), sind die doch auch Teil eines lebendigen, manchmal wilden und gefährlichen Aventurien...

So, jetzt kannst du dir eine Antwort aussuchen :bae:

 

[de_maggus]

@ L-ectron-X: Sorry für den ungeeigneten Titel

@ Beni: Danke für Deine Hilfe. Der Fehler lag tatsächlich daran, dass ich die Strings in der Klasse Modifikator mit == verglichen habe. Daran habe ich nicht gedacht

Und ich will nicht den kompletten Kampf automatisieren, sondern vielmehr die Attribute wie Lebensenergie usw. immer aktuell halten. Den Schaden würfeln wir normal aus, diesen will ich dann aber im Programm eingeben und somit die Auswirkungen automatisieren, z.b. AT und PA vermindern, wenn LE nur noch die Hälfte hat, die Kampfreihenfolge automatisieren. Das ist mir während dem Spiel zuviel Würfelei und Schreiberei.

Gruß
Markus