PROBLEMA Esercizio di programmazione con Array

[slyhardrock]

Salve a tutti, ho da poco iniziato a programmare e non riesco a fare correttamente questo esercizio. Non riesco ad ottenere il risultato desiderato. Ecco l'esercizio:

Scrivere un programma che acquisisca da tastiera un intero
x
e un array
di 5 altri interi e restituisca il numeri di elementi maggiori di
x
presenti
nell’array.Il programma deve usare una funzione
count
larger
che prende
come parametri
x
, l’array e la lunghezza dell’array e restituisce il numero
di elementi maggiori di
x
.

Esempio input esempio output:
5 2
0
3
65
52
1


Ecco il mio codice
https://imgur.com/a/5TYbU

Per favore non mi insultate, sono alle primissime armi. Grazie in anticipo.

 

[Roberto_Trevigiani]

Ciao, prima di parlare dell'esercizio, vorrei dirti di utilizzare il tag code per mostrare il codice e non le immagini, e di dare più dettagli sull'errore che ti da il programma, così sarà più facile darti un aiuto. Ad una prima occhiata nel main manca l'istruzione "return 0;"

 

[qwerty92]

Ciao,
la soluzione è "banale"... quando passi la variabile "c" non devi dargli il numero 5 ma lo zero o il numero dal quale vuoi partire... Se scrivi &c[5] vuol dire che vuoi passare alla variabile c[5], c[6], ecc, che ovviamente sono numeri random per lui. (se vuoi verificarlo aggiungi printf("%d\n", arr[0]) dopo l'inizializzazione di "j" e vedrai che non sono assolutamente i numeri da te passati.
In alternativa se vuoi passare tutto l'array puoi semplicemente scrivere solo "c".

#include <stdio.h>

void count (int x, int *arr)
{
int trovato=0;

for (int j=0; j<5; j++)
{
if (arr[j] > x )
trovato++;
}

printf("%d\n", trovato);

return;
}

int main()
{
    int a=0;
    int c[5];
    int m=0;
    scanf("%d", &a);

    for (int i=0; i<5; i++)
    {
    scanf("%d", &m);
    c[i]=m;
    }

    count (a, &c[0]);  //oppure count (a, c)


    return 0;
}

qualche piccoli appunto:
1 - Inizializza sempre una variabile quando la dichiari.
2 - Come dice anche Roberto verifica che ci sia sempre un codice di uscita dalla funzione (return).
3 - Se la variabile ti serve solo come contatore del ciclo puoi inzializzarla all'interno delle parentesi.
4 - Se vuoi farlo ancora più "bello" usa "int *arr" nella funzione ricevente. L'asterisco sta ad indicare che la funzione riceverà un riferimento e non un solo valore.

Il codice comunque dovrebbe essere a posto, non l'ho testato completamente. Facci sapere.
Buona serata!

 

[slyhardrock]

Ciao,
la soluzione è "banale"... quando passi la variabile "c" non devi dargli il numero 5 ma lo zero o il numero dal quale vuoi partire... Se scrivi &c[5] vuol dire che vuoi passare alla variabile c[5], c[6], ecc, che ovviamente sono numeri random per lui. (se vuoi verificarlo aggiungi printf("%d\n", arr[0]) dopo l'inizializzazione di "j" e vedrai che non sono assolutamente i numeri da te passati.
In alternativa se vuoi passare tutto l'array puoi semplicemente scrivere solo "c".

#include <stdio.h>

void count (int x, int *arr)
{
int trovato=0;

for (int j=0; j<5; j++)
{
if (arr[j] > x )
trovato++;
}

printf("%d\n", trovato);

return;
}

int main()
{
    int a=0;
    int c[5];
    int m=0;
    scanf("%d", &a);

    for (int i=0; i<5; i++)
    {
    scanf("%d", &m);
    c[i]=m;
    }

    count (a, &c[0]);  //oppure count (a, c)


    return 0;
}

qualche piccoli appunto:
1 - Inizializza sempre una variabile quando la dichiari.
2 - Come dice anche Roberto verifica che ci sia sempre un codice di uscita dalla funzione (return).
3 - Se la variabile ti serve solo come contatore del ciclo puoi inzializzarla all'interno delle parentesi.
4 - Se vuoi farlo ancora più "bello" usa "int *arr" nella funzione ricevente. L'asterisco sta ad indicare che la funzione riceverà un riferimento e non un solo valore.

Il codice comunque dovrebbe essere a posto, non l'ho testato completamente. Facci sapere.
Buona serata!

Ciao,
la soluzione è "banale"... quando passi la variabile "c" non devi dargli il numero 5 ma lo zero o il numero dal quale vuoi partire... Se scrivi &c[5] vuol dire che vuoi passare alla variabile c[5], c[6], ecc, che ovviamente sono numeri random per lui. (se vuoi verificarlo aggiungi printf("%d\n", arr[0]) dopo l'inizializzazione di "j" e vedrai che non sono assolutamente i numeri da te passati.
In alternativa se vuoi passare tutto l'array puoi semplicemente scrivere solo "c".

#include <stdio.h>

void count (int x, int *arr)
{
int trovato=0;

for (int j=0; j<5; j++)
{
if (arr[j] > x )
trovato++;
}

printf("%d\n", trovato);

return;
}

int main()
{
    int a=0;
    int c[5];
    int m=0;
    scanf("%d", &a);

    for (int i=0; i<5; i++)
    {
    scanf("%d", &m);
    c[i]=m;
    }

    count (a, &c[0]);  //oppure count (a, c)


    return 0;
}

qualche piccoli appunto:
1 - Inizializza sempre una variabile quando la dichiari.
2 - Come dice anche Roberto verifica che ci sia sempre un codice di uscita dalla funzione (return).
3 - Se la variabile ti serve solo come contatore del ciclo puoi inzializzarla all'interno delle parentesi.
4 - Se vuoi farlo ancora più "bello" usa "int *arr" nella funzione ricevente. L'asterisco sta ad indicare che la funzione riceverà un riferimento e non un solo valore.

Il codice comunque dovrebbe essere a posto, non l'ho testato completamente. Facci sapere.
Buona serata!

Grazie mille per la risposta, appena torno a casa lo testo! :)

 

[slyhardrock]

Ciao,
la soluzione è "banale"... quando passi la variabile "c" non devi dargli il numero 5 ma lo zero o il numero dal quale vuoi partire... Se scrivi &c[5] vuol dire che vuoi passare alla variabile c[5], c[6], ecc, che ovviamente sono numeri random per lui. (se vuoi verificarlo aggiungi printf("%d\n", arr[0]) dopo l'inizializzazione di "j" e vedrai che non sono assolutamente i numeri da te passati.
In alternativa se vuoi passare tutto l'array puoi semplicemente scrivere solo "c".

#include <stdio.h>

void count (int x, int *arr)
{
int trovato=0;

for (int j=0; j<5; j++)
{
if (arr[j] > x )
trovato++;
}

printf("%d\n", trovato);

return;
}

int main()
{
    int a=0;
    int c[5];
    int m=0;
    scanf("%d", &a);

    for (int i=0; i<5; i++)
    {
    scanf("%d", &m);
    c[i]=m;
    }

    count (a, &c[0]);  //oppure count (a, c)


    return 0;
}

qualche piccoli appunto:
1 - Inizializza sempre una variabile quando la dichiari.
2 - Come dice anche Roberto verifica che ci sia sempre un codice di uscita dalla funzione (return).
3 - Se la variabile ti serve solo come contatore del ciclo puoi inzializzarla all'interno delle parentesi.
4 - Se vuoi farlo ancora più "bello" usa "int *arr" nella funzione ricevente. L'asterisco sta ad indicare che la funzione riceverà un riferimento e non un solo valore.

Il codice comunque dovrebbe essere a posto, non l'ho testato completamente. Facci sapere.
Buona serata!

Funziona! Grazie mille ancora!