Java substring vereinfachen

[Greek]

Hiho,

ich wollte eine neue Funktion für substring schreiben, weil mir die vorhandene nicht gefällt.

Um ab Position 6 die nächsten 3 Zeichen zu haben muss man text.substring(6,9); schreiben. Ich würde das lieber so haben:
text.substringex(6,3);

So eine Funktion hab ich mir jetzt gemacht, aber geht das auch das ich es so als Klasse benutze?

Meins sieht so aus: substringex(text,6,3); Aber das ist 1 Parameter mehr als das Original, lieber wäre es mir so: text.substringex(6,3);

Kumpel von mir hat gesagt das ich das mit Erben machen muss, aber da steht das man von einer final Klasse nicht erben kann.

Gibts eine Möglichkeit das so zu machen text.substringex(6,3); oder brauch man dafür 3 Parameter?

Gruss Greek

 

[Firestorm87]

Hiho,

ich wollte eine neue Funktion für substring schreiben, weil mir die vorhandene nicht gefällt.

Um ab Position 6 die nächsten 3 Zeichen zu haben muss man text.substring(6,9); schreiben.
Ich würde das lieber so haben:
text.substringex(6,3);

So eine Funktion hab ich mir jetzt gemacht, aber geht das auch das ich es so als Klasse benutze?

Meins sieht so aus: substringex(text,6,3); Aber das ist 1 Parameter mehr als das Original, lieber wäre es mir so: text.substringex(6,3);

Kumpel von mir hat gesagt das ich das mit Erben machen muss, aber da steht das man von einer final Klasse nicht erben kann.

Gibts eine Möglichkeit das so zu machen text.substringex(6,3); oder brauch man dafür 3 Parameter?

Gruss Greek

Da Substring eine Methode innerhalb der Klasse "String" ist, kannst du diese ersetzen, in dem Du eine neue Klasse erstellt, von String erbst...
Dann kannst du hier die gewünschte Methode überschreiben und musst fortan, alle deine Strings nicht als String alegen, sondern als GString (G hier für Greek )

Gruß
Firestorm

 

[Der Müde Joe]

Da gibts keine Möglichkeit, ausser mit Utilityklassen.
In der Art:
StringUtils.sub("bla",from,to)

EDIT:
>in dem Du eine neue Klasse erstellt, von String erbst...

String ist final, und das ist auch gut so (immutable)

 

[citizen_erased]

Um ab Position 6 die nächsten 3 Zeichen zu haben muss man text.substring(6,9); schreiben. Ich würde das lieber so haben:
text.substringex(6,3);

wo ist da eigentlich genau der unterschied?
du willst die methode aufrufen mit (start,anzahlDerZeichen)
ist im prinzip ja nichts anderes, als substring der String-Klasse vorgibt: (start,start+anzahlDerZeichen).

mit kopfrechnen hat das auch nichts zu tun: meist sucht man über indexOf ein bestimmtes zeichen, erhält dessen position. die position wird in int i gespeichert.
der substring würde dann wie folgt aufgerufen:
text.substring(i,i+3);

 

[Firestorm87]

Da gibts keine Möglichkeit, ausser mit Utilityklassen.
In der Art:
StringUtils.sub("bla",from,to)

EDIT:
>in dem Du eine neue Klasse erstellt, von String erbst...

String ist final, und das ist auch gut so (immutable)

Hmm warum sind alle schönen Klassen final
/EDIT: Aber daran sieht man doch, dass davon auch keien Sau jemals zwingend erben musste und auch Ich es noch nie in erwägung gezogen habe :rtfm:/END-EDIT:

Aber er würde immernoch funktionieren eine Art Wrapper-Klasse zu basteln, wo man ein String reinsteckt...
Hier dann die Substringmethode anpassen....

ABER: Das ist eher eine theoretische Machbarkeit, die bitte NIEMALS anwendung findet

 

[Tomate_Salat]

Aber er würde immernoch funktionieren eine Art Wrapper-Klasse zu basteln

Wie bastel ich mir den eine eigene Wrapper-Klasse :shock::rtfm:

