Skip to main content
adicionou 146 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Primeiro, está usando apenas uma expressão de condição. O fato dela ter sub-expressões, não faz ter várias condições. A condição continua sendo única.

Não me parece difícil de ler. Acha que seria mais fácil se estivesse em um if, por exemplo?

Bom, algumas pessoas não vão gostar disso, mas o que eu vejo que dá para simplificar é usar uma linguagem que preze pela sintaxe simplificada. Ajudaria um pouquinho.

O que poderia simplificar um pouco o uso da lambda em si, embora isto seja questionável, é criar um método que receba o parâmetro necessário e calcule o resultado retornando para a lambda. Sinceramente costuma ser desnecessário na maior parte dos casos. Eventualmente pode ter um ganho deslocando de local. Faz mais sentido se for usar em mais de um lugar.

Outra possibilidade é ter um método para cada sub-expressão. Tem vantagens e desvantagens.

Fora isto, não vejo como poderia ser mais fácil de ler a não ser organizando melhor:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  pessoa.getIdade() >= 18 &&
                  pessoa.getGenero().equals("Feminino"))
              .collect(Collectors.toList());

Usando métodos:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  ÉMaior(pessoa) &&
                  ÉFeminino(pessoa))
              .collect(Collectors.toList());

Coloquei no GitHub para referência futura.

Algumas pessoas acham que deslocar a real expressão para outro lugar até atrapalha a legibilidade, já que para saber o que realmente está acontecendo é necessário olhar outro lugar. Claro que isto depende um pouco de cada caso e a maneira como está sendo usado.

Primeiro, está usando apenas uma expressão de condição. O fato dela ter sub-expressões, não faz ter várias condições. A condição continua sendo única.

Não me parece difícil de ler. Acha que seria mais fácil se estivesse em um if, por exemplo?

Bom, algumas pessoas não vão gostar disso, mas o que eu vejo que dá para simplificar é usar uma linguagem que preze pela sintaxe simplificada. Ajudaria um pouquinho.

O que poderia simplificar um pouco o uso da lambda em si, embora isto seja questionável, é criar um método que receba o parâmetro necessário e calcule o resultado retornando para a lambda. Sinceramente costuma ser desnecessário na maior parte dos casos. Eventualmente pode ter um ganho deslocando de local. Faz mais sentido se for usar em mais de um lugar.

Outra possibilidade é ter um método para cada sub-expressão. Tem vantagens e desvantagens.

Fora isto, não vejo como poderia ser mais fácil de ler a não ser organizando melhor:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  pessoa.getIdade() >= 18 &&
                  pessoa.getGenero().equals("Feminino"))
              .collect(Collectors.toList());

Usando métodos:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  ÉMaior(pessoa) &&
                  ÉFeminino(pessoa))
              .collect(Collectors.toList());

Algumas pessoas acham que deslocar a real expressão para outro lugar até atrapalha a legibilidade, já que para saber o que realmente está acontecendo é necessário olhar outro lugar. Claro que isto depende um pouco de cada caso e a maneira como está sendo usado.

Primeiro, está usando apenas uma expressão de condição. O fato dela ter sub-expressões, não faz ter várias condições. A condição continua sendo única.

Não me parece difícil de ler. Acha que seria mais fácil se estivesse em um if, por exemplo?

Bom, algumas pessoas não vão gostar disso, mas o que eu vejo que dá para simplificar é usar uma linguagem que preze pela sintaxe simplificada. Ajudaria um pouquinho.

O que poderia simplificar um pouco o uso da lambda em si, embora isto seja questionável, é criar um método que receba o parâmetro necessário e calcule o resultado retornando para a lambda. Sinceramente costuma ser desnecessário na maior parte dos casos. Eventualmente pode ter um ganho deslocando de local. Faz mais sentido se for usar em mais de um lugar.

Outra possibilidade é ter um método para cada sub-expressão. Tem vantagens e desvantagens.

Fora isto, não vejo como poderia ser mais fácil de ler a não ser organizando melhor:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  pessoa.getIdade() >= 18 &&
                  pessoa.getGenero().equals("Feminino"))
              .collect(Collectors.toList());

Usando métodos:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  ÉMaior(pessoa) &&
                  ÉFeminino(pessoa))
              .collect(Collectors.toList());

Coloquei no GitHub para referência futura.

Algumas pessoas acham que deslocar a real expressão para outro lugar até atrapalha a legibilidade, já que para saber o que realmente está acontecendo é necessário olhar outro lugar. Claro que isto depende um pouco de cada caso e a maneira como está sendo usado.

Fonte Link
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

Primeiro, está usando apenas uma expressão de condição. O fato dela ter sub-expressões, não faz ter várias condições. A condição continua sendo única.

Não me parece difícil de ler. Acha que seria mais fácil se estivesse em um if, por exemplo?

Bom, algumas pessoas não vão gostar disso, mas o que eu vejo que dá para simplificar é usar uma linguagem que preze pela sintaxe simplificada. Ajudaria um pouquinho.

O que poderia simplificar um pouco o uso da lambda em si, embora isto seja questionável, é criar um método que receba o parâmetro necessário e calcule o resultado retornando para a lambda. Sinceramente costuma ser desnecessário na maior parte dos casos. Eventualmente pode ter um ganho deslocando de local. Faz mais sentido se for usar em mais de um lugar.

Outra possibilidade é ter um método para cada sub-expressão. Tem vantagens e desvantagens.

Fora isto, não vejo como poderia ser mais fácil de ler a não ser organizando melhor:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  pessoa.getIdade() >= 18 &&
                  pessoa.getGenero().equals("Feminino"))
              .collect(Collectors.toList());

Usando métodos:

List<Pessoa> resultadoPesquisa = pessoas.stream()
              .filter(pessoa ->
                  ÉMaior(pessoa) &&
                  ÉFeminino(pessoa))
              .collect(Collectors.toList());

Algumas pessoas acham que deslocar a real expressão para outro lugar até atrapalha a legibilidade, já que para saber o que realmente está acontecendo é necessário olhar outro lugar. Claro que isto depende um pouco de cada caso e a maneira como está sendo usado.