So Im having some trouble coming up with a idea to make it work I´ve at it for the past 4 hours and still cant put it to work I returned to where I start to make things simple to understand.
Here´s the deal I have 2 menus on the first one I print all of the arrayList like it is supposed but one the second menu i can only print the first five products does anyone has a solution?
heres the code:
package projeto;
import java.util.*;
public class GestorDeCartoes {
public static void main(String[] args){
Scanner scanner = new Scanner(System.in);
//inicializa a lista de clientes e a de produtos
ArrayList<Cliente> listaClientes = new ArrayList<>();
ArrayList<Produto> listaProdutos = new ArrayList<>();
long id = 2021129038;
int pos = 1;
int menu, menuCompra;
do {
System.out.println("\nEscolha a opção:\n1-Inserir Clientes\n2-Listar Clientes\n3-Inserir Produtos\n4-Listar Produtos\n5-Menu de Compra\n6-Sair");
menu = scanner.nextInt();
scanner.nextLine();
switch (menu)
{
case (1):
System.out.println("Insira os dados do cliente:");
String inputCliente = scanner.nextLine();
String[] dadosCliente = inputCliente.split(",");
String nome = dadosCliente[0];
int idade = Integer.parseInt(dadosCliente[1]);
double pontos = Double.parseDouble(dadosCliente[2]);
//verifica se o nome introduzido é numerico
if(nome.matches("-?\\d+(\\.\\d+)?")){
System.out.println("Dados inválidos");
break;
}
//verifica de a idade e os pontos introduzidos sao positivos
if ((idade < 0) || (pontos < 0)){
System.out.println("Dados inválidos");
break;
}
//cria o cliente, o cartao, adiciona a lista de clientes e incrementa o id
Cartao cartao = new Cartao(id, pontos);
Cliente cliente = new Cliente( pos , id, nome , idade , cartao );
listaClientes.add(cliente);
id++;
pos++;
break;
case (2):
//lista os Clientes
for (Cliente ClienteIterador : listaClientes) {
System.out.println(ClienteIterador.toString());
}
break;
case (3):
System.out.println("Insira os dados do produto:");
//le a linha a ser introduzida
String inputProduto = scanner.nextLine();
//separa a linha lida pelas virgulas num vetor
String[] dadosProduto = inputProduto.split(",");
//introduz os dados lidos para o vetor nas variaveis
int custo = Integer.parseInt(dadosProduto[1]);
String nomeProduto = dadosProduto[0];
//verifica se o nome introduzido é numerico
if(nomeProduto.matches("-?\\d+(\\.\\d+)?")){
System.out.println("Dados inválidos");
break;
}
//verifica se o custo introduzido é positivo
if (custo < 0){
System.out.println("Dados inválidos");
break;
}
//cria o produto e adiciona a lista
Produto produto = new Produto( id , custo , nomeProduto );
listaProdutos.add(produto);
id++;
break;
case (4):
//lista os produtos
for (Produto produtoIterador : listaProdutos) {
System.out.println(produtoIterador.toString());
}
break;
case (5):
System.out.println("\nMENU DE COMPRA\n");
do {
System.out.println("1-Ver a Lista de Produtos\n2-Comprar Produto\n3-Consultar o Saldo\n4-Sair\n5-Voltar\nEscolha a opção:");
menuCompra = scanner.nextInt();
switch (menuCompra) {
case (1):
//lista os produtos
for (Produto produtoIterador : listaProdutos) {
System.out.println(produtoIterador.toString());
}
break;
case (2):
System.out.println("Introduza o nome do produto que quer comprar\n");
String compraProdutoString = scanner.next();
//verifica se o nome introduzido é numerico
if(compraProdutoString.matches("-?\\d+(\\.\\d+)?")){
System.out.println("Dados inválidos");
break;
}
//filtra a lista de produtos pelo produto com o nome igual ao que queremos comprar
Produto produtoAComprar = null;
for (Produto compraProduto : listaProdutos) {
if (compraProduto.obterNome().equals(compraProdutoString)) {
produtoAComprar = compraProduto;
break;
}
continue;
}
if (produtoAComprar == null) {
System.out.println("Produto inválido");
break;
}
System.out.println("Introduza o nome do cliente:\n");
String compraClienteString = scanner.next();
//verifica se o nome introduzido é numerico
if(compraClienteString.matches("-?\\d+(\\.\\d+)?")){
System.out.println("Dados inválidos");
break;
}
//filtra a lista de clientes pelo cliente com o nome igual ao que queremos utilizar
for (Cliente compraCliente : listaClientes) {
if (compraCliente.obterNome().equals(compraClienteString)) {
compraCliente.comprar(produtoAComprar);
break;
}
continue;
}
break;
case (3):
//mostra o saldo do cliente
System.out.print("Introduza o nome do cliente:");
String ConsultaSaldoString = scanner.next();
for (Cliente compraCliente : listaClientes) {
if (compraCliente.obterNome().equals(ConsultaSaldoString)) {
compraCliente.mostrarSaldo();
break;
}
continue;
}
break;
case (4):
//fecha o scanner e sai do programa
System.out.println("Volte Sempre");
scanner.close();
return;
case(5):
break;
}
} while (menuCompra!= 5);
break;
case (6):
//fecha o scanner e sai do programa
System.out.println("Volte Sempre");
scanner.close();
break;
default:
System.out.println("Escolha um numero entre 1 e 5");
break;
}
} while (menu != 6);
}
}
Please, I need help to make this script work.
I dont know if I can make maths with an string.
I need a program that save numbers inside an array and after that i need to calculate the average betwen those numbers.
When trying to compile this program the compiler only gives me this message:
Basquete2.java:6: error: illegal start of expression
private Pontos [] pontos;
^
1 error
And this is the complete Script:
import javax.swing.JOptionPane;
public class Basquete {
public static void main(String[]args){
//criando array nulo
private Pontos [] pontos;
//variaveis
String desicaoUsuario = "sim";
int i = 0;
int pos = -1;
int soma = 0;
int divisor = 0;
float media;
while(desicaoUsuario == "sim" ){
if(pontos == null){
//criando array
pontos = new Pontos[1];
//posicionando
pos = 0;
//inserindo dados pelo usuario na posiçao
pontos[pos] = JOptionPane.showInputDialog("Informe o número:");
//pergunta se deseja continuar
desicaoUsuario = JOptionPane.showInputDialog("Deseja adicionar outro número? sim/nao");
}else{
//criando novo array guardando os objetos
Pontos [] backup = pontos;
//ampliando o array original
pontos = new Pontos[pontos.lenght + 1];
//copiando objetos para o array original
for(int i = 0; i < backup.lenght; i++){
pontos[i] = backup[i];
}
//posicionando
pos = backup.lenght;
//ciando objeto na posição pronta para inserir
pontos[pos] = JOptionPane.showInputDialog("Informe o número:");
//pergunta se deseja continuar
desicaoUsuario = JOptionPane.showInputDialog("Deseja adicionar outro número? sim/nao");
}
}
//mostrando mensagem
JOptionPane.showMessageDialog("Vamos calcular a media de pontos do time por partida!");
//calculando soma
i = 0;
while(i < pontos.lenght){
soma = soma + pontos[i];
i++;
}
//calcula media
divisor = pontos.lenght;
media = soma / divisor;
//mostra resultado da media
JOptionPane.showMessageDialog("A media de pontos é de " + media + " pontos por partida!");
//saida do sistema
System.exit(0);
}
}
I've got the following code:
It's a class used to store a monthly summary of expenses. voci is an arraylist used to store the expenses (name and cost). mese is the month of the summary, anno the year of summary. My problem is that when I call the showRiepilogo() method i must see the list of the expenses of that month but i can't see nothing. Someone can tell me why please?
class RiepilogoMensile extends Riepilogo {
private ArrayList<Voce> voci = new ArrayList<Voce>();
private int mese;
private int anno;
public RiepilogoMensile(int mese, int anno, String autore, ArrayList<Voce> voci) {
super(autore);
this.mese = mese;
this.anno = anno;
// il blocco successivo è necessario???in teoria il costruttore viene invocato solo una volta.. bisogna cambiare solo la set e la get?.. provo!
// aggiunta delle voci
boolean voceExist = false;
// variabile temporanea che contiene gli anni da aggiungere
ArrayList<Voce> oldVoci = new ArrayList<Voce>();
oldVoci.addAll(this.voci);
ArrayList<Voce> newVoci = new ArrayList<Voce>();
newVoci.addAll(voci);
// controllo se le voci da aggiungere sono già presenti
for (Voce a : oldVoci) {
voceExist = false;
for (int i = 0; i < newVoci.size(); i++) {
if (a.equals(newVoci.get(i))) {
voceExist = true;
break;
}
}
if (!voceExist)
voci.add(a);
}
// test costruttore
System.out.println("Test costruttore voci--> INIZIO");
for(Voce v:voci)
System.out.println(v.getNome()+" "+v.getSpesa());
System.out.println("Test costruttore voci--> FINE");
}
public ArrayList<Voce> getVoci() {
ArrayList<Voce> temp = new ArrayList<Voce>();
temp.addAll(this.voci);
return (temp);
}
public void setVoci(ArrayList<Voce> voci) {
// aggiunta delle voci
boolean voceExist = false;
// variabile temporanea che contiene gli anni da aggiungere
ArrayList<Voce> oldVoci = new ArrayList<Voce>();
oldVoci.addAll(this.voci);
ArrayList<Voce> newVoci = new ArrayList<Voce>();
newVoci.addAll(voci);
// controllo se le voci da aggiungere sono già presenti
for (Voce a : oldVoci) {
voceExist = false;
for (int i = 0; i < newVoci.size(); i++) {
if (a.equals(newVoci.get(i))) {
voceExist = true;
break;
}
}
if (!voceExist)
this.voci.add(a);
}
}
#Override
public void showRiepilogo() {
String messaggio = "\nRiepilogo del " + anno + " di " + Voce.intToString(mese) + " creato da " + getAutore()
+ ": ";
System.out.println(messaggio);
utils.showRow(messaggio.length() - 2, '-'); // tolgo 2 perchè non devo considerare \n
System.out.println();
System.out.println("Test stampa voci in showRiepilogo() voci--> INIZIO");
// stampa voci
for (Voce v : voci)
System.out.println(v.getNome() + " " + v.getSpesa() + " EURO " + (v.getSpesa() > 0 ? "<--ENTRATA" : "USCITA-->"));
for(int i=0;i<voci.size();i++) {
System.out.println(voci.get(i).getNome() + " " + voci.get(i).getSpesa() + " EURO " + (voci.get(i).getSpesa() > 0 ? "<--ENTRATA" : "USCITA-->"));
}
System.out.println("Test stampa voci in showRiepilogo() voci--> FINE");
}
}
When I call showRiepilogo() from main i can't see nothing instead i could see the content of voci ArrayList.
riepiloghiMensili.get(index).showRiepilogo();
Someone know why?
Thanks in advice, Elias.
You have a very weird constructor, you try to loop over the member this.voci but since your doing this in the constructor this.voci will always be an empty array. Look at the declaration:
private ArrayList<Voce> voci = new ArrayList<Voce>();
Your setVoci(...) method will fail for the same reason, this.voci is empty and the for loop will never be entered and no objects added to the array.
Change your constructor to just set the array to the given parameter
this.voci = voci
I would also recommend to do the same for the setVoci method or to rename it because a set... method is expected to set a member to the given parameter.
the reference of the ArrayList I've tried and it doesn't works well!Doing as what you tell to do assigns the same ArrayList to all the istances of RiepilogoMensile.
I've done that and all works fine:
class RiepilogoMensile extends Riepilogo {
private ArrayList<Voce> voci = new ArrayList<Voce>();
private int mese;
private int anno;
public RiepilogoMensile(int mese, int anno, String autore, ArrayList<Voce> voci) {
super(autore);
this.mese = mese;
this.anno = anno;
this.voci.addAll(voci);
}
public ArrayList<Voce> getVoci() {
ArrayList<Voce> temp = new ArrayList<Voce>();
temp.addAll(this.voci);
return (temp);
}
public void setVoci(ArrayList<Voce> voci) {
this.voci.addAll(voci);
// aggiunta delle voci
boolean voceExist = false;
// variabile temporanea che contiene gli anni da aggiungere
ArrayList<Voce> oldVoci = new ArrayList<Voce>();
oldVoci.addAll(this.voci);
ArrayList<Voce> newVoci = new ArrayList<Voce>();
newVoci.addAll(voci);
// controllo se le voci da aggiungere sono già presenti
for (Voce a : oldVoci) {
voceExist = false;
for (int i = 0; i < newVoci.size(); i++) {
if (a.equals(newVoci.get(i))) {
voceExist = true;
break;
}
}
if (!voceExist)
this.voci.add(a);
}
}
Thanks insted!
Bye, Elias.
Well im trying to change an array to arraylist, the create account seems to be working but the "show accounts" method keeps returning me that there's not account register.
this is the add account method (the commentary's are the original array)
//Metodo para agregar Cuentas
public void agregarCuenta(Cuenta c){
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
cuentas.add(c);
/*
Cuenta aux[] = new Cuenta[cuentas.length+1];
System.arraycopy(cuentas, 0, aux, 0, cuentas.length);
aux[aux.length-1] = c;
cuentas = aux;
*/
}
and this is the show accounts method
public static void mostrarAhorro()
{
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
//Si no hay cuentas en el arreglo, nos muestra el siguiente error
if (cuentas.size() == 0)
{
System.out.println("No hay cuentas registradas.");
return;
}
System.out.println("****Lista de cuentas de ahorro****");
//For para mostrar todas las cuentas de ahorro
for (int i = 0; i<cuentas.size();i++) {
if (cuentas.get(i) instanceof CuentaAhorros)
{
System.out.println("Id de cuenta: " + (i+1) +"\n"+ cuentas);
}
}
}
this is the rest of the code
package Programas;
import java.util.Scanner;
import Banco.Cuenta;
import Banco.CuentaAhorros;
import Banco.CuentaCheques;
import Banco.Movimientos;
import java.text.DecimalFormat;
import java.util.ArrayList;
public class programa1 {
//Instanciar scanner y formato de moneda
private static Scanner sc = new Scanner(System.in);
private static DecimalFormat MONEDA = new DecimalFormat("$#,###.00");;
//Arreglo de clases instanciada.
//private static Cuenta[] cuentas = new Cuenta[0];
//private static Movimientos [] movimientos = new Movimientos[0];
public static void main(String[] args) {
//ArrayList cuentas = new ArrayList();
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
ArrayList<Movimientos> movimientos = new ArrayList<Movimientos>();
//Selec para mantener el while corriendo, y while para mantener el menu.
int selec = 0;
while (selec <7)
{
//Imprimir lista de opciones
System.out.println("Seleccione una opcion: "+ "\n1.-Crear cuenta de ahorros."+ "\n2.-Crear cuenta de cheques."+ "\n3.-Mostrar listado de cuentas de ahorro."+ "\n4.-Mostrar listado de cuentas de cheques."+ "\n5.-Mostrar listado de todas las cuentas."+ "\n6.-Operaciones de una cuenta."+ "\n7.-Salir");
//Pedirle al usuario una opcion
selec = sc.nextInt();
//Switch para que cuando selec cambie se ejecute una opcion.
switch (selec)
{
//Caso 1 para agregar Cuenta de ahorros
case 1:
System.out.println("Ha escogido crear una cuenta de ahorros."+ "\nIngrese el saldo inicial, y la tasa de interes: ");
//Instancia de Cuenta tipo ahorros
Cuenta c1 = new CuentaAhorros(sc.nextDouble(),sc.nextDouble());
cuentas.add(c1);
System.out.println("\nCuenta de ahorros agregada exitosamente."+ "\nTotal de cuentas: " + cuentas.size() + "\nfecha de ingreso: " + movimientos.size());
break;
//Caso 2 para agregar cuenta de ahorros.
case 2:
System.out.println("Ha escogido crear una cuenta de cheques."+ "\nIngrese el saldo inicial, y el costo de manejo de la cuenta: ");
//Instancia de cuenta tipo cheques
Cuenta c2 = new CuentaCheques(sc.nextDouble(),sc.nextDouble());
cuentas.add(c2);
System.out.println("\nCuenta de cheques agregada exitosamente."+ "\nTotal de cuentas: "+ cuentas.size());
break;
//Caso para mostrar la lista de cuentas de ahorro
case 3:
System.out.println("Listado de cuentas de ahorro: ");
mostrarAhorro();
break;
//Caso para mostrar la lista de cuentas de cheques
case 4:
System.out.println("Listado de cuentas de Cheques: ");
mostrarCheques();
break;
//Caso para mostrar todas las cuentas
case 5:
System.out.println("Listado de todas las cuentas: ");
mostrarCuentas();
break;
//Caso para realizar operaciones de una cuenta
case 6:
operacionesCuenta();
break;
}
}
}
//Metodo para agregar Cuentas
public void agregarCuenta(Cuenta c){
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
cuentas.add(c);
//cuentas = aux;
/*
Cuenta aux[] = new Cuenta[cuentas.length+1];
System.arraycopy(cuentas, 0, aux, 0, cuentas.length);
aux[aux.length-1] = c;
cuentas = aux;
*/
}
//Metodo para mostrar las cuentas de tipo ahorro
public static void mostrarAhorro()
{
ArrayList<Cuenta> cuentas = new ArrayList<>();
//Si no hay cuentas en el arreglo, nos muestra el siguiente error
if (cuentas.isEmpty())
{
System.out.println("No hay cuentas registradas.");
return;
}
System.out.println("****Lista de cuentas de ahorro****");
//For para mostrar todas las cuentas de ahorro
for (int i = 0; i<cuentas.size();i++) {
if (cuentas.get(i) instanceof CuentaAhorros)
{
System.out.println("Id de cuenta: " + (i+1) +"\n"+ cuentas);
}
}
}
//Metodo para cuenta cheques
public static void mostrarCheques()
{
//Si no hay cuentas en el arreglo, nos muestra el siguiente error
ArrayList<Cuenta> cuentas = new ArrayList<>();
if (cuentas.size() == 0)
{
System.out.println("No hay cuentas registradas.");
return;
}
System.out.println("****Lista de cuentas de Cheques****");
//For para mostrar todas las cuentas de ahorro
for (int i = 0; i<cuentas.size();i++) {
if (cuentas.get(i) instanceof CuentaCheques)
{
//CuentaCheques a = (CuentaCheques)cuentas.get(1);
//System.out.println("Id de cuenta: " + (i+1) +"\n"+ a);
System.out.println(cuentas);
}
}
}
public static void mostrarCuentas()
{
//Si no hay cuentas en el arreglo, nos muestra el siguiente error
ArrayList<Cuenta> cuentas = new ArrayList<>();
if (cuentas.size() == 0)
{
System.out.println("No hay cuentas registradas.");
return;
}
System.out.println("****Lista de todas las cuentas****");
//For para mostrar todas las cuentas de ahorro
for (int i = 0; i<cuentas.size();i++) {
if (cuentas.get(i) instanceof CuentaCheques)
{
CuentaCheques a = (CuentaCheques)cuentas.get(i);
System.out.println("Tipo de cuenta: Cheques. "+ "\nId de cuenta: " + (i+1) +"\n"+ a.toString());
}
else
{
CuentaAhorros a = (CuentaAhorros)cuentas.get(i);
System.out.println("Tipo de cuenta: Ahorro."+ "\nId de cuenta: "+ (i+1) +"\n"+ a.toString());
}
}
}
public static void operacionesCuenta()
{
System.out.println("Seleccione la Id de la cuenta que desea usar: ");
int id = (sc.nextInt() -1);
ArrayList<Cuenta> cuentas = new ArrayList<>();
if(id > cuentas.size())
{
System.out.println("Escogio una cuenta que no existe.");
return;
}
if(cuentas.get(id) instanceof CuentaCheques)
{
int choice = 0;
while(choice<6)
{
CuentaCheques a = (CuentaCheques)cuentas.get(id);
System.out.println("Selecciono cuenta de cheques."+ "\n1.-Consultar Saldo."+ "\n2.-Retirar."+ "\n3.-Depositar." + "\n4.-Aplicar manejo de cuenta."+ "\n5.-Ver informacion de la cuenta."+ "\n6.-Salir."+ "\nSeleccione una opcion:");
choice = sc.nextInt();
switch(choice)
{
case 1:
System.out.println("\nEl saldo actual de la cuenta es: " +MONEDA.format(a.getSaldo()));
break;
case 2:
System.out.println("\nIngrese la cantidad que desea retirar: ");
double retirarAhorros = sc.nextDouble();
a.retirar(retirarAhorros);
break;
case 3:
System.out.println("\nIngrese la cantidad que desea depositar: ");
double depositarAhorros = sc.nextDouble();
a.depositar(depositarAhorros);
break;
case 4:
System.out.println("Selecciono aplicar manejo de la cuenta.");
a.aplicarCostoManejoCuenta();
break;
case 5:
System.out.println("Selecciono ver informacion de la cuenta.");
System.out.println(a);
break;
}
}
}
else
{
int choice2 = 0;
while(choice2<6)
{
CuentaAhorros a = (CuentaAhorros)cuentas.get(id);
System.out.println("Escogio cuenta de ahorros."+ "\n1.-Consultar Saldo."+ "\n2.-Retirar."+ "\n3.-Depositar."+ "\n4.-Aplicar manejo de cuenta."+ "\n5.-Ver informacion de la cuenta."+ "\n6.-Salir."+ "\nSeleccione una opcion:");
choice2 = sc.nextInt();
switch(choice2)
{
case 1:
System.out.println("\nEl saldo actual de la cuenta es: " +MONEDA.format(a.getSaldo()));
break;
case 2:
System.out.println("\nIngrese la cantidad que desea retirar: ");
double retirarCheques = sc.nextDouble();
a.retirar(retirarCheques);
break;
case 3:
System.out.println("\nIngrese la cantidad que desea depositar: ");
double depositarCheques = sc.nextDouble();
a.depositar(depositarCheques);
break;
case 4:
System.out.println("Selecciono aplicar manejo de la cuenta.");
a.aplicarInteres();
break;
case 5:
System.out.println("Selecciono ver informacion de la cuenta.");
System.out.println(a);
break;
}
}
}
}
}
The problem is, every time you call the mostrarAhorro method, and even every time you call the agregarCuenta method, you are overwriting the cuentas array list with:
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
Every time you call this, it creates an empty array and ignores any data it previously had.
You only need to create it once and store it as a member variable in the class.
ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
//Metodo para agregar Cuentas
public void agregarCuenta(Cuenta c){
cuentas.add(c);
/*
Cuenta aux[] = new Cuenta[cuentas.length+1];
System.arraycopy(cuentas, 0, aux, 0, cuentas.length);
aux[aux.length-1] = c;
cuentas = aux;
*/
}
public static void mostrarAhorro()
{
//Si no hay cuentas en el arreglo, nos muestra el siguiente error
if (cuentas.size() == 0)
{
System.out.println("No hay cuentas registradas.");
return;
}
System.out.println("****Lista de cuentas de ahorro****");
//For para mostrar todas las cuentas de ahorro
for (int i = 0; i<cuentas.size();i++) {
if (cuentas.get(i) instanceof CuentaAhorros)
{
System.out.println("Id de cuenta: " + (i+1) +"\n"+ cuentas);
}
}
}
EDIT: You need to declare the cuentas array at the top of your class so all methods have access to it. Delete the creation of cuentas in the main method as well.
public class programa1 {
//Instanciar scanner y formato de moneda
private static Scanner sc = new Scanner(System.in);
private static DecimalFormat MONEDA = new DecimalFormat("$#,###.00");
private ArrayList<Cuenta> cuentas = new ArrayList<Cuenta>();
//Arreglo de clases instanciada.
//private static Cuenta[] cuentas = new Cuenta[0];
//private static Movimientos [] movimientos = new Movimientos[0];
public static void main(String[] args) {
//ArrayList cuentas = new ArrayList();
ArrayList<Movimientos> movimientos = new ArrayList<Movimientos>();
You are creating a complete new instance of the arrayList in the display method.
If this list is common across the current run , you can just declare it as
private static List cuentaList = new ArrayList();
I know there are a lot of questions about that topic. In fact, I was searching in others questions and I tried to use some solutions but it was not enough.
I have to do a program that simulate puffball, using backtracking, and now I am developing the moves, and when I do a move I need to save the table because is a possible solution, so THE PROBLEM IS NEXT:
When a I try to keep off a table the first one save it right but the next is added and overwrite the earlier. I show you the code, it is not all, only methods and other info that I think can be useful for all of you.
In main() you can see how I get all information, and then when I start to play, I call to soplarBola(), It is here where you could see the problem.
Excuse me, because all is in Spanish. I hope you could understand my handicap.
Thanks a lot!
public class Main {
static Scanner sc = new Scanner(System.in);
static int[] dimension = new int[2];
static int numPoscionesFinales;
static int numBolas;
static ArrayList<Integer> totalIdBolas;
static ArrayList<ArrayList<Integer>> totalesIdBolas;
static ArrayList<int[]> coordenadasFinales;
static ArrayList<int[]> coordenadasBolas;
static Casilla ca;
static ArrayList<Casilla> tablero;
static ArrayList<ArrayList<Casilla>> tablerosSol;
static ArrayList<Integer> bolasSopladas; //Recoge todos los caminos posibles
static ArrayList<ArrayList<Casilla>> tablerosVisitados;
static ArrayList<ArrayList<Integer>> caminos; //Recoge los int[] que contienen caminos solucion
public static void main(String[] args) {
//Dimensiones del tablero
//Numero de posiciones finales que son las mismas que bolas habra
//Generamos un ArrayList con los idBolas posibles para despues permutar todas las posibles soluciones y generar los tableros solucion con los que compararemos los tableros que vayamos creando
//coordenadas de posiciones finales
//coordenadas de bolas
crearTablero(dimension);
crearTablerosSol(tablero,totalesIdBolas);
bolasSopladas = new ArrayList<Integer>();
resolverSoplarBola(bolasSopladas,tablero,totalesIdBolas,tablerosVisitados, caminos);
}
/* Esta es la semilla de la que nacerá el árbol solucion */
public static void resolverSoplarBola(ArrayList<Integer> bolasSopladas, ArrayList<Casilla> ta,
ArrayList<ArrayList<Integer>> totalesIdBolas, ArrayList<ArrayList<Casilla>> tablerosVisit, ArrayList<ArrayList<Integer>> caminos) {
ArrayList<Integer> solParcial = bolasSopladas;
int nBolas = 1;
caminos = new ArrayList<ArrayList<Integer>>();
//imprimirTab1(ta,nBolas);
while(nBolas<numBolas){
//Sea b la bola que sopla ...sopla y mueve todas las bolas generando un nuevo tablero añadido a tablerosVisitados
tablerosVisit.add(soplarBola(nBolas,ta));
// imprimirTableros(tablerosVisitados,nBolas);
//y calculamos el nuevo tablero a partir de tablero_actual - ta - y...
//solucionesParciales - bolasSopladas - añade b
solParcial.add(nBolas);
if(esTableroSolucion(ta, tablerosSol)){
caminos.add(bolasSopladas);
}else if(esTableroVisitado(ta, tablerosVisit)){
//se para aquí y no avanzará
break;
}else{
resolverSoplarBola(solParcial, ta, totalesIdBolas, tablerosVisit, caminos);
}
nBolas++;
}
}
private static void imprimirTab1(ArrayList<Casilla> ta, String m) {
Iterator<Casilla> itTab= ta.iterator();
System.out.println("");
System.out.println("Tablero actual: " + m);
for(int i=1; i<=dimension[0];i++){
for(int j=1; j<=dimension[1]; j++){
System.out.print("["+itTab.next().idBola+"] ");
}
System.out.println(".");
}
}
private static void imprimirTabVisitados(ArrayList<ArrayList<Casilla>> tabVisitados){
ArrayList<ArrayList<Casilla>> tabVis = tabVisitados;
Iterator<ArrayList<Casilla>> itVis = tabVis.iterator();
ArrayList<Casilla> tabN;
//Los tableros que se imprimen aquí ya han sido comprobados de que no están duplicados
while(itVis.hasNext()){
tabN = itVis.next();
imprimirTab1(tabN," visitados");
}
}
public static ArrayList<Casilla> soplarBola(int idBola,ArrayList<Casilla> tabl){
tablerosVisitados = new ArrayList<ArrayList<Casilla>>();
//ArrayList<Casilla> tabl = tab;
boolean movimientoHecho = false;
Iterator<Casilla> it = tabl.iterator();
int x =1;
int y =1;
Casilla casi;
Casilla casi2;
boolean hayMovimiento = false;
int cont = 0;
idBola = 3;
while(it.hasNext()){//Recorremos todo el tablero buscando la bola con idBola para obtener sus coordenadas
casi = it.next();
if(casi.idBola == idBola){
x = casi.coordX;
y = casi.coordY;
}
}
**tablerosVisitados.add(cont, tabl);
// imprimirTab1(tablerosVisitados.get(0), "0");
imprimirTodo(tablerosVisitados, "inicial");** <- //Here is all OK, print the rigth table.
//MOVIMIENTOS EN LA FILA
it = tabl.iterator();
//Vamos a analizar las filas desde el borde hasta la bola
casi = casillaBuscada(x,1,tabl); //Desde la izq
while(casi.coordY < y && casi.coordX == x){
//coordenadas que están a la izquierda de la idBola
if(casi.idBola!=0){ //Si hay bola no se podrá mover a esa posicion
hayMovimiento=false;
}else{
hayMovimiento=true; //si no hay bola, es que hay hueco y se puede mover a esa posicion
}
casi2 =casillaBuscada(casi.coordX,casi.coordY+1,tabl);
if(hayMovimiento && casi2.getIdBola()!=0 && casi2.getIdBola()!=idBola){ //si hay movimiento y en la posterior hay bola, se traslada a ocupar ese hueco
casi.setIdBola(casi2.getIdBola());
casi2.setIdBola(0); //Desplazamiento a la derecha cambiado el idBola
movimientoHecho = true;
}
casi = it.next();//pasamos a la siguiente casilla
}
**//imprimirTab1(tabl, "1"); If I use that line print the rigth table in that moment**
if(movimientoHecho){
cont++;
tablerosVisitados.add(cont, tabl);
//tablerosVisitados.add(tabl);
imprimirTodo(tablerosVisitados,"1");
movimientoHecho=false;
}
//coordenadas que están a la derecha de la idBola
casi = casillaBuscada(x,dimension[1],tabl); //nos colocamos a la derecha de la bola y vamos ---->>>
while(casi.coordY > y && casi.coordX == x){
if(casi.idBola!=0){ //Si hay bola no se podrá mover a esa posicion
hayMovimiento=false;
}else{
hayMovimiento=true; //si no hay bola, es que hay hueco y se puede mover a esa posicion
}
casi2 =casillaBuscada(casi.coordX,casi.coordY-1,tabl);
if(hayMovimiento && casi2.getIdBola()!=0){ //si hay movimiento y en la posterior hay bola, se traslada a ocupar ese hueco
casi.setIdBola(casi2.getIdBola());
casi2.setIdBola(0); //Desplazamiento a la derecha cambiado el idBola
movimientoHecho=true;
}
casi = it.next();//pasamos a la siguiente casilla
}
// imprimirTab1(tabl,"2");
if(movimientoHecho){
cont++;
**tablerosVisitados.add(cont, tabl);**
//This is the line of horror. Delete the first one and set all table like that
//tablerosVisitados.add(tabl);
imprimirTodo(tablerosVisitados,"2");
movimientoHecho=false;
}
//MOVIMIENTO EN LA COLUMNA
it = tabl.iterator();
//Vamos a analizar las columnas desde arriba hacia abajo
casi = casillaBuscada(1,y,tabl); //Desde arriba
while(casi.coordY == y && casi.coordX <= x){
//coordenadas que están por encima de la idBola
if(casi.idBola!=0){ //Si hay bola no se podrá mover a esa posicion
hayMovimiento=false;
}else{
hayMovimiento=true; //si no hay bola, es que hay hueco y se puede mover a esa posicion
}
casi2 =casillaBuscada(casi.coordX-1,casi.coordY,tabl);
if(hayMovimiento && casi2.getIdBola()!=0 && casi2.getIdBola() != idBola){ //si hay movimiento y en la posterior hay bola, se traslada a ocupar ese hueco
casi.setIdBola(casi2.getIdBola());
casi2.setIdBola(0); //Desplazamiento hacia arriba cambiado el idBola
movimientoHecho=true;
}
casi = casi2;//pasamos a la siguiente casilla
}
//imprimirTab1(tabl,"3");
if(movimientoHecho){
cont++;
**tablerosVisitados.add(cont, tabl);** //This is the line of horror. Delete the first one and set all table like that
//tablerosVisitados.add(tabl);
imprimirTodo(tablerosVisitados,"3");
movimientoHecho=false;
}
//coordenadas que están por debajo de la idBola
casi = casillaBuscada(dimension[0],y,tabl); //nos colocamos a la abajo del todo
casi2 =casillaBuscada(casi.coordX-1,casi.coordY,tabl);
while(casi2.coordY == y && casi2.coordX > x){
if(casi2.idBola!=0 && casi.idBola == 0){ //Si hay bola no se podrá mover a esa posicion
hayMovimiento=true;
}else{
hayMovimiento=false; //si no hay bola, es que hay hueco y se puede mover a esa posicion
}
if(hayMovimiento && casi.getIdBola()==0 && casi2.getIdBola() != idBola){ //si hay movimiento y en la posterior hay bola, se traslada a ocupar ese hueco
casi.setIdBola(casi2.getIdBola());
casi2.setIdBola(0); //Desplazamiento a la derecha cambiado el idBola
movimientoHecho=true;
}
casi = casi2;//pasamos a la siguiente casilla
casi2 = casillaBuscada(casi.coordX-1,casi.coordY,tabl);
}
//imprimirTab1(tabl,"4");
if(movimientoHecho){
cont++;
**tablerosVisitados.add(cont, tabl);**
//This is the line of horror. Delete the first one and set all table like that
//tablerosVisitados.add(tabl);
imprimirTodo(tablerosVisitados,"4");
movimientoHecho=false;
}
suprTablVisitDuplicados(tablerosVisitados);
imprimirTabVisitados(tablerosVisitados);
return tabl;
}
private static void imprimirTodo(ArrayList<ArrayList<Casilla>> tablerosVisitados2, String m) {
for(int i = 0; i<tablerosVisitados2.size();i++){
imprimirTab1(tablerosVisitados.get(i), m);
}
}
private static void suprTablVisitDuplicados(ArrayList<ArrayList<Casilla>> tabVis) {
ArrayList<Casilla> tabTemp;
for(int i=0; i<tabVis.size(); i++){
tabTemp = tablerosVisitados.get(i);
if(esTableroVisitado(tabTemp, tablerosVisitados)){
tablerosVisitados.remove(i);
}
}
}
private static Casilla casillaBuscada(int cX, int cY, ArrayList<Casilla> tab) {
Casilla buscada = null;
boolean encontrada = false;
Iterator<Casilla> it = tab.iterator();
while(it.hasNext() && !encontrada){
buscada = it.next();
if(buscada.coordY==cY && buscada.coordX==cX){
encontrada=true;
}
}
return buscada;
}
private static boolean esTableroSolucion(ArrayList<Casilla> tab1, ArrayList<ArrayList<Casilla>> tabSol){
Iterator<ArrayList<Casilla>> it0 = tabSol.iterator();
Iterator<Casilla> it1 = tab1.iterator();
ArrayList<Casilla> tsol;
boolean esIgual = false;
while(it0.hasNext() && !esIgual){ //bucle que recorre el ArrayList de tableros visitados
tsol = it0.next();
Iterator<Casilla> it2 = tsol.iterator(); //iterator de una de los tableros visitados
while(it1.hasNext() && it2.hasNext() && !esIgual){ //bucle que recorrera los tableros a comparar
if(it1.next().equals(it2.next())){
esIgual=true;
}else{
esIgual=false;
}
}
}
return esIgual;
}
private static boolean esTableroVisitado(ArrayList<Casilla> tab1, ArrayList<ArrayList<Casilla>> tabVisitadas){
//Partimos de la suposición de que los 2 tableros son iguales y en caso de encontrar una diferencia break
Iterator<ArrayList<Casilla>> it0 = tabVisitadas.iterator();
Iterator<Casilla> it1 = tab1.iterator();
ArrayList<Casilla> tv;
boolean esIgual = true;
imprimirTab1(tab1, " original");
while(it0.hasNext() && esIgual){ //bucle que recorre el ArrayList de tableros visitados
tv = it0.next();
Iterator<Casilla> it2 = tv.iterator(); //iterator de una de los tableros visitados
while(it1.hasNext() && it2.hasNext() && esIgual){ //bucle que recorrera los tableros a comparar
imprimirTab1(tv, " comparado.");
if(it1.next() == it2.next()){
esIgual=true;
}else{
esIgual=false;
break;
}
}
}
return esIgual;
}
private static boolean estaContenido(int[] posicion, ArrayList<int[]> coords) {
boolean r = false;
int[] option;
for(int i=0; i<coords.size();i++){
option=new int[2];
option = coords.get(i);
if(option[0] == posicion[0]){
if(option[1]==posicion[1]){
r=true;
break;
}else{
r=false;
}
}else{
r=false;
}
}
return r;
}
}
The problem is result.
When my ArrayList tabVisited.add(newTable) is using references, so it is not copying another different, but the same. And like, all are connected between references, all changes.
The solution is here: https://energiaenescena.wordpress.com/2013/03/11/copia-profunda-de-objetos-en-java-por-serializacion/