Comment: Btw, ich finde das eigentl. ziemlich unnötig. Ich weis nicht wie selten ich substring überhaupt benötige. Und dafür eine eigene Methode schreiben? Also entweder eine "eigene" String klasse mit getter/setter/substring oder halt eine Art Utility-Klasse.

In meinen Augen definitiv zuviel Aufwand. Da ist 6+3 noch schneller im Kopf gerechnet und wer das nicht kann soll halt einen Taschenrechner benutze .
Und ich wüsste auch nicht, wieso der 2te Parameter als entfernungsmaß so vorteilhaft sein soll. Meistens kennt man ja weder den Start noch den Endwert. Wenn ich jz zum Beispiel alle Zeichen zwischen x und y in diesem String haben will:
---x---------y---
dann suche ich mir doch lieber die Positionen und nciht die Entfernung.

MFG

Tomate_Salat

 

[Firestorm87]

Ich schrieb "eine Art Wrapper-Klasse" .... das eine Art war hier schon angebracht, da Ich natürlich weiß, dass außer mit selber niemand diese Klasse als String identifizieren würde
Wie gesagt war ja nur ne Schnapsidee

 

[Tomate_Salat]

Und was verstehst du unter "eine Art Wrapper-Klasse"? ;-)

 

[bygones]

Wie bastel ich mir den eigene Wrapper-Klasse :shock::rtfm:

public class StringWrapper {
 private String theString;
 
 public String substring(int start, int length) {
    return theString.substring(start, start + length);
 }
}

problem halt ist dass man dann nicht [c]"Hallo Welt".substring(0,2);[/c] schreiben kann sondern [c]new StringWrapper("Hallo Welt").substring(0,2)[/c]

ansonsten Groovy zb nutzen aber selbst da muss man ziemlich tricksen
[groovy]
def oldSub = String.metaClass.getMetaMethod("substring", [Integer, Integer] as Class[])
String.metaClass.substring = { int start, int length ->
return oldSub.invoke(delegate, [start, start + length] as Object[])
}
println "Hallo".substring(1,1) // a
[/groovy]
fuer den Fall du willst substring direkt ueberschreiben
wenn du nur die Methode [c]substringLength[/c] an String haben willst:
[groovy]
String.metaClass.substringLength = { int start, int length ->
return delegate.substring(start, start + length)
}
println "Hallo".substring(1,1) // a
[/groovy]

 

[0x7F800000]

Das würde auch mit Scala's impliziten casts nicht soo toll gehen, immer noch zuviel Aufwand :noe:

Was willst du überhaupt mit diesem lowleveligen substring, warum nimmst du nicht einfach Regex-Matcher, und sägst den String so auseinander, wie es dir passt?

 

[Tomate_Salat]

oder ganz simpel

int i = "abcdefg".indexOf("b");
text.substring(i, i+3);

 

[bygones]

oder ganz simpel

int i = "abcdefg".indexOf("b");
text.substring(i, i+3);

auch das wuerde man dann natuerlich in eine hilfsklasse verlagern und code kopien zu vermeiden.

seine Frage war ja eigentlich nur, ob es nicht moeglich ist einen String selbst zu aendern, also ohne utility klassen.

Antwort ist: nein nicht mit Java

dass es zig andere loesungen gibt ist logisch

 

[Tomate_Salat]

auch das wuerde man dann natuerlich in eine hilfsklasse verlagern und code kopien zu vermeiden.

Würde ich nur, wenn ich mir sicher bin dass das noch öfters vor kommt. Was ich eigentl. eher bezweifle

 

[Landei]

Das würde auch mit Scala's impliziten casts nicht soo toll gehen, immer noch zuviel Aufwand :noe:

Häh?

implicit def strWrapper(s : String) = new Object {
   def substringEx(a : Int, b: Int) = s.substring(a, a + b)
}
//--> strwrapper: (String)java.lang.Object{def substringEx(Int,Int): java.lang.String}

"abcdef".substringEx(2,2)
//--> res0: java.lang.String = cd

 

[0x7F800000]

Schön! Nur muss diese Methode irgendwohin, die muss man irgendwie dauernd mitschleppen und importieren, und alleine das wäre imho schon zuviel Lärm um Nichts.

