Compiler-Fehler Stringvariable ändern | Compiler warnung

[Dit_]

Hallo
Wollte schon immer wissen warum der Compiler an dieser Stelle warnt...

public void doit(String string) {
        string = string.trim();//hier warnung
        //...
}

Kompiler möchte das haben.

public void doit(String string) {
        String neuerString = string.trim();
        //...
}

Warum?

Danke schon mal

 

[eRaaaa]

Und mit was "warnt" er dich? =)

/edit: Ok mein Fehler: "Wie lautet die Warnmeldung in deinem Eclipse denn GENAU?" :noe:

 

[Dit_]

na ja im Eclipse wird die Stelle gelb unterstrichen er der Compiler meint ich soll neue Variable definieren.

 

[RySa]

Ich hoffe dir ist bewusst, dass du den Inhalt des Originalstrings so nicht ändern kannst (call by Value). Wenn das nicht dein Ziel ist und du den String nur für die weiterverarbeitung dieses String ihn trimmen möchtest, weiß nicht wo dein Problem liegt (vermutlich bei der Syntax irgendwo) da es bei mir so einwandfrei funktioniert.

 

[AlexSpritze]

Bei NetBeans kann man auch einstellen, dass er diese Semantik als Warnung kennzeichnet. Die Beschreibung zu diesem Hint lautet:

Reports any instances of assignment to a variable declared as a method parameter. It also reports any attempt to increment or decrement the variable. While occasionally intended, this construct can be extremely confusing, and is often the result of a typo.

Also wenn du sowas hast:

public void method(String input)
{
String _input = "asdf";
....

for(char c : _input.toCharArray())
...
}

könnte schnell ein Typo zwischen input und _input geschehen, mal abgesehen von der Bezeichnung der Variablen

Man könnte den Methodenparameter auch final deklarieren:

public void method(final String input){...}

 

[Ariol]

public void doit(String string) {
        string = string.trim();
}

public void test() {
      String string = "  Hallo Welt!   ";
      System.out.println(string);
      doit(string);
      System.out.println(string);
}

Ausgabe:

  Hallo Welt!   
  Hallo Welt!

--------------------------------------------------------------

public String doit(String string) {
        return string.trim();
}

public void test() {
      String string = "  Hallo Welt!   ";
      System.out.println(string);
      string = doit(string);
      System.out.println(string);
}

Ausgabe:

  Hallo Welt!   
Hallo Welt!

--------------------------------------------------------------

 

[ARadauer]

Wollte schon immer wissen warum der Compiler an dieser Stelle warnt...

Weil er eine Tussi ist ;-)

 

[Dit_]

das es funktioniert ist ja klar. Und das man das abstellen kann ist auch klar.

Also ist es bloß "CodeStyle warnung", hat nichts mit performance zu tun?

P.S. kann in Eclipse das Häckchen nicht finden um die Warnung hier zu zitieren...

 

[SlaterB]

'Parameter assigment', falls interessant

und, grad gefunden, um überhaupt mal eine Begründung zu geben, vielleicht nicht die schlauste:
Remove Assignments to Parameters

 

[Dit_]

ok danke hab heute auch endlich gefunden.

The parameter flagsExtra should not be assigned

 

[ARadauer]

Ich glaub Fowler schreibt auch im Refactoring Buch darüber...

Meiner Meinung nach ist es eigentlich kein Code Smell... gibt es wirklich Programmierer, die bei so einer Zeile denken, dass auch die Variable im aufrufenden Code geändert wird? Das kann mal einem Anfänger passieren, man sollte es aber schnell verstanden haben, dass dem nicht so ist...

 

[RySa]

Naja, vlt kann der Editor nicht unterscheiden ob es primitive Werte sind, die übergeben werden, oder Objekte, deswegen meldet er bei allem eine Warnung (da ja Objekte ByRef übergeben werden und somit so verändert werden können).

 

[SlaterB]

das Verändern an sich wird ja gerade eher nicht verhindert und auch final hat damit nichts zu tun,
das Verändern von Parametern geht, mal nützlich, mal weniger..,

