[C] Chiarimento codice

[sparpazeo]

Salve a tutti, so che vi farò ridere con questa mia discussione, ma ho iniziato 2 giorni fa con la programmazione in C e nonostante studio informatica ho sempre considerato la programmaazione un tabù.

Vengo al problema: sto scrivendo un codice che calcoli l'iva utilizzando una libreria, inizialmente volevo solo calcolare il valore aggiunto di iva e andava bene, ma poi ho pensato di voler calcolare anche l'iva (sempre tramite libreria). Il problema è che adesso in uscita mi da solo il valore dell'iva. Vi posto il codice, spero di essere stato chiaro

float aggiungiIVA (float importo)
{
      float importoIVA,IVA;
      importoIVA = (importo * 0.2) + importo;
      IVA = importo * 0.2;
      return importoIVA, IVA;
}

#include <stdio.h>
#include "aggiungiIVA.h"

float main ()
{
      float conIVA,importo,IVA;
      printf ("Inserisci l'importo al quale vuoi aggiungere l'IVA: \n");
      scanf ("%f",&importo);
      conIVA = aggiungiIVA (importo);
      IVA = aggiungiIVA (importo);
      printf ("Il valore aggiunto di IVA è: %f \n",conIVA);
      printf ("L'IVA è: %f \n",IVA);
}

[andbin]

Per fare in modo che una funzione restituisca al "chiamante" più di un valore ci sono svariate strade:
- Ritornare una struct.
- Passare alla funzione dei puntatori a variabili in cui la funzione scriverà.
- Passare alla funzione il puntatore ad una struct in cui la funzione scriverà.

[sparpazeo]

In parole un po povere?
I puntatori ancora li devo fare...praticamente nn riesco a capire come posso far uscire due risultati dalla libreria che ho creato...

[andbin]

Originariamente inviato da sparpazeo

In parole un po povere?
I puntatori ancora li devo fare...praticamente nn riesco a capire come posso far uscire due risultati dalla libreria che ho creato...

Ok, lasciamo perdere per il momento i puntatori (meglio non confonderti le idee....)

Restituzione di una struttura per valore:

#include <stdio.h>

typedef struct
{
    double importo_iva;
    double iva;
} CALCIVA;

CALCIVA CalcIva (double importo, float perc)
{
    CALCIVA calciva;

    calciva.iva = importo * perc / 100.0;
    calciva.importo_iva = importo + calciva.iva;

    return calciva;
}

int main (void)
{
    double imp = 100.0;
    CALCIVA calciva;

    calciva = CalcIva (imp, 20.0f);

    printf ("Importo IVA = %.2f\n", calciva.importo_iva);
    printf ("IVA = %.2f\n", calciva.iva);

    return 0;
}

[sparpazeo]

Praticamente ho fatto in questo modo e funziona:

float aggiungiIVA (float);
float calcolaIVA (float);

float aggiungiIVA (float importo)
{
      float importoIVA;
      importoIVA = (importo * 0.2) + importo;
      return importoIVA; 
}

float calcolaIVA (float importo)
{
      float IVA;
      IVA = importo * 0.2;
      return IVA;
}

#include <stdio.h>
#include "aggiungiIVA.h"

float main ()
{
      float conIVA,importo,IVA;
      printf ("Inserisci l'importo al quale vuoi aggiungere l'IVA: ");
      scanf ("%f",&importo);
      conIVA = aggiungiIVA (importo);
      IVA = calcolaIVA (importo);
      printf ("Il valore aggiunto di IVA e': %.3f \n",conIVA);
      printf ("L'IVA e': %.3f \n",IVA);
      
}

L'errore mio stava che dentro la libreria creavo una funzione che mandava in uscita due risultati, uno che calcola l'iva e uno che aggiunge l'IVA all'importo... invece si dovevano creare due funzioni che quando chiamate facevano ognuna il proprio lavoro...

[andbin]

Originariamente inviato da sparpazeo

Praticamente ho fatto in questo modo e funziona

Ho quasi paura a chiedertelo : hai messo le due funzioni fisicamente nel file aggiungiIVA.h??? Noooo.
Poi da quando in qua il main ritorna un float?

Originariamente inviato da sparpazeo

invece si dovevano creare due funzioni che quando chiamate facevano ognuna il proprio lavoro...

Beh, è solo una delle tante soluzioni .... a meno che qualcuno ti abbia detto esplicitamente che devi usare funzioni separate.

[sparpazeo]

ahahahahah aspe che sto capendo poco, per quanto riguarda il file aggiungiIVA è il primo che vedi come codice e al suo interno ci sono due funzioni: aggiungiIVA e calcolaIVA, in un certo senso volevo creare una sorta di libreria, per capire meglio come funzionava. Il fatto di usare le due funzioni separate nn mi è stato detto da nessuno, era la soluzione che mi è venuta piu semplice.
per quanto riguarda il main, se mettevo void nn funzionava, nn so il perke...invece scritto cosi va tutto alla perfezione...
Credi che come codice sia pietoso? cioè hai qualche consiglio da darmi?

[andbin]

Originariamente inviato da sparpazeo

per quanto riguarda il file aggiungiIVA è il primo che vedi come codice e al suo interno ci sono due funzioni: aggiungiIVA e calcolaIVA, in un certo senso volevo creare una sorta di libreria

Nei file di header non si devono fare definizioni di variabili o funzioni. Non perché non sia tecnicamente fattibile (è fattibilissimo, come hai fatto tu) ma perché i file di header dovrebbero contenere solo dichiarazioni di strutture, macro, prototipi di funzioni e quant'altro (il minimo in genere) è necessario affinché un certo sorgente, includendo l'header, possa usare le funzionalità di un altro sorgente o altri sorgenti.

Originariamente inviato da sparpazeo

Credi che come codice sia pietoso?

Ehm .... purtroppo sì.


File: prova.c

#include <stdio.h>
#include "iva.h"

int main (void)
{
    float conIVA,importo,IVA;

    printf ("Inserisci l'importo al quale vuoi aggiungere l'IVA: ");
    scanf ("%f",&importo);

    conIVA = aggiungiIVA (importo);
    IVA = calcolaIVA (importo);

    printf ("Il valore aggiunto di IVA e': %.3f \n",conIVA);
    printf ("L'IVA e': %.3f \n",IVA);

    return 0;
}

File: iva.h

float aggiungiIVA (float);
float calcolaIVA (float);

File: iva.c

#include "iva.h"

float aggiungiIVA (float importo)
{
    float importoIVA;
    importoIVA = (importo * 0.2) + importo;
    return importoIVA;
}

float calcolaIVA (float importo)
{
    float IVA;
    IVA = importo * 0.2;
    return IVA;
}