[C]Le basi

[wingman87]

Ok, a logica ci stai, forse qualche indice di fine e di inizializzazione va corretto ma inizia a scrivere il codice completo e a testarlo, poi se ci sono problemi che non riesci a risolvere posta ancora.

[kwb]

Bene, ti ringrazio dell'aiuto, sono riuscito a far funzionare l'ordinamento, questo è il codice completo del programma:

#include <stdio.h>
#define N 10

main()
{
int marks[N];
int mark = 0, i = 0, mean = 0, sum = 0, max = 0, min, tmp = 0, end = N;

/* Insert marks*/
printf("Insert 10 marks\n");

for ( i=0; i < N; i++)
{
    scanf("%d", &mark);
    marks[i] = mark;
}
/* Order marks section */
while ( end > 1)
{
    for ( i=0; i < N; i++)
    {
        if ( marks[i] > marks[i+1] )
        {
            tmp = marks[i];
            marks[i] = marks[i+1];
            marks[i+1] = tmp;
        }
    }
end--;
}

/* Compute sum and mean then print them */
for ( i=0; i < N; i++)
{
    sum = sum + marks[i];

}
mean = sum / N;

/* Compute maximum and minimum */
for ( i = 0; i < N; i++)
{
    if ( marks[i] > max )
    {
        max = marks[i];
    }
    if ( marks[i] < min )
    {
        min = marks[i];
    }
}
printf("Sum: %d\n", sum);
printf("Mean: %d\n", mean);
printf("Max: %d\n", max);
printf("Min: %d\n", min);
printf("Order: ");
for ( i = 0; i < N; i++)
{
    printf("%d ", marks[i]);
}
}

Se avete dei suggerimenti dal punto di vista della codifica per migliorare la stesura del codice sono, ovviamente, ben accetti!

[wingman87]

Puoi ottimizzarlo così:

#include <stdio.h>
#define N 10

int main() {
    int marks[N];
    int mark = 0, i = 0, mean = 0, sum = 0, max = 0, min, tmp = 0, end = N-1;

    /* Insert marks*/
    printf("Insert 10 marks\n");

    for ( i=0; i < N; i++) {
        scanf("%d", &mark);
        marks[i] = mark;
    }
    /* Order marks section */
    while ( end > 0) {
        for ( i=0; i < end; i++) {
            if ( marks[i] > marks[i+1] ) {
                tmp = marks[i];
                marks[i] = marks[i+1];
                marks[i+1] = tmp;
            }
        }
        end--;
    }

    /* Compute sum and mean then print them */
    for ( i=0; i < N; i++) {
        sum = sum + marks[i];

    }
    mean = sum / N;

    /* Compute maximum and minimum */
    for ( i = 0; i < N; i++) {
        if ( marks[i] > max ) {
            max = marks[i];
        }
        if ( marks[i] < min ) {
            min = marks[i];
        }
    }
    printf("Sum: %d\n", sum);
    printf("Mean: %d\n", mean);
    printf("Max: %d\n", max);
    printf("Min: %d\n", min);
    printf("Order: ");
    for ( i = 0; i < N; i++) {
        printf("%d ", marks[i]);
    }
}

Inoltre potresti aggiungere un controllo che se non vengono effettuati scambi dopo un intero ciclo for puoi terminare l'ordinamento perché significa che è già ordinato.

[kwb]

Originariamente inviato da wingman87

Puoi ottimizzarlo così:

#include <stdio.h>
#define N 10

int main() {
    int marks[N];
    int mark = 0, i = 0, mean = 0, sum = 0, max = 0, min, tmp = 0, end = N-1;

    /* Insert marks*/
    printf("Insert 10 marks\n");

    for ( i=0; i < N; i++) {
        scanf("%d", &mark);
        marks[i] = mark;
    }
    /* Order marks section */
    while ( end > 0) {
        for ( i=0; i < end; i++) {
            if ( marks[i] > marks[i+1] ) {
                tmp = marks[i];
                marks[i] = marks[i+1];
                marks[i+1] = tmp;
            }
        }
        end--;
    }

    /* Compute sum and mean then print them */
    for ( i=0; i < N; i++) {
        sum = sum + marks[i];

    }
    mean = sum / N;

    /* Compute maximum and minimum */
    for ( i = 0; i < N; i++) {
        if ( marks[i] > max ) {
            max = marks[i];
        }
        if ( marks[i] < min ) {
            min = marks[i];
        }
    }
    printf("Sum: %d\n", sum);
    printf("Mean: %d\n", mean);
    printf("Max: %d\n", max);
    printf("Min: %d\n", min);
    printf("Order: ");
    for ( i = 0; i < N; i++) {
        printf("%d ", marks[i]);
    }
}

Inoltre potresti aggiungere un controllo che se non vengono effettuati scambi dopo un intero ciclo for puoi terminare l'ordinamento perché significa che è già ordinato.

Ok grazie. Per la cosa che se l'intero ciclo non riportava cambiamenti avevo intenzione di metterlo. Si tratta della storia del flag vero?

[wingman87]

Originariamente inviato da kwb

Ok grazie. Per la cosa che se l'intero ciclo non riportava cambiamenti avevo intenzione di metterlo. Si tratta della storia del flag vero?

Sì

[kwb]

Penso di essere incappato in un problema che va oltre le mie conoscenze: le sequenze di fibonacci coi vettori.
Ho pensato che sarebbe stato bello poter fare in modo che il programma chiedesse in input la dimensione del vettore, ovviamente so che coi normali vettori ( o con le normali tecniche che a me sono state spiegate ) non è possibile assegnare una dimensione 'x' ad un vettore una volta che il programma è in esecuzione. Mi sono allora imbattuto in pointer, array definiti in modo strano ( con un * ) e robe simili.
La cosa credo sia abbastanza complicata viste le mie conoscenze, meglio lasciar stare?