[C++] Matrice con diagonale in ordine crescente

[Redix123]

Salve, oggi stavo provando a fare un programma in c++ che mi dicesse se una matrice letta da input ha la diagonale principale in ordine crescente oppure no.

void diagonale_ord(int x[][3], int n)
{
    int verif = 0;
    for (int i = 0; i < n; i++)
    for (int j = 0; j < n; j++)
    {
        if (i == j)
        {
            if (x[i][j] > x[i + 1][j + 1])
                verif = 1;
        }
    }
    if (verif == 0)
        cout << "Diagonale ordinata" << endl;
    else
        cout << "Diagonale non ordinata" << endl;
}

So che non va bene perché mi dice che è ordinata anche se solo gli ultimi due elementi sulla diagonale sono in ordine mentre gli altri non lo sono.
Però non capisco come evitare che vengano valutati solo gli ultimi due elementi

[minomic]

Ciao,
per prima cosa non servono due cicli for, ma ne basta uno solo:

for(int i=0; i<n-1; i++)
{
    /* l'elemento che ti interessa è x[i][i] */
}

Poi per la verifica puoi fare così:

int diagonale_ord(...)
{
    for(...)
    {
        if(ordinamento_sbagliato)
            return 1;
    }
    return 0;
}

In questo modo esci dalla funzione non appena trovi due elementi che non sono in ordine. Se invece tutti gli elementi sono in ordine allora il ciclo for termina senza mai entrare nell'if e la funzione restituisce 0.

[Redix123]

grazie ora mi funziona alla grande

[minomic]

Prego!

[oregon]

A parte il fatto del singolo ciclo for, perché dici che non funziona? Con quali valori hai provato? Con quale main hai chiamato la funzione?

[Redix123]

Beh in realtà il problema era che nei due cicli for mettevo i<n e j<n e non fino a n-1, poichè arrivando ad n confronterebbe anche l'ultimo valore con quello successivo che in realtà non c'è

[oregon]

Veramente hai modificato poco fa il codice nel tuo primo post, infatti è indicato

(Ultima modifica di Redix123; Oggi a 16:15)

con <n perché quello che avevi postato e che avevo provato era

for (int i = 0; i < n - 1; i++)

ed era corretto ... Non capisco anche se importa poco ...

[Redix123]

L'ho modificato per sottolineare il problema ma comunque non sapevo di averlo già risolto mettendo n-1 al posto di n, poiché mi è stato detto che il problema era un altro come infatti ho detto all'inizio:

So che non va bene perché mi dice che è ordinata anche se solo gli ultimi due elementi sulla diagonale sono in ordine mentre gli altri non lo sono.
Però non capisco come evitare che vengano valutati solo gli ultimi due elementi