Ein Import einer einzelnen unbedeutenden Hilfsfunktion, mit der sonst kein Mensch was anfangen kann, finde ich grundsätzlich eben nicht so sinnvoll. Wie würde das denn aussehen, wenn man für jedes "+" im Argument eine Extra-Methode schreiben würde, das wäre doch ein Riesenchaos... :bahnhof:

 

[Greek]

Dank euch allen

Damit wäre das geklärt :toll:

 

[Landei]

Schön! Nur muss diese Methode irgendwohin, die muss man irgendwie dauernd mitschleppen und importieren, und alleine das wäre imho schon zuviel Lärm um Nichts.

Ein Import einer einzelnen unbedeutenden Hilfsfunktion, mit der sonst kein Mensch was anfangen kann, finde ich grundsätzlich eben nicht so sinnvoll. Wie würde das denn aussehen, wenn man für jedes "+" im Argument eine Extra-Methode schreiben würde, das wäre doch ein Riesenchaos... :bahnhof:

Nein, das Riesenchaos wäre, wenn man sie nicht mitschleppen müsste, sie also auf einmal aus heiterem Himmel überall sichtbar wäre. In Scala 2.8 hat man Package-Objekte, wo man sie im ganzen Package sichtbar machen kann, aber mehr geht nicht (sofern man nicht Martin O. heißt und in Predef rumwurschteln darf). In einem kleinen, überschaubaren Sichtbarkeitsbereich, wo man eine spezielle Funktion (nun nicht gerade diese hier) öfters braucht, ist das aus meiner Sicht eigentlich kein Problem, sondern eine riesige Erleichterung.

 

[0x7F800000]

In einem [1]kleinen, überschaubaren Sichtbarkeitsbereich, wo man eine spezielle Funktion (nun nicht gerade diese hier) [2]öfters braucht, ist das aus meiner Sicht eigentlich kein Problem, sondern eine riesige Erleichterung.

Das mit Sicherheit. Nur:
[2] wie oft verwendet man schon substring(x,y)? Eigentlich nur bei der Implementierung des Regex-Matchers (oder einer vergleichbaren Konstruktion), und danach praktisch nie wieder...

[1] damit sich diese 3 Zeilen Code lohnen, müsste man imho diese Hilfsfunktion bis zum Ende des Lebens mit sich rumschleppen.

Ich hab definitiv nichts gegen "don't repeat yourself"-Prinzip, aber imho gehört nicht jeder mikroskopischer Unsinn sofort in eine spezielle Utility-Klasse, nur weil man damit 2 Mal im leben 5 Buchstaben sparen kann :autsch: Also, nochmal: ich diskriminiere eigentlich nur diese eine konkrete Methode, weil sie keinen Ersparnis bringt, und vor allem keine auch nur ansatzweise schwer lesbare oder komplizierte Konstrukte verbirgt. Es ist einfach nur reine Gewohnheitssache. Wenn ich sowas sehe:

s.substring(start,start+length);

weiß ich sofort was los ist, wenn ich dagegen sowas entdecke:

substringEx(s,start,length);

dann muss ich erstmal 15 Sekunden dumm guggen und danach die dokumentation auspacken (vorausgesetzt Threadersteller stellt eine zur Verfügung)

Es ist doch schon eine gute Orientierungshilfe: wenn einem für die Methode kein guter Name einfällt, sollte man sich doppelt und dreifach überlegen, ob man das in eine Util-Klasse mit reinnimmt. Was is'n "substringEx" omg...

 

[Tomate_Salat]

*zustimm in jedem Bereich*

 

[bygones]

Wenn ich sowas sehe:

s.substring(start,start+length);

weiß ich sofort was los ist, wenn ich dagegen sowas entdecke:

substringEx(s,start,length);

was aber nicht am Konstrukt liegt sondern an der Gewohnheit. substring ist ein Konstrukt das eben schon in und auswendig bekannt ist. Bei den ersten Begegnungen musste man auch immer schauen was nun der 2. Parameter bedeutet (v.a. weils in anderen Sprachen eben die laenge ist). Somit ist das nicht wirklich ein haltbares Gegenargument.

wobei natuerlich substringEx auch nicht wirklich ein aussagekraeftiger Name ist