[c] ordine crescente

[spode]

//verificare che gli elementi di a[n] siano in ordine crescente

#include <stdio.h>
#include <stdlib.h>
#define max 100

int main()
{

    int n=0,m=0,appo=0,ciao=0,i=0,a[max];
    char risposta;
    int SonoOrdineCrescente=0;

    do{
    printf("Ciao. Inserisci n: ");
    scanf("%d",&n);
    do{
        i=0;
    printf("Inserisci il NUMERO %d di %d: ",i,n-1);
    scanf("%d",a[i]);
    i++;
    }while(i<n);


    //Verificare che l' elemento n-(n-1) sia maggiore di n-(n-2)
    for(i=1,m=0;i<n,m<n;i++)
    {
        if(a[m]>=a[i])
        {
            SonoOrdineCrescente=1;
        }
        else
        {
            SonoOrdineCrescente=2;
        }
        if(i=n-1)
        {
            m++;
        }
    }

    if(SonoOrdineCrescente=1)
    {
        printf("\nGli elementi sono in ordine crescente!");
    }
    else
    {
        printf("\nGli elementi non sono in ordine crescente");
    }

        printf("Vuoi riprovare?(s\n) ");
        risposta=getch();
    }while(risposta=='s');

    return 0;
}

l'errore che mi dà è:

Ciao. Inserisci n: 2
Inserisci il NUMERO 0 di 1: 34

Process returned -1073741819 (0xC0000005)   execution time : 4.219 s
Press any key to continue.

[Cell]

Probabilmente il problema è qui:

    do{
        i=0;
    printf("Inserisci il NUMERO %d di %d: ",i,n-1);
    scanf("%d",&a[i]);       <----- scanf vuole l'indirizzo della variabile
    i++;
    }while(i<n);

Fammi sapere.

