Problema con funzione di controllo java!

[PasC]

questo è il codice che ho realizzato, funziona fino quando j è diverso da zero, quando lo diventa, se inserisco un valore minore, devo tornare nella posizione precedente e lo confronta appunto con questo, solo che se faccio j-1 da outofbounds.
ho provato anche a mettere la lunghezza delle righe ma nulla!!
Qualcuno per favore può gentilmente aiutarmi??
grazie in anticipo!!

R=numero di righe
C=numero di colonne
m1=matrice
i=contatore delle righe
j=contatore delle colonne

static void inserimento2(int [][] m1, int R, int C){ 
   for(int i=0; i<R; i++){ 
      for(int j=0; j<C; j++){ 
         System.out.println("Inserisci l'elemento [" + (i + 1) +" , "+ (j+1) + "] della matrice-->"); 
         m1[i][j]=scan.nextInt(); 
         if(i==0){ 
            if(j==0){ 
               System.out.print(); 
            } 
            else if(j!=0){ 
               while(m1[i][j]<m1[i][j-1]){ 
                  System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i][j-1]); 
                  m1[i][j]=scan.nextInt(); 
               } 
            } 
         } 
         else if(i!=0){ 
            if(j==0){ 
               while(m1[i][j]<m1[i-1][m1[C].length-1]){ 
                  System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i][m1[0].length]); 
                  m1[i][j]=scan.nextInt(); 
               } 
            } 
            else if(j!=0){ 
               while(m1[i][j]<m1[i][j-1]){ 
               System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i][j-1]); 
               m1[i][j]=scan.nextInt(); 
            } 
        } 
     } 
     System.out.println(""); 
  } 
}

[LeleFT]

l'identazione verrà completamente sbagliata!! scusa ma la mette così!

La mette così perchè tu non hai messo i tag CODE, che servono proprio a formattare correttamente il codice.
In futuro usa una delle due possibilità:

1) Scrivere in "Modalità avanzata" (pulsante in basso a destra) e usare l'apposito pulsante "#", che automaticamente inserisce i tag

2) Scrivere direttamente i tag CODE nel corpo della discussione e incollarci dentro il codice:

[CODE]
incolli il codice
[/CODE]


Ciao.

[PasC]

grazie mille!! la prossima volta farò così! ciao!

[Alex'87]

Cosa dovrebbe fare quel codice?

[LeleFT]

Non ho letto tutto il codice, ma una cosa la devi imparare e, di conseguenza, sistemare:

L'istruzione "if" funziona in questo modo:

if ( condizione ) {
   // Questo ramo viene eseguito se la condizione è vera
} else {
   // Questo ramo viene eseguito se la condizione è falsa
}

Quindi, se la condizione è (i == 0), non ha nessun senso controllare nel ramo "else" se i è diverso da 0. Perchè lo sappiamo già che nel ramo else è diverso da zero, altrimenti non ci sarebbe mai entrato.

Tu hai scritto così:

if (i == 0) {
   ...
} else {
   if (i != 0) {   // Questo if non serve a nulla: se siamo qui è perchè i è diverso da 0.
      ...          // Fa diventare ridondante il codice, rendendolo difficile da seguire e manutenere
   }
}

E la stessa cosa l'hai fatta per le condizioni su "j".

Quindi, correggi quelle if, togliendo dai rami else il controllo della condizione inversa, che non servono a nulla e rendono inutilmente complicato e prolisso il codice.

Poi, riposta il codice con le correzioni che ci ragioniamo su.


Ciao.

[PasC]

Grazie mille lo correggo subito!!

[PasC]

praticamente fa inserire all'utente dei valori in ordine crescente!
se inserisce uno minore lo confronta con quello prima e dice di inserirne uno maggiore!

[PasC]

ecco il codice corretto! ho tolto gli if dagli else come controllo inverso!

static void inserimento2(int [][] m1, int R, int C){  //inserire valori in ordine crescente
      for(int i=0; i<R; i++){ 
         for(int j=0; j<C; j++){
            System.out.println("Inserisci l'elemento [" + (i + 1) +" , "+ (j+1) + "] della matrice-->");
            m1[i][j]=scan.nextInt();
            if(i==0){
               if(j==0){
                  System.out.print("");        //se il valore si trova alla posizione 0,0 non deve essere confrontato
               }
               else{
                  while(m1[i][j]<m1[i][j-1]){    //controlla il valore inserito con quello precedente
                     System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i][j-1]);
                     m1[i][j]=scan.nextInt();
                  }                
               }
            }
            else{
               if(j==0){
                  while(m1[i][j]<m1[i-1][j+C]){  //controlla il valore inserito con quello precedente 
                     System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i-1][m1[0].length]);
                     m1[i][j]=scan.nextInt();  //se inserisce un valore minore, chiede di reinserire un valore però maggiore
                  }
               }
               else{
                  while(m1[i][j]<m1[i][j-1]){      //controlla il valore inserito con quello precedente
                     System.out.println("Per favore inserisci un numero maggiore o uguale a " + m1[i][j-1]);
                     m1[i][j]=scan.nextInt();
                  }   
               }   
            }          
         }
         System.out.println("");         
      }          
   }

[LeleFT]

Un primo errore evidente che ho visto:

if(i==0){
   ...
else{
   if(j==0){
      while(m1[i][j]<m1[i-1][j+C]){  //controlla il valore inserito con quello precedente
      ...
   }
}

j è uguale a 0, quindi scrivere "j + C" è inutile... tanto vale scrivere direttamente "C":

// Questo è equivalente: ho sostituito "j+C" con "C"
while(m1[i][j]<m1[i-1][C]){

Dato che gli array vanno da 0 a C-1, stai sforando i limiti.


Ciao.

[PasC]

Grazie mille per la disponibilità e per l'aiuto!! alla fine è andato!! Ancora grazie di cuore! Ciao