0

Me gustaría mostrar los datos del array. He creado una clase Nota que reciben dos int y un string y la utilizo para construir un arreglo bidimensional en el que inserto una nota en cada celda.

Me gustaría imprimir el array pero no sé cómo hacerlo.

Por otra parte, como es un array de objetos y cada uno tiene 3 datos, ¿de que manera puedo acceder a cada uno de los datos que alberga cada objeto?

Clase Nota:

package ejemplo;

public class Nota{
    private String seller;
    private int numero_producto;
    private int valor_producto;


    public Nota (String s, int p,  int v) {
        seller=s;
        numero_producto=p;
        valor_producto=v;
    }

    public String toString(){
        return "Vendedor" + seller+ "\t"+numero_producto +"\t"+ valor_producto;
    }
}

Clase Dia:

package ejemplo;

import java.util.Random;

public class Dia{
    private Nota[][]dia;
    private static final Random random = new Random(); 

    public Dia(){
        String[]vendedor = {"Juan", "Mike", "Pepe", "Nino"};
        int[]producto = {99, 119, 89, 39, 79};

        dia = new Nota[5][4];
        for(int i=0; i<dia.length; i++) {
            for(int j=0; j<dia[i].length; j++) {
                dia[i][j] = new Nota(vendedor[j%4], producto[i%5], producto[i%5]*random.nextInt(5));                                                          
            }
        }
    }

    public void getDatos(Dia[][]array) {
        for(int i=0; i<array.length; i++) {
            for(int j=0; j<array[i].length; j++) { 
                System.out.println(array[i][j]);
            }
        }
    }
 }

Clase Ventas:

package ejemplo;
import ejemplo.Dia;

import java.util.Random;

public class Ventas{
    public static void main(String[] args) {
        Dia data = new Dia();
        data.getDatos();
    } 
}
6
  • Bienvenido a esta comunidad. Lo primero te recomiendo que mires otras preguntas para saber cómo agregar el código y poner una explicación clara del problema con separaciones, comas, puntos, etc. El enunciado está todo alborotado, y también, es aconsejabe que pongas sangrías en el código para que sea mas legible. Gracias.
    – omaza1990
    Commented el 6 may. 2020 a las 12:55
  • @omaza1990 gracias. Así lo haré. A ver si aprendo a hacer sangrías aquí, que el tabulador en el editor me salta a otro campo de la página
    – Javi
    Commented el 6 may. 2020 a las 13:01
  • Con que tengas bien tabulado tu código en tu IDE es suficiente. Añade el código aquí haciendo un copy-paste y dando al símbolo {} y automáticamente te acepta la sangría.
    – omaza1990
    Commented el 6 may. 2020 a las 13:02
  • Mira la edición que te ofrecí del enunciado y mira qué diferencia y organización respecto a la inicial. ;)
    – omaza1990
    Commented el 6 may. 2020 a las 13:20
  • @omaza1990 ciertamente la diferencia es más que notable y teniendo en cuenta que lo que se escribe es código, entiendo perfectamente que sea más que aconsejable escribir de forma clara y concisa. Me proponía eliminar la pregunta formulada e intentarlo de nuevo rehaciéndola
    – Javi
    Commented el 6 may. 2020 a las 13:30

2 respuestas 2

2

Hay dos problemas en tu código:

1. En ningún momento le pasas un argumento al parámetro del método getDatos.

2. El tipo de parámetro que le has puesto al método getDatos no es el correcto:

public void getDatos(Dia[][]array)

El tipo Dia[][] indica que el parámetro debe recibir una matriz de objetos de tipo Dia, sin embargo, hacer esto no tiene mucho sentido, porque el objetivo de tu código es imprimir los datos de la matriz Nota[][].

Entonces debemos cambiar el encabezado del método a:

public void getDatos(Nota[][] array)

Ahora necesitamos crear un getter (en realidad es un método) para poder acceder al contenido de la variable dia (porque es privada).

El getter debe estar definido en la clase Dia de esta forma:

public Nota[][] getNotas()
{
    return dia;
}

Por último, después de instanciar la clase Dia, debes invocar el getter y pasar el objeto al parámetro del método getDatos, de este modo, podremos imprimir los datos de la matriz.

public class Ventas{
    public static void main(String[] args) {
        Dia data = new Dia();
        data.getDatos(data.getNotas());
    } 
}

Básicamente esto es lo que intentaste hacer, solo que faltó pasar el argumento correcto.

Otra solución es quitar el parámetro que tenga el método getDatos y esto se debe porque la clase Dia declara la variable dia como global, por lo tanto, la podemos usar en cualquier método y de este modo evitamos invocar a getNotas().

public void getDatos() 
{
    for(int i=0; i<dia.length; i++)
    {
         for(int j=0; j<dia[0].length; j++) 
             System.out.println(dia[i][j]);
    }
}

Este código simplemente invocará implícitamente al método toString de la clase Nota e imprimirá la información del objeto.

¿de que manera puedo acceder a cada uno de los datos que alberga cada objeto?

Esto da acceso a la dirección base (la del primer elemento) de X objeto (de tipo Nota):

System.out.println(array[i][j]);

Donde el código de arriba el compilador lo convierte a:

System.out.println(array[i][j].toString());

Entonces si quieres acceder al contenido de cualquier atributo de la clase Nota, debes crear algún getter que lo permita y de este modo, podrás acceder a su contenido.

Ejemplo:

//El método getValorProducto debe estar definido en la clase Nota.
System.out.println(array[i][j].getValorProducto());
2
  • 1
    Por favor, rectifica tu respuesta: No puedes responder con "data.getDatos(data.getNotas());" no tiene sentido, data ya tiene en su clase notas, no necesitas pasarselo con getter. Mira mi respuesta y si la quieres explicar más en la tuya adelante pero contesta intentando hacer buenas prácticas
    – Gonzalo-GC
    Commented el 6 may. 2020 a las 16:30
  • @Gonzalo-GC En mi respuesta trate de explicar cual era el problema con el código que estaba intentando hacer el OP. En fin, editaré la respuesta añadiendo otra solución.
    – MrDave1999
    Commented el 6 may. 2020 a las 17:13
1

El problema es que estabas imprimiendo las instancias del objeto Nota, a parte de que no le tendrías que pasar parámetros a la función ya que tienes la lista en los atributos del objeto, prueba así:

public void getDatos() {
        for(int i=0; i<this.dia.length; i++) {
            for(int j=0; j<this.dia[i].length; j++) { 
                System.out.println(this.dia[i][j].toString());
            }
        }
}
3
  • Esto funciona jeje. Mil gracias. Voy a estudiar como y porqué. Mil gracias
    – Javi
    Commented el 6 may. 2020 a las 13:35
  • @Javi El motivo del porque no te funciona tu código es porque no le pasaste el argumento adecuado al método getDatos.
    – MrDave1999
    Commented el 6 may. 2020 a las 14:32
  • 1
    @MrDave1999 gracias, estoy en ello. Aún ando encajando piezas con la respuesta que me ha dado Gonzalo-GC, que funciona e intento terminar el ejercicio. No obstante, en cuanto tome cuerpo, voy a implementar lo que has apuntado, que quiero combinar con un enum todo esto. Mil Gracias. Es increíble lo que ando avanzando ahora gracias a vuestras respuestas respuestas.
    – Javi
    Commented el 6 may. 2020 a las 14:43

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.