[Alex'87]

Originariamente inviato da Cell

    do{
        i=0;
    printf("Inserisci il NUMERO %d di %d: ",i,n-1);
    scanf("%d",&a[i]);       <----- scanf vuole l'indirizzo della variabile
    i++;
    }while(i<n);

Inoltre questo è un ciclo infinito: l'azzeramente di i va fatto prima del do.

@l'autore del thread
Il do-while non è adatto al tuo scopo: tu sai esattamente quante iterazioni fare (da 0 incluso a n escluso), per questo la buona programmazione vorrebbe che venga utilizzato un ciclo for.

for (i = 0; i < n; i++) {
  printf("Inserisci il NUMERO %d di %d: ", i, n - 1);
  scanf("%d", &a[i]);
}

[Cell]

Originariamente inviato da Alex'87
Inoltre questo è un ciclo infinito: l'azzeramente di i va fatto prima del do.

Sì, infatti. Hai pienamente ragione.

[spode]

grazie. in qualsiasi caso l'errore lo dà sempre,anche con for. è come se non riuscisse a leggere l'ultima variabile. Il nuovo codice (corretto senza vedere le rsp...(faccina: vanitoso ) )

//verificare che gli elementi di a[n] siano in ordine crescente

#include <stdio.h>
#include <stdlib.h>
#define max 100

    //prototipo = dichiarazione funzioni
    void Riprova();
    void NonCrescente();
    //fine prototipo (nota: i prototipi vanno prima di main() perchè il pc legge dall'alto al basso)

    void NonCrescente()
    {
        printf("\nGli elementi non sono in ordine crescente");
        Riprova();
    }

    void Riprova()
    {
        char risposta;
        printf("\nVuoi riprovare?(s/n) ");
        risposta=getche();
        printf("\n");
        if(risposta=='s')
        {
            main();
        }
    }

int main()
{

    int n=0,m=0,ciao=0,i=0,a[max];
    int SonoOrdineCrescente=0;

    printf("Ciao. Inserisci n: ");
    scanf("%d",&n);
    for(i=0;i<n;i++)
    {//ERROR: chissà perchè il while non funziona?
     //ERROR: avevo aggiunto i=0;
    printf("Inserisci il NUMERO %d di %d: ",i,n-1);
    scanf("%d",a[i]);
    }


    //Verificare che l' elemento n-(n-1) sia maggiore di n-(n-2)
    for(i=1,m=0;m<n;i++)
    {
        while(a[m]<a[i])
        {
            SonoOrdineCrescente=1;
        }
        while(a[m]>=a[i])
        {
            NonCrescente();
        }
        if(i=n-1)
        {
            m++;
            i=m++;
        }
        else{}
    }
    printf("\nGli elementi sono in ordine crescente");
    Riprova();
}

[Alex'87]

Originariamente inviato da spode
if(risposta=='s')
{
main();
}

Richiamare il main in questo modo è la cosa più orribile che si possa fare.

Originariamente inviato da spode
scanf("%d",a[i]);

scanf("%d", &a[i]);

Originariamente inviato da spode

        if(i=n-1)
        {
            m++;
            i=m++;
        }

Questo if darà sempre "vero" come risultato e verrà sempre eseguito. Controlla quella condizione...

[spode]

Citazione: Originariamente inviato da spode if(risposta=='s') { main(); } Richiamare il main in questo modo è la cosa più orribile che si possa fare.

ah, sì? e come dovrei scriverlo? cosa c'è di male (me lo hanno già detto, ma voglio sapere come fai tu)?

Citazione: Originariamente inviato da spode codice: if(i=n-1) { m++; i=m++; } Questo if darà sempre "vero" come risultato e verrà sempre eseguito. Controlla quella condizione...

perchè restituisce sempre vero? all'inizio n può essere 3 (n-1=2) e i parte da 1...

[Alex'87]

Originariamente inviato da spode
ah, sì? e come dovrei scriverlo? cosa c'è di male (me lo hanno già detto, ma voglio sapere come fai tu)?

Il main è una funzione particolare in quanto è la prima a venire eseguita all'avvio del programma. Richiamarla è ridondante e ti fa sprecare memoria inutilmente (hai presente come vengono chiamante le funzioni? Lo stack, il fare "push" ecc?[/QUOTE]

Se non vuoi far terminare il programma usa un ciclo all'interno del main.

Originariamente inviato da spode
perchè restituisce sempre vero? all'inizio n può essere 3 (n-1=2) e i parte da 1...

i = n - 1 è un ASSEGNAMENTO, non un confronto. Quello che hai scritto significa "assegna il valore di n - 1 a i". E poiché il C considera "vero" qualsiasi valore diverso da 0 (zero), quella i varrà sempre "VERO" a parte il caso in cui n vale 1. L'uguaglianza è ==.

[spode]

tengo a precisare che la discussione è a fini più "generali", nel senso che forse ho problemi di logica col while, dato che mi è venuta voglia di dare una capocciata al pc dopo 40000 volte che provo a farlo andare con questa consegna.
Per ale: il codice l'ho scritto direttamente sul forum, quindi non me ne sono accorto. scusa e cerca di comprendere che non l'ho fatto per sfiducia.

Citazione: Originariamente inviato da spode ah, sì? e come dovrei scriverlo? cosa c'è di male (me lo hanno già detto, ma voglio sapere come fai tu)? Il main è una funzione particolare in quanto è la prima a venire eseguita all'avvio del programma. Richiamarla è ridondante e ti fa sprecare memoria inutilmente (hai presente come vengono chiamante le funzioni? Lo stack, il fare "push" ecc?

Se non vuoi far terminare il programma usa un ciclo all'interno del main.[/QUOTE] no, non so cosa siano lo stack e il fare il push, ecc. me lo diresti?

Citazione: Originariamente inviato da spode ah, sì? e come dovrei scriverlo? cosa c'è di male (me lo hanno già detto, ma voglio sapere come fai tu)? Il main è una funzione particolare in quanto è la prima a venire eseguita all'avvio del programma. Richiamarla è ridondante e ti fa sprecare memoria inutilmente (hai presente come vengono chiamante le funzioni? Lo stack, il fare "push" ecc?

Se non vuoi far terminare il programma usa un ciclo all'interno del main. Citazione: Originariamente inviato da spode perchè restituisce sempre vero? all'inizio n può essere 3 (n-1=2) e i parte da 1... i = n - 1 è un ASSEGNAMENTO, non un confronto. Quello che hai scritto significa "assegna il valore di n - 1 a i". E poiché il C considera "vero" qualsiasi valore diverso da 0 (zero), quella i varrà sempre "VERO" a parte il caso in cui n vale 1. L'uguaglianza è ==.[/QUOTE]
grazie mille. mi sbaglio sempre!

allora mi potete aiutare a capire il while in italiano e poi in c? da come uso io il while, il valore della variabile non viene modificata (Ordine o SonoOrdineCrescente): devo capire come cambiarla!

per l'amministratore/moderatore:
per favore, metti le discussione iniziate da x (nel mio caso, da spode) per prime nell'elenco. in questo modo non devo per forza impazzire nel cercarle (considera che navigo in EDGE->max 25Kb/s).