Operazioni con matrici in java

[chri.ten.94]

Salve a tutti, avrei una domanda da favi.
Devo leggere tale matrice da un file txt:
1,2,3,6
4,5,6,6
0,0,1,7
Siccome non h specificato il numero di colonne e il numero di righe ho pensato di usare un ArrayList<int[]>. Fin qui tutto bene. I problema lo trovo quando voglio fare delle operazioni con le colonne(ad esempio la somma di una colonna). Sapete dirmi se per caso vi è un metodo per poter fare tali operazioni?
Io ho provato facendo una cosa del genere:
for(int j=0;j<m.mat.get(0).length;j++){
int s=0;
for(int i=0;i<m.mat.size();i++)
s+=m.mat.get(i)[j];
}

[franksisca]

metti tutto il codice, possibilmente su pastebin, che altrimenti non si capisce niente.

Inoltre non dici che errore ti da!!!

[chri.ten.94]

public static void punto1() throws Exception {
PrintWriter out = new PrintWriter(new FileWriter("1.txt"));

int max=0;
int cmx=0;
for(Matrice m:matrice){
for(int j=0;j<m.mat.get(0).length;j++){
int s=0;
for(int i=0;i<m.mat.size();i++)
s+=m.mat.get(i)[j];
if(s>max){
max=s;
cmx=j+1;
}
}
}
out.println("la colonna di somma massima è la "+cmx+"a colonna");

out.close();
}

Non mi da errori, semplicemente non fa l'operazione che voglio. Cioè non mi somma gli elementi che hanno lo stesso indice di colonna( j in questo caso). Non metto l'intero codice perché c'è anche la lettura del file e una Classe Matrice che non servono a molto per capire il problema

[franksisca]

l'oggetto Matrice com'è composto?

m da dove viene fuori? Cosa ti stampa comunque?

E comunque quando qualcuno ti dice di usare pastebin, usa pastebin, altrimenti fai passare la voglia di farti aiutare!!!

https://pastebin.com/7madrbRN

[chri.ten.94]

guarda grazie comunque, vai ad aiutare chi usa pastebin

[LukeIlBello]

Credi sia errato il 2 for(int i=0;i<m.mat.size();i++)

il primo for è quello per cui iteri l'indice su tutte le colonne, e va bene,
questo qui invece lo iteri per mat.size()... invece dovrebbe ciclare per il numero di righe appartenenti alla singola colonna del ciclo for iniziale..

cambia quindi il 2 ciclo di for in
for(int i=0;i<m.mat.get(j).length; i++)

[chri.ten.94]

public class Matrice {
ArrayList<int []> mat;

public Matrice(){
mat=new ArrayList<int[]>();
}

public String toString(){
StringBuffer sb=new StringBuffer();
for(int[] riga:mat){
for(int x:riga)
sb.append(x+"\t");
sb.append("\n");
}
return sb.toString();
}

}

La classe Matrice è composta così, quindi il secondo ciclo for non dovrebbe essere sbagliato poiché gli vado a dire "blocca un'indice di colonna e poi scorri tutti gli elementi dell'ArrayList mat con quell'indce e sommali". Almeno così dovrebbe fare, ma non o fa . Ho provato a sostituire il ciclo for che mi hai consigliato ma mi da errore di lunghezza

[LukeIlBello]

Originariamente inviato da chri.ten.94

La classe Matrice è composta così, quindi il secondo ciclo for non dovrebbe essere sbagliato poiché gli vado a dire "blocca un'indice di colonna e poi scorri tutti gli elementi dell'ArrayList mat con quell'indce e sommali". Almeno così dovrebbe fare, ma non o fa . Ho provato a sostituire il ciclo for che mi hai consigliato ma mi da errore di lunghezza

attenzione che se usi toString() come metodo dovresti mettere un @override sopra, altrimenti dagli un altro nome (tipo getSommaMatrice() )..
per quanto riguarda il ciclo, ora mi pare corretto (stai usando foreach e non piu for indicizzati), anche se non sembra eseguire somme ma fare il semplice append uno stringbuffer..