zamiana listy na stringa

zamiana listy na stringa
AL
  • Rejestracja:ponad 9 lat
  • Ostatnio:około 7 lat
  • Postów:143
0

Witam, mogłoby się wydawać że temat banalny ale jednak...

dostaje sobie taki o to List<integer> result który daje mi id, i chce to spreparować na stringa po przecinkach (każdy id oddzielony przecinkami). na tą chwilę robię to tak :

Kopiuj
  private String prepareJoinedItemIdsForProcedure(List<Integer> result) {
        String idList = "";
        for (Integer s : result ){
            idList += s + ",";

        }
        return idList;
    }

Aczkolwiek powoduje to że nawet jeśli mam podanego jednego id, to na końcu dodaje mi ",". Chciałbym to tak zmienić by na końcu string-a nie było przecinka

edytowany 1x, ostatnio: Altaro
Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:około 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
4
Kopiuj
return result.stream()
          .map(String::valueOf)
          .collect(Collectors.joining(","));

"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 2x, ostatnio: Shalom
AL
  • Rejestracja:ponad 9 lat
  • Ostatnio:około 7 lat
  • Postów:143
0

Wymyśliłem jeszcze to :

Kopiuj

 private String prepareJoinedItemIdsForProcedure(List<Integer> result) {
        String idList = "";
        for (Integer s : result ){
            if (StringUtils.isNotBlank(idList)) {
                idList += ",";
            }
            idList += s;
        }
        return idList;
    }
Zobacz pozostałe 3 komentarze
caer
@Shalom nie ma co strzępić języka, i tak zrobi po swojemu
datdata
A jeśli już koniecznie nie chcesz streamów (które rządzą) to przynajmniej zmień to idList na StringBuildera i dopiero w returnie toString().
bakeraw2
Od wersji javy 1.6 kompilator przy konkatenacji Stringów i tak łączy je za pomocą StringBuilder'a. Mi osobiście najbardziej przypadł do gustu sposób @Shaloma ze stream'ami :)
bogdans
Nawet w 1.8 łączenie 100 tysięcy liter "A"StringBuilderem jest około 6 tysięcy razy szybsze niż operatorem konkatenacji.
AL
  • Rejestracja:ponad 9 lat
  • Ostatnio:około 7 lat
  • Postów:143
0

To nie jest nic skomplikowanego. na początku jest textarea w której wprowadzamy po przecinku identyfikatory, potem tworzona jest lista Integer z separatorem ",". Potem wyrzucane są z tej listy wszystkie nieprawidłowe wpisy np te33kst(zabezpieczenie, by poprzez textarea nie móc wprowadzić na przykład zapytania sql: update, insert, delete) ... itp i pozostawione tylko prawidłowe ID. Potem w pętli przekazuje pojedynczy Id z listy do metody zmieniającej (JPAQuery) datę premiery i do metody zmieniającej status Id-ka, tam Lista<Integer> tych id-ków, preparowana jest jeszcze raz na stringa ( patrz post wyżej ) i dalej parsowane jest to w procedurze sql.

edytowany 2x, ostatnio: Altaro
Zobacz pozostałe 7 komentarzy
niezdecydowany
niezdecydowany
trza być inteligentnym, sorry @tdudzik
krzysiek050
@niezdecydowany Ironia jest wtedy, kiedy każdy może się jej domyśleć, bo nie potrzebuje specjalnej wiedzy. Tutaj @Altaro mógł wziąć to na poważnie więc to był zwykły trolling i wprowadzanie w błąd.
niezdecydowany
niezdecydowany
Nie Ty będziesz decydował czy coś jest ironią.
krzysiek050
@niezdecydowany aleś mi pojechał. Jeszcze w gimnazjum mógłbym to wziąć do siebie. Spóźniłeś się parę lat.
niezdecydowany
niezdecydowany
no, ze 2
krzysiek050
  • Rejestracja:ponad 12 lat
  • Ostatnio:ponad 4 lata
  • Postów:1272
0

@Altaro
Jeżeli chcesz sklejać więcej niz 3-4 stringi, to stosuj StringBuilder. Sama klasa String jest Immutable i sklejanie ich poprzez + jest niewydajne. Jest to do zaakceptowania w prostym sklejeniu, ale w przypadku większej liczby wartości, warto użyć albo wspomnianego StringBuilder, albo jeszcze lepie, gotowych rozwiązań takich jak String.join(), StringJoiner, czy przedstawiony wyżej odpowiedni collector.

AL
  • Rejestracja:ponad 9 lat
  • Ostatnio:około 7 lat
  • Postów:143
0

Przede wszystkim prosiłbym żebyście się tak nie kłócili ;) A co do mojego rozwiązania, uważam za wystarczające, ponieważ nie będzie w nim sklejanych więcej niż~50 stringów z listy. Najczęściej będzie to pojedynczy string lub kilka. Nie będzie to też częścią jakiegoś schedulera czy innego procesu. Będzie to wsadowa zmiana wywoływana przez użytkownika portalu.

Shalom
Tylko ze twoje rozwiązanie jest mało czytelne a do tego nadal dodaje ci ten przecinek na końcu...
AL
może i nie jest najprostszą drogą ale właśnie o to chodzi. Druga sprawa że przecinka mi na końcu już nie dodaje
0

A czemu nie w scali:

Kopiuj
result.mkString(",")
 

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.