A resposta do Math já demonstrou o problema no seu método consultaLivro()
. Mas há ainda outros problemas.
Basicamente, na sua classe Livro
, você tem esses dois campos:
public final ArrayList<Livro> livros = new ArrayList<>();
private String consultaLivros;
Isso significa que cada livro tem vários livros e que cada livro tem uma String
de consulta de livros. Isso não é uma modelagem orientada a objetos adequada, pois a lista de livro não pertence a nenhum livro. A String
de busca também não pertence a um livro. Além disso, o campo livros
é público, e você deve saber que não é boa prática manter campos de instância públicos.
Além disso, você não precisa do campo consultaLivros
para nada, e nem o getter e setter correspondente. Veja no seu método consultaLivro
:
this.setConsultaLivros(input.nextLine());
for (int i = 0; i < livros.size(); i++) {
if (livros.get(i).getTitulo().contains(this.getConsultaLivros())) {
Observe que você lê um dado (consultaLivros
), o coloca dentro do objeto e tira esse mesmo dado deste mesmo objeto. Isso daí é um dado que uma vez lido, só tem sentido dentro desse método, e uma vez finalizado o método, ele perde o sentido de existir. Além disso, só faria sentido armazenar isso na instância (this
) se esse dado fosse transportado para algum outro lugar, o que não é o caso aqui pois o consultaLivros
perde o propósito de existir dentro do mesmo método no qual ele nasce. Logo, isso deveria ser uma variável local, e não um campo do objeto. Assim sendo, já contando com as alterações do Math, o seu método ficaria assim:
public void consultaLivro() {
System.out.print("Digite o nome do título a ser pesquisado: ");
Scanner input = new Scanner(System.in);
String consultaLivros = input.nextLine();
boolean achou = false;
for (int i = 0; i < livros.size(); i++) {
if (livros.get(i).getTitulo().contains(consultaLivros)) {
System.out.println("O livro cadastrado está na posição: " + livros.indexOf(i));
livros.get(i).informacoes();
System.out.println("");
achou = true;
}
if (!achou) {
System.out.println("Título não encontrado!");
System.out.println("");
}
}
E o campo consultaLivros
e o getter e setter correspondentes podem ser apagados.
Como já mencionei antes, não faz sentido um livro ter vários livros ou um livro pertencer a outro livro. Aliás, é pior que isso. Na sua classe principal, você tem isso:
Livro book = new Livro();
// ... várias linhas de código aqui.
book.livros.add(book);
Esses são os únicos dois lugares onde você cria um livro ou o adiciona à lista de livros. Isso significa que:
- O seu sistema só tem um único livro (afinal de contas, você só instanciou ele uma única vez).
- Que a lista de livros vai conter várias vezes o mesmo livro.
- Que um livro contém a si mesmo várias vezes.
- Que um livro pertence a si mesmo múltiplas vezes.
Ou seja, isso daí não vai fazer nada que seja parecido com o que você quer.
Isso pode ser resolvido ao criar uma classe Biblioteca
que contém uma lista de livros:
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;
public class Biblioteca {
private final List<Livro> livros;
public Biblioteca() {
this.livros = new ArrayList<>(10);
}
public void adicionarLivro(Livro livro) {
this.livros.add(livro);
}
public void consultarLivro() {
System.out.print("Digite o nome do título a ser pesquisado: ");
Scanner input = new Scanner(System.in);
String consultaLivros = input.nextLine();
boolean achou = false;
for (int i = 0; i < livros.size(); i++) {
if (livros.get(i).getTitulo().contains(consultaLivros)) {
System.out.println("O livro cadastrado está na posição: " + livros.indexOf(i));
livros.get(i).informacoes();
System.out.println("");
achou = true;
}
if (!achou) {
System.out.println("Título não encontrado!");
System.out.println("");
}
}
}
Além disso, vamos ver o seu método virarPagina
:
public void virarPagina(int pagAtual) {
this.setPagAtual(this.getPagAtual() + 1);
}
Observe que o parâmetro pagAtual
não é utilizado.
Veja bem, no seu código, o book.setAutor(entrada.nextLine());
está alterando o autor do livro, ao invés de estar definindo o autor do livro que está sendo cadastrado. Esse é um outro problema que tende a levar você a cometer enganos que fazem estar sempre alterando o mesmo livro ao invés de criar livros novos. Conceitualmente, quando um livro é cadastrado, ele já contém todas as informações necessárias para ser um livro e considerando que tenham sido cadastradas corretamente, não faz sentido que sejam alteradas. Ou seja, o construtor já deve devolver um objeto Livro
pronto para uso, e não um esqueleto incompleto a ser consertado depois por meio da invocação de um monte de setters.
Outro detalhe, é que os métodos setPagAtual
e virarPagina
não respeitam o número de páginas do livro. Você não pode virar a página se já estiver na última.
Abreviar nomes (ex: pagAtual
) também não é boa prática de programação. Use paginaAtual
.
Considerando tudo isso, eis a sua nova classe Livro
:
public class Livro {
private String titulo;
private String autor;
private String editora;
private int numeroPaginas;
private int paginaAtual;
public Livro(String titulo, String autor, String editora, int numeroPaginas, int paginaAtual) {
if (numeroPaginas <= 0) throw new IllegalArgumentException();
if (paginaAtual <= 0 || paginaAtual > numeroPaginas) throw new IllegalArgumentException();
this.titulo = titulo;
this.autor = autor;
this.editora = editora;
this.numeroPaginas = numeroPaginas;
this.paginaAtual = paginaAtual;
}
public String getTitulo() {
return titulo;
}
public void setTitulo(String titulo) {
this.titulo = titulo;
}
public String getAutor() {
return autor;
}
public void setAutor(String autor) {
this.autor = autor;
}
public String getEditora() {
return editora;
}
public void setEditora(String editora) {
this.editora = editora;
}
public int getNumeroPaginas() {
return numeroPaginas;
}
public void setNumeroPaginas(int numeroPaginas) {
this.numeroPaginas = numeroPaginas;
}
public int getPaginaAtual() {
return paginaAtual;
}
private boolean paginaExiste(int numeroPagina) {
return numeroPagina >= 1 && numeroPagina <= numeroPaginas;
}
public void setPaginaAtual(int novaPaginaAtual) {
if (!paginaExiste(novaPaginaAtual)) throw new IllegalArgumentException();
this.paginaAtual = novaPaginaAtual;
}
public void virarPagina() {
if (paginaExiste(paginaAtual + 1)) this.paginaAtual++;
}
public void voltarPagina() {
if (paginaExiste(paginaAtual - 1)) this.paginaAtual--;
}
}
Não é boa prática de programação misturar lógica de visualização (seja ela System.out.println
, telas e layout desktop, HTML, android, etc) com lógica de negócio (que é aquela que define o que o seu sistema faz e como se comporta). A ideia é que uma mesma lógica de negócio deveria ser capaz de ser expressa e manuseada independente daquilo que é utilizado para exibi-la e interagir com o usuário, o que a torna portável para vários ambientes. Entretanto, sempre que você usa um System.out
, um JOptionPane
ou um pedaço de HTML dentro da sua lógica de negócio, você joga essa premissa por terra. Há ainda outros vários motivos para não se misturar essas coisas (mais sobre isso abaixo).
Assim sendo, retirei os métodos informacoes()
e apresentacao()
da sua classe Livro
acima. Vamos refazer a sua classe Biblioteca
pensando nisso:
import java.util.ArrayList;
import java.util.List;
import java.util.Optional;
public class Biblioteca {
private final List<Livro> livros;
public Biblioteca() {
this.livros = new ArrayList<>(10);
}
public void adicionarLivro(Livro livro) {
this.livros.add(livro);
}
public Optional<Livro> consultarLivro(String consultaLivros) {
for (Livro book : livros) {
if (book.getTitulo().contains(consultaLivros)) return Optional.of(book);
}
return Optional.empty();
}
}
O Optional
é um objeto que pode ou não conter um outro dentro. Se conter, significa que aquilo que era produrado foi encontrado. Se não, é porque não foi encontrado. E note que o método consultaLivro
se preocupa apenas em consultar o livro e devolvê-lo se for encontrado. Não é responsabilidade dele pedir ao usuário qual é o livro a ser consultado e nem exibir esses dados em qualquer lugar. Uma vez que esse método está desacoplado da lógica de visualização, você não vai se ferrar se quiser colocar as informações do livro em um arquivo ao invés de colocar no System.out
. Algo parecido pode ser argumentado acerca do System.in
.
Por fim, as suas classes principais:
import java.util.Scanner;
import java.util.Optional;
public class CadastroBiblioteca {
private final Scanner entrada;
private final Biblioteca biblioteca;
public CadastroBiblioteca() {
this.biblioteca = new Biblioteca();
this.entrada = new Scanner(System.in);
}
public void lerLivro() {
System.out.print("Digite o nome do livro: ");
String titulo = entrada.nextLine();
System.out.print("Digite o nome do autor: ");
String autor = entrada.nextLine();
System.out.print("Digite o nome da editora: ");
String editora = entrada.nextLine();
System.out.print("Digite o número de páginas: ");
int numeroPaginas = Integer.parseInt(entrada.nextLine());
System.out.print("Digite a página atual: ");
int paginaAtual = Integer.parseInt(entrada.nextLine());
biblioteca.adicionarLivro(new Livro(titulo, autor, editora, numeroPaginas, paginaAtual));
}
public void consultarLivro() {
System.out.print("Digite o nome do título a ser pesquisado: ");
String consulta = entrada.nextLine();
Optional<Livro> opt = biblioteca.consultarLivro();
if (!opt.isPresent()) {
System.out.println("Título não encontrado!");
return;
}
Livro livro = opt.orElseThrow(AssertionError::new);
System.out.println("_____________________________________");
System.out.println("Título: " + livro.getTitulo());
System.out.println("Autor: " + livro.getAutor());
System.out.println("Editora: " + livro.getEditora());
System.out.println("Número de páginas: " + livro.getNumeroPaginas());
System.out.println("Página atual: " + livro.getPaginaAtual());
}
}
public class SistemaBiblioteca {
public static void main(String[] args) {
CadastroBiblioteca cadastro = new CadastroBiblioteca();
while (true) {
System.out.println("Bem vindo a sua biblioteca. ");
System.out.println("Digite 1 para NOVO livro.");
System.out.println("Digite 2 para CONSULTAR livro.");
System.out.println("Digite 0 para SAIR.");
String verificarCadastro = entrada.nextLine();
System.out.println("");
switch (verificarCadastro) {
case "0":
System.out.println("Você saiu do programa.");
return;
case "1":
cadastro.lerLivro();
break;
case "2":
cadastro.consultarLivro();
break;
default:
System.out.println("Você digitou uma opção inválida.");
System.out.println("");
break;
}
}
}
}
Dividi em duas para separar o menu da implementação das funcionalidades.
public void informacoes() {
não é um construtor.Livro
tem umArrayList<Livro>
- Ou seja, você diz que um livro tem vários livros. Isso não parece ser uma modelagem orientada a objetos adequada.