Cześć, prosze o potwierdzenie, że w tym przypdaku jest lepiej używać StringBuildera niż Stringów?
StringBuilder:
protected void polishFinalTextBuilder(){
StringBuilder polishBuild = new StringBuilder();
polishBuild.append("\n+++Naprawy niezbedne: \n")
.append(PagesControl.taNecessary.getText())
.append("\n\n+++Naprawy zalecane: \n")
.append(PagesControl.taRecommended.getText())
.append("\n\n+++Akcesoria: \n")
.append(PagesControl.taAccessories.getText())
.append("\n\n+++Uwagi: \nGwarancja tylko na wymienione części! \n")
.append(PagesControl.cautions.getText())
.append("\n\n"+ getBatteryIsSelected())
.append("\n" + getStylusIsSelected())
.append("\n" + getSealOkIsSelected())
.append("\n" + getSealNotIsSelected())
.append("\nSystem operacyjny: ")
.append(PagesControl.cbDeviceSystem.getText())
.append("\nModel lasera: ")
.append(PagesControl.cbLaserModel.getText())
.append("\nTechnik: ")
.append(PagesControl.cbEngineer.getText());
PagesControl.taAllDescription.append(polishBuild.toString());
}
String:
protected String getFinalText() {
if (isPolish())
return ("\n"+ "+++Naprawy niezbedne:"+"\n"+ PagesControl.taNecessary.getText()+"\n"+"\n"
+ "+++Naprawy zalecane:"+"\n"
+PagesControl.taRecommended.getText()+"\n"+"\n"
+ "+++Akcesoria:"+"\n"
+ PagesControl.taAccessories.getText()+"\n"+ "\n"
+ "+++Uwagi:"+"\n"+"Gwarancja tylko na wymienione części!"+"\n"
+ PagesControl.cautions.getText()+"\n"+"\n"
+ getBatteryIsSelected()+"\n"
+ getStylusIsSelected()+"\n"
+ getSealOkIsSelected()+"\n"
+ getSealNotIsSelected()+"\n"
+ "System operacyjny: "
+ PagesControl.cbDeviceSystem.getText()+"\n"
+ "Model lasera: "
+ PagesControl.cbLaserModel.getText()+"\n"
+ "Technik: "+PagesControl.cbEngineer.getText());
if (isEnglish())
return ("\n"+ "+++Necessary repairs:"+"\n"
+ PagesControl.taNecessary.getText()+"\n"+ "\n"
+ "+++Recommended repairs:"+"\n"
+ PagesControl.taRecommended.getText()+"\n"+ "\n"
+ "+++Accessories to buy:"+"\n"
+ PagesControl.taAccessories.getText()+"\n"+ "\n"
+ "+++Cautions:"+"\n"+"Warranty only for exchanged spare parts!"+"\n"
+ PagesControl.cautions.getText()+"\n"+"\n"
+ getBatteryIsSelected()+"\n"
+ getStylusIsSelected()+"\n"
+ getSealOkIsSelected()+"\n"
+ getSealNotIsSelected()+"\n"
+ "Device System: "
+ PagesControl.cbDeviceSystem.getText()+"\n"
+ "Scan engine model: "
+ PagesControl.cbLaserModel.getText()+"\n"
+ "Technican: "
+PagesControl.cbEngineer.getText());
return "";
}
Ogólnie, poczytałem troche na ten temat i lepszym rozwiązaniem jest StringBuilder. Przeważnie należy go stosować, nie tworzy za każdym razem nowych Stringów, tylko operuje na jednym itp.
Wiem że pytanie, banalne, ale chciałbym żebyscie mi to potwierdzili i podpowiedzieli może czy coś jeszcze musiałbym tu poprawić? Dzięki