multipli

[pasco78]

Questo programma dovrebbe determinare e visualizzare dopo aver letto 2 numeri, se il primo sia un multiplo del secondo o viceversa. Immettendo 2 numeri di cui il primo multiplo del secondo o viceversa il programma funziona es. (2,4) o (4,2) .Immettendo (5,3) funziona ma se inserisco (3,5) non si fa niente...Dove sbaglio??







#include<stdio.h>
int main()
{
int x,y;
scanf("%d%d",&x,&y);
if(x >= y)
{
if(x%y==0)
{
printf("%d e' un multiplo di %d",x,y);
}
if (x%y!=0)
{
printf("%d non e' un multiplo di %d",x,y);
}
}
if(x<y)
{
if(y%x==0);
{
printf("%d e' un multiplo di %d",y,x);
}
if (y%x!=0)
{
printf("%d non e' un multiplo di %d",y,x);
}
}
getchar();
getchar();
return 0;
}

[Alex'87]

Originariamente inviato da pasco78
if(x<y)
{
if(y%x==0);
{
printf("%d e' un multiplo di %d",y,x);
}
if (y%x!=0)
{
printf("%d non e' un multiplo di %d",y,x);
}
}

Togli quel ;

Il codice che hai scritto inoltre è piuttosto prolisso: prima controlli se x >= y ed in caso contrario controlli se x < y... Ma non serve, il ramo else non ha altre alternative!

#include <stdio.h>

int main(int argc, char **argv) {
    int x, y;
    
    printf("Inserisci due numeri interi e premi [INVIO]: ");
    scanf("%d%d", &x, &y);
    
    if(x >= y) {
        if(x % y == 0) {
            printf("%d e' un multiplo di %d\n", x, y);
        } else {
            printf("%d non e' un multiplo di %d\n", x, y);
        }
    } else {
        if(y % x == 0) {
            printf("%d e' un multiplo di %d\n", y, x);
        } else {
            printf("%d non e' un multiplo di %d\n", y, x);
        }
    }

    return 0;
}

[ant_alt]

hai un ; dopo if(y%x==0)

#include<stdio.h>
int main()
{
int x,y;
scanf("%d%d",&x,&y);

    if(x%y==0 || y%x==0)
            printf("%d e' un multiplo di %d",x,y);
    else
        printf("%d non e' un multiplo di %d",x,y);
getchar();
getchar();
return 0;
}

inoltre osserva bene come è possibile ridurre il tuo programma, osserva l'utilizzo (non) delle parentesi

[pasco78]

Grazie !!!

[QUOTE]Originariamente inviato da ant_alt
hai un ; dopo if(y%x==0)

#include<stdio.h>
int main()
{
int x,y;
scanf("%d%d",&x,&y);

    if(x%y==0 || y%x==0)
            printf("%d e' un multiplo di %d",x,y);
    else
        printf("%d non e' un multiplo di %d",x,y);
getchar();
getchar();
return 0;
}

inoltre osserva bene come è possibile ridurre il tuo programma, osserva l'utilizzo (non) delle parentesi [/QUOT

Wow! || stanno per o??
Però in questo modo il programma dice che 2 e' multiplo di 8 e non il contrario...

[ant_alt]

Originariamente inviato da pasco78
Wow! || stanno per o??

http://programmazione.html.it/guide/...tori-booleani/

ora però nota che dovresti cambiare il messaggio con uno più generale , se inserisci 2 e 4 nel programma il messaggio è "2 è multiplo di 4"

[pasco78]

non avevo letto il tuo messaggio.. sorry

[ant_alt]

#include<stdio.h>
#define MIN(a,b) ((a<b) ? (a) : (b))
#define MAX(a,b) ((a>b) ? (a) : (b))
int main()
{
int x,y;
scanf("%d%d",&x,&y);

    if(x%y==0 || y%x==0)
            printf("%d e' un multiplo di %d",MAX(x,y),MIN(x,y));
    else
        printf("%d non e' un multiplo di %d",MAX(x,y),MIN(x,y));
getchar();
getchar();
return 0;
}

puoi usare l'escamotage delle macro, un pò macchinoso ma almeno fai vedere al prof che sai fare qualcosa
se non le avete studiate ancora puoi usare le funzioni min e max

[pasco78]

Sto studiano d'autodidatta... sono proprio all'inizio ,non ho fatto nè la ''macro'' ,nè min e max..
Ora me li vado a vedere sul libro.Grazie.

[ant_alt]

Originariamente inviato da pasco78
Sto studiano d'autodidatta... sono proprio all'inizio ,non ho fatto nè la ''macro'' ,nè min e max..

per min e max intendevo che ti definisci due funzioni che restituiscono minimo e massimo..ma vai per gradi, credo sia un pò presto per questo
buon studio

[Alex'87]

Originariamente inviato da ant_alt
inoltre osserva bene come è possibile ridurre il tuo programma, osserva l'utilizzo (non) delle parentesi

Questo si può fare solo se il blocco di codice consiste in un'unica istruzione! Ad ogni modo sarebbe buona programmazione usare sempre le parentesi, in modo da rendere più chiaro il codice...