Javadoc do Collector mostra como coletar elementos de um fluxo em uma nova lista. Existe um one-liner que adicione os resultados em um ArrayList existente?
java
java-8
java-stream
collectors
codefx
fonte
fonte
Collection
"Respostas:
NOTA: a resposta do nosid mostra como adicionar a uma coleção existente usando
forEachOrdered()
. Essa é uma técnica útil e eficaz para alterar as coleções existentes. Minha resposta aborda por que você não deve usar aCollector
para alterar uma coleção existente.A resposta curta é não , pelo menos não em geral, você não deve usar a
Collector
para modificar uma coleção existente.O motivo é que os coletores são projetados para oferecer suporte ao paralelismo, mesmo em coleções que não são seguras para threads. A maneira como eles fazem isso é fazer com que cada thread opere independentemente em sua própria coleção de resultados intermediários. A maneira como cada encadeamento obtém sua própria coleção é chamar o
Collector.supplier()
que é necessário para retornar uma nova coleção a cada vez.Essas coleções de resultados intermediários são mescladas, novamente de maneira confinada por encadeamento, até que haja uma única coleção de resultados. Este é o resultado final da
collect()
operação.Algumas respostas de Balder e assylias sugeriram o uso
Collectors.toCollection()
e a transmissão de um fornecedor que retorna uma lista existente em vez de uma nova. Isso viola o requisito do fornecedor, que é o de retornar uma coleção nova e vazia a cada vez.Isso funcionará para casos simples, como demonstram os exemplos em suas respostas. No entanto, ele falhará, principalmente se o fluxo for executado em paralelo. (Uma versão futura da biblioteca pode mudar de alguma forma imprevista que fará com que ela falhe, mesmo no caso seqüencial.)
Vamos dar um exemplo simples:
Quando executo esse programa, geralmente recebo um
ArrayIndexOutOfBoundsException
. Isso ocorre porque vários threads estão operandoArrayList
, uma estrutura de dados insegura. OK, vamos sincronizar:Isso não falhará mais com uma exceção. Mas em vez do resultado esperado:
dá resultados estranhos como este:
Este é o resultado das operações de acumulação / mesclagem confinadas em threads que descrevi acima. Com um fluxo paralelo, cada encadeamento chama o fornecedor para obter sua própria coleção para acumulação intermediária. Se você passar um fornecedor que retorna a mesma coleção, cada thread anexa seus resultados a essa coleção. Como não há nenhuma ordem entre os encadeamentos, os resultados serão anexados em alguma ordem arbitrária.
Então, quando essas coleções intermediárias são mescladas, isso basicamente mescla a lista consigo mesma. As listas são mescladas usando
List.addAll()
, o que indica que os resultados serão indefinidos se a coleção de origem for modificada durante a operação. Nesse caso,ArrayList.addAll()
faz uma operação de cópia de matriz, por isso acaba se duplicando, o que é uma espécie do que se esperaria, eu acho. (Observe que outras implementações da lista podem ter um comportamento completamente diferente.) De qualquer forma, isso explica os resultados estranhos e os elementos duplicados no destino.Você pode dizer: "Certifico-me de executar meu fluxo seqüencialmente" e vá em frente e escreva um código como este
de qualquer forma. Eu recomendaria não fazer isso. Se você controla o fluxo, com certeza, pode garantir que ele não será executado em paralelo. Espero que um estilo de programação surja onde os fluxos são entregues em vez de coleções. Se alguém lhe entregar um fluxo e você usar esse código, falhará se o fluxo for paralelo. Pior ainda, alguém pode entregar a você um fluxo seqüencial e esse código funcionará bem por um tempo, passará em todos os testes etc. Então, durante algum tempo arbitrário depois, o código em outra parte do sistema poderá mudar para usar fluxos paralelos que causarão seu código quebrar.
OK, lembre-se de ligar
sequential()
para qualquer fluxo antes de usar este código:Claro, você vai se lembrar de fazer isso toda vez, certo? :-) Digamos que você faça. Então, a equipe de desempenho se perguntará por que todas as suas implementações paralelas cuidadosamente criadas não estão fornecendo nenhuma aceleração. E mais uma vez, eles rastrearão o código, o que está forçando todo o fluxo a executar sequencialmente.
Não faça isso.
fonte
toCollection
método retorne uma coleção nova e vazia a cada vez me convence a não fazê-lo. Eu realmente quero quebrar o contrato javadoc das principais classes Java.forEachOrdered
. Os efeitos colaterais incluem adicionar elementos a uma coleção existente, independentemente de ela já ter elementos. Se você deseja que os elementos de um fluxo sejam inseridos em uma nova coleção, usecollect(Collectors.toList())
outoSet()
outoCollection()
.Até onde eu posso ver, todas as outras respostas até agora usaram um coletor para adicionar elementos a um fluxo existente. No entanto, existe uma solução mais curta e funciona para fluxos sequenciais e paralelos. Você pode simplesmente usar o método forEachOrdered em combinação com uma referência de método.
A única restrição é que a origem e o destino são listas diferentes, porque você não tem permissão para fazer alterações na origem de um fluxo, desde que seja processado.
Observe que esta solução funciona para fluxos sequenciais e paralelos. No entanto, ele não se beneficia da simultaneidade. A referência do método passada para forEachOrdered sempre será executada sequencialmente.
fonte
forEach(existing::add)
como possibilidade uma resposta há dois meses . Eu deveria ter acrescentadoforEachOrdered
, bem ...forEachOrdered
vez deforEach
?forEachOrdered
funciona para fluxos sequenciais e paralelos . Por outro lado,forEach
pode executar o objeto de função passado simultaneamente para fluxos paralelos. Nesse caso, o objeto de função deve ser sincronizado corretamente, por exemplo, usando aVector<Integer>
.target::add
. Independentemente de quais threads o método é chamado, não há corrida de dados . Eu esperava que você soubesse disso.A resposta curta é não (ou deveria ser não). EDIT: sim, é possível (veja a resposta dos assylias abaixo), mas continue lendo. EDIT2: mas veja a resposta de Stuart Marks por mais uma razão pela qual você ainda não deve fazê-lo!
A resposta mais longa:
O objetivo dessas construções no Java 8 é introduzir alguns conceitos de Programação Funcional na linguagem; na Programação Funcional, as estruturas de dados normalmente não são modificadas; em vez disso, novas são criadas fora das antigas por meio de transformações como mapa, filtro, dobra / redução e muitas outras.
Se você precisar modificar a lista antiga, basta coletar os itens mapeados em uma nova lista:
e faça
list.addAll(newList)
novamente: se você realmente deve.(ou construa uma nova lista concatenando a antiga e a nova e atribua-a de volta à
list
variável - isso é um pouco mais no espírito do FP do queaddAll
)Quanto à API: mesmo que a API permita (novamente, consulte a resposta de assylias), você deve evitar fazer isso independentemente, pelo menos em geral. É melhor não combater o paradigma (FP) e tentar aprendê-lo, em vez de combatê-lo (mesmo que Java geralmente não seja uma linguagem FP), e apenas recorrer a táticas "mais sujas" se for absolutamente necessário.
A resposta realmente longa: (ou seja, se você incluir o esforço de realmente encontrar e ler uma introdução / livro sobre FP, conforme sugerido)
Para descobrir por que modificar listas existentes geralmente é uma má idéia e leva a um código menos sustentável - a menos que você esteja modificando uma variável local e seu algoritmo seja curto e / ou trivial, o que está fora do escopo da questão da manutenção de códigos - encontre uma boa introdução à Programação Funcional (existem centenas) e comece a ler. Uma explicação de "visualização" seria algo como: é matematicamente mais fácil e mais racional para não modificar dados (na maioria das partes do seu programa) e leva a um nível mais alto e menos técnico (além de mais amigável ao ser humano, uma vez que seu cérebro transições do pensamento imperativo à moda antiga) definições da lógica do programa.
fonte
Erik Allik já deu boas razões, por que você provavelmente não desejará coletar elementos de um fluxo em uma lista existente.
De qualquer forma, você pode usar a seguinte linha, se realmente precisar dessa funcionalidade.
Mas, como Stuart Marks explica em sua resposta, você nunca deve fazer isso, se os fluxos puderem ser paralelos - use por seu próprio risco ...
fonte
Você apenas precisa indicar sua lista original para ser a que
Collectors.toList()
retorna.Aqui está uma demonstração:
E aqui está como você pode adicionar os elementos recém-criados à sua lista original em apenas uma linha.
É isso que este Paradigma de Programação Funcional fornece.
fonte
targetList = sourceList.stream (). flatmap (List :: stream) .collect (Collectors.toList ());
fonte
Concatenaria a lista antiga e a nova lista como fluxos e salvaria os resultados na lista de destinos. Funciona bem em paralelo também.
Usarei o exemplo de resposta aceita dada por Stuart Marks:
Espero que ajude.
fonte