es geht hier immer um Neuzuweisung der Variablen, da ist es egal ob Objekt oder primitiv,
in beiden Fällen hilft final, in beiden Fällen kann es dem Aufrufer schnuppe sein,
ich halte von der Warnung auch nicht so viel

 

[Mitleserin]

Weil er eine Tussi ist ;-)

Ich finde, dass solche Kommentare den höflichen und respektvollen Umgang in einem Forum stören. Auf einer Site wie dieser sind weibliche Mitglieder erfahrungsgemäß ohnehin seltener anzutreffen als männliche und wenn sie dann über derartige Sätze stolpern, werden sie vielleicht geneigt sein, ihre Besuche in diesem Forum noch weiter zu reduzieren.

 

[Sonecc]

Ich finde, dass solche Kommentare den höflichen und respektvollen Umgang in einem Forum stören. Auf einer Site wie dieser sind weibliche Mitglieder erfahrungsgemäß ohnehin seltener anzutreffen als männliche und wenn sie dann über derartige Sätze stolpern, werden sie vielleicht geneigt sein, ihre Besuche in diesem Forum noch weiter zu reduzieren.

Ohne nun eigentlich ins Off-Topic abdriften zu wollen, kann ich mich dennoch nicht zurückhalten:

Was bitte schön ist an der Aussage denn schlimm? Es wurde niemand beleidigt und es ist keinesfalls sexistisch oder ähnliches.

 

[Ariol]

Ich glaub Fowler schreibt auch im Refactoring Buch darüber...

Meiner Meinung nach ist es eigentlich kein Code Smell... gibt es wirklich Programmierer, die bei so einer Zeile denken, dass auch die Variable im aufrufenden Code geändert wird? Das kann mal einem Anfänger passieren, man sollte es aber schnell verstanden haben, dass dem nicht so ist...

Das Problem ist glaube ich eher wenn Attribute innerhalb der Variable geändert werden.
Dieser Code hier...

public class ValueTest 
{
	int a;
	public ValueTest(int a) {
		this.a = a;
	}
	
	public static void edit1(ValueTest v)
	{
		v = new ValueTest(-1);
	}
	
	public static void edit2(ValueTest v)
	{
		v.a = -1;
	}
	
	public static ValueTest edit3(ValueTest v)
	{
		v = new ValueTest(-1);
		return v;
	}
	
	public static void edit4(final ValueTest v)
	{
		v.a = -1;
	}	
	
	public static void main(String[] args) {
		ValueTest a1 = new ValueTest(1);
		ValueTest a2 = new ValueTest(1);
		ValueTest a3 = new ValueTest(1);
		ValueTest a4 = new ValueTest(1);
		
		edit1(a1);
		edit2(a2);
		a3 = edit3(a3);
		edit4(a4);

		System.out.println(a1.a);
		System.out.println(a2.a);
		System.out.println(a3.a);
		System.out.println(a4.a);
	}
}

... gibt das aus:

1
-1
-1
-1

 

[Dit_]

Ich finde, dass solche Kommentare den höflichen und respektvollen Umgang in einem Forum stören. Auf einer Site wie dieser sind weibliche Mitglieder erfahrungsgemäß ohnehin seltener anzutreffen als männliche und wenn sie dann über derartige Sätze stolpern, werden sie vielleicht geneigt sein, ihre Besuche in diesem Forum noch weiter zu reduzieren.

ehmm... ja... wir sind alle untröstlich... :lol:

 

[ARadauer]

Achtung!!!

Naja, vlt kann der Editor nicht unterscheiden ob es primitive Werte sind, die übergeben werden, oder Objekte, deswegen meldet er bei allem eine Warnung (da ja Objekte ByRef übergeben werden und somit so verändert werden können).

ja verändern kann man Objekte schon, aber natürlich nicht die Referenz! die Zuweisung wirst du drausen nie mitbekommen ob es jetzt nun 5, "BIER" oder dein wichtigestes Kundenobjekt war..

, werden sie vielleicht geneigt sein, ihre Besuche in diesem Forum noch weiter zu reduzieren.

