Piccolo (?) problema con C

[innet]

#include <stdio.h>

void grandi(int a[], int b, int *menomeno, int *piupiu);

#define n 10

/************************
* FUNZIONE MAIN *
************************/

int main()
{
int i, v[n], piu, meno;

printf ("Inserisci %d numeri naturali: ",n);
for ( i = 0 ; i < n ; i++ )
{scanf("%d", &v[i]);}

grandi (v, n, &meno, &piu);

printf ("Il valore più grande è: %d \n",piu);
printf ("Il secondo valore più grande è: %d\n",meno);

return (0);
}

/************************************************
* FUNZIONE PER CALCOLARE I DUE NUMERI MAGGIORI *
************************************************/


void grandi (int a[], int b, int *menomeno, int *piupiu)
{
int i;

*piupiu = *menomeno = a[0];

for ( i = 1 ; i < b ; i++ )
{
if ( a[i] > *piupiu )
*piupiu = a[i];
}

for ( i = 0 ; i < b ; i++ )
{
if (a[i] < *piupiu && a[i] > *menomeno || a[i] != *piupiu)
*menomeno = a[i];
}
}

Due semplici funzioni per stampare i due numeri più grandi di una serie di 10 numeri...
L'errore è presente nel for..
Il prototipo della funzione deve starci.
Riuscite a trovarmi la soluzione?

[KoroshiyaIchi]

La soluzione è dentro di te, e però è sbagliata.

[Amdir]

void grandi (int a[], int b, int *m, int *p)
{
  int i;

  *p = *m = a[0];

  for ( i = 1 ; i < b ; ++i )
  {
    if ( a[i] > *p )
    {
      *m = *p;
      *p = a[i];
    }
    else if ( a[i] > *m )
    {
      *m = a[i];
    }
  }
}

Dovrebbe essere cosi' ma non l'ho provato, l'ho scritto direttamente sul forum

Amdir

[thedeerhunter]

for ( i = 0 ; i < b ; i++ )
{
if (a[i] < *piupiu && a[i] > *menomeno || a[i] != *piupiu)
*menomeno = a[i];
}
}

A che serve la condizione logica or quando hai già scritto a[i]<*piupiu, inoltre il for dovrebbe partire da 1 non da 0, la prima locazione del vettore è già puntata da menomeno, il primo ciclo è a vuoto.

void grandi (int a[], int b, int *m, int *p)
{
  int i;

  *p = *m = a[0];

  for ( i = 1 ; i < b ; ++i )
  {
    if ( a[i] > *p )
    {
      *m = *p;
      *p = a[i];
    }
    else if ( a[i] > *m )
    {
      *m = a[i];
    }
  }
}

Dovrebbe essere cosi' ma non l'ho provato, l'ho scritto direttamente sul forum
Amdir

con questo calcoli solo il numero più grande, manca il secondo.

[ascotfmh]

amduz basta mettere && al posto di ||

edito: intendevo dire che il for basta mettere
for ( i = 0 ; i < b ; i++ )
{
if (a[i] < *piupiu && a[i] > *menomeno && a[i] != *piupiu)
*menomeno = a[i];
}
}

[thedeerhunter]

ma quella condizione finale è gia verificata da a[i]<*piupiu, anche se hai un vettore con numeri ridondanti, tipo [1,2,4,4] non viene verificata la prima condizione del primo AND sul terzo elemento.

[ascotfmh]

si hai ragione, a livello logico non cambia niente, se è vera una lo è anche l'altra

[Amdir]

con questo calcoli solo il numero più grande, manca il secondo.

Ma hai letto il codice?

L'hai provato?

Secondo me funziona

Amdir

[thedeerhunter]

Ho letto male l'else if

[innet]

void grandi (int a[], int b, int *m, int *p)
{
  int i;

  *p = *m = a[0];

  for ( i = 1 ; i < b ; ++i )
  {
    if ( a[i] > *p )
    {
      *m = *p;
      *p = a[i];
    }
    else if ( a[i] > *m )
    {
      *m = a[i];
    }
  }
}

Dovrebbe essere cosi' ma non l'ho provato, l'ho scritto direttamente sul forum

Amdir

no la cosa non funziona, su una serie di 10 1 2 3 4 5 6 7 8 9, essendo a[0] il numero maggiore, fa "fallire" tutte le if, lasciando quindi tutto come == a[0]
maggiore 10
secondo maggiore 10

[innet]

Ho risolto la cosa in questa maniera:

/*******************************
* TROVA I DUE NUMERI MAGGIORI *
*******************************/

#include <stdio.h>

void grandi(int a[], int b, int *menomeno, int *piupiu);

#define n 10

/*****************
* FUNZIONE MAIN *
*****************/

int main()
{
int i, v[11], piu, meno;

v[0]=0;

printf ("Inserisci %d numeri naturali: ",n);
for ( i = 1 ; i < 11 ; i++ )
{scanf("%d", &v[i]);}

grandi (v, n, &meno, &piu);

printf ("Il valore più grande è: %d \n",piu);
printf ("Il secondo valore più grande è: %d\n",meno);

return (0);
}

/************************************************
* FUNZIONE PER CALCOLARE I DUE NUMERI MAGGIORI *
************************************************/


void grandi (int a[], int b, int *menomeno, int *piupiu)
{
int i;

*piupiu = *menomeno = a[0];

for ( i = 1 ; i < b ; i++ )
{
if ( a[i] > *piupiu )
*piupiu = a[i];
}
cacca:
for ( i = 1 ; i <= b ; i++ )
{
if (a[i] < *piupiu && a[i] >= *menomeno)
*menomeno = a[i];
}
}

L'unico problema è che su una lista 1 1 1 1 1 1 1 1 1 1 mi da
maggiore 1
secondo maggiore 0

ps: forse ho trovato la soluzione ma per ora ho sonno e vado a dormire :V

[thedeerhunter]

void grandi (int a[], int b, int *m, int *p)
{
int i;

*p = *m = a[0];

for ( i = 1 ; i < b ; ++i )
{
if ( a[i] > *p )
{
*m = *p;
*p = a[i];
}
else if ( a[i] > *m || *m=*p )
{
*m = a[i];
}
}
}

prova cosi, anche se non penso funzioni

[thedeerhunter]

Ho provato e mi sembra che funzioni su tutti i tipi di vettore.

[innet]

il ragionamento ci sta, però a me da:
maggiora.c: In function ‘grandi’:
maggiora.c:40: error: lvalue required as left operand of assignment

[thedeerhunter]

Assumendo che la riga 40 sia l'else if, non ricordo bene come si gestiscono i valori dei puntatori, nel senso che forse la scrittura della condizione *p=*m è sbagliata