tja ich bin untröstlich :gaen:

 

[Mitleserin]

Sonecc, ich habe ARadauers Beitrag so verstanden, dass ein Compiler, der Warnungen wirft, vergleichbar sei mit einer nervigen pingeligen Frau, die unnützes Gewäsch von sich gibt. Einen solchen Vergleich empfinde ich persönlich als sexistisch. Macht es das für dich klarer?

ehmm... ja... wir sind alle untröstlich... :lol:

tja ich bin untröstlich :gaen:

Wow… Das finde ich einfach nur krass, wie auf einen vollkommen sachlichen Kommentar von mir das Niveau hier so nach unten ausschlägt.

 

[anonym2]

*edit (besser entfernt..)*

Auf einer Site wie dieser sind weibliche Mitglieder erfahrungsgemäß ohnehin seltener anzutreffen

Das stimmt, aber was würde sich ändern, wenn dem nicht so wäre?

(Ich stelle konfuse/verworrene Fragen. )

 

[Murray]

Ich finde, dass solche Kommentare den höflichen und respektvollen Umgang in einem Forum stören. Auf einer Site wie dieser sind weibliche Mitglieder erfahrungsgemäß ohnehin seltener anzutreffen als männliche und wenn sie dann über derartige Sätze stolpern, werden sie vielleicht geneigt sein, ihre Besuche in diesem Forum noch weiter zu reduzieren.

Würde hier ewig diskutiert werden, wenn statt "Tussi" an dieser Stelle "Weichei" stünde (was m.E. ebenfalls nicht wirklich bedeutungsgleich mit dem - an dieser Stelle eher angemessenen - "überempfindlich" wäre?

 

[mvitz]

Ich schreib dann einfach noch mal was zum Topic

Ich kann verstehen, wieso da eine Warnung geschmissen wird, angenommen man hat eine etwas längere Methode, kennt aber die Parameter-Variablen Namen. Jetzt wird genau so eine Variable weiter unten an eine andere Methode übergeben. Hierbei übersieht man jetzt schnell, dass es sich gar nicht mehr um das Argument der Methode, sondern bereits um eine neue Variable handelt.

 

[Dit_]

... nervigen pingeligen Frau, die unnützes Gewäsch von sich gibt. Einen solchen Vergleich empfinde ich persönlich als sexistisch. Macht es das für dich klarer?

also jetzt mal im Ernst, das war ganz bestimmt nicht böse gemeint, und dieser scherzhafte Vergleich ist bestimmt nur wenigen aufgefallen... Aber dank dir, ist es jetzt zum Hauptthema des Threads geworden. Bravo! :bahnhof:

nervigen pingeligen Frau, die unnützes Gewäsch von sich gibt

du muss dich doch nicht gleich angesprochen fühlen ueh:

 

[maki]

ARadauer ist Österreicher, die haben kürzlich sogar ihre Nationalhymne korrigiert im Sinne der Gleichberechtigung.
Natürlich sind Tussis nicht weniger schlecht als Weicheier, man sollte schon ein "equal opportunity offender" sein, also jemand der alle Seiten abwechselnd gleich beleidigt, schon der Balance wegen.

Zum Thema:
Parametern neue Werte zuzuweisen ist schlechter Stil, kann zu schwer auffindbaren Bugs führen, Andi_CH hatte vor ein paar Monaten so ein Beispiel.
Wenn die Parameter als final deklariert werden, wird aus einer Warnung ein Error.

Es kostet nix da eine neue Variable einzuführen, ausser eine Zeile Code.

String trimmedString = string.trim();

 

[Tomate_Salat]

@Mitleserin: Naja. Hier wurde wieder aus einer Mücke ein Elefant gemacht.

Es kostet nix da eine neue Variable einzuführen, ausser eine Zeile Code.

String trimmedString = string.trim();

In diesem Falle nicht einmal. Imho sollte man Parameter, auch wenn sie nicht final sind, so behandeln.

 

[maki]

Imho sollte man Parameter, auch wenn sie nicht final sind, so behandeln.

Sehe ich genauso.