Problem z funkcją

Question

Witam
Mam problem z tym kodem:

#include <iostream>
using namespace std;

double zamiana(double,double);

int main()
{
  double c;

    cout<<"Podaj temperature w celcujszach: ";
    cin>>c;

    cout<<endl;

    zamiana(c);
    cout<<"Celcjusze: "<<c<<" = "<<f<<"farenhajta";

 return 0;
}

double zamiana(double f,double c)
{
    f=1.8*c+32;
    return f;
}

Podczas kompilacji kompilator wyrzuca błędy:

error: too few arguments to function 'double zamiana(double, double)'

error: 'f' was not declared in this scope'

Wiem że kompilator mówi że zmienna 'f' nie jest zdeklarowana no ale nie wiem jak to naprawic. :x
W czym leży błąd co zrobiłem źle ?

Z góry dzięki za pomoc ;)

Answer 1

Zadeklarowałeś funkcje, która otrzymuje 2 argumenty typu double, podczas gdy ty przesyłasz 1.

Comments

No tak.. ;x

Zrobiłem tak:

#include <iostream>
using namespace std;

double zamiana(double,double);

int main()
{
  double c,f;

    cout<<"Podaj temperature w celcujszach: ";
    cin>>c;

    cout<<endl;

    zamiana(c,f);
    cout<<"Celcjusze: "<<c<<" = "<<f<<" farenhajta";

 return 0;
}

double zamiana(double c,double f)
{
    f=1.8*c+32;
    return f;
}

Teraz piknie się kompiluje ale wyrzuca jakiś dziwny wynik.
Np.

Podaj temperature w celcujszach: 10
Celcjusze: 10 = 8,89462e+265 farenhajta

Co tu się staneło ? ;P

---

Notacja naukowa. Przed wypisaniem daj:

std::cout.setf(std::ios::fixed);

Poza tym to:

double zamiana(double c,double f)
{
    f=1.8*c+32;
    return f;
}

można skrócić do:

double zamiana(double c)
{
    return 1.8*c+32;
}

 

---

Notacja naukowa, mówi o przesunieciu przecinka o ileś miejsc w tym przypadku o 265

---

Teraz wyrzuca to:

Podaj temperature w celcujszach: 10
Celcjusze: 10.000000 = 889462264712478112049907796649030166290167191649239588369
58035625835421378018589973327306972321633758444827277904199658254480727141402118
61480766563178854076275193298356151508186792219758643200867939234086698885658793
4045628719390226331286811968471250123331414786048.000000 farenhajta
Process returned 0 (0x0)   execution time : 32.042 s
Press any key to continue.

:x

#include <iostream>
using namespace std;

double zamiana(double,double);

int main()
{
  double c,f;

    cout<<"Podaj temperature w celcujszach: ";
    cin>>c;

    cout<<endl;

    zamiana(c,f);

    std::cout.setf(std::ios::fixed);
    cout<<"Celcjusze: "<<c<<" = "<<f<<" farenhajta";

 return 0;
}

double zamiana(double c,double f)
{
    f=1.8*c+32;
    return f;
}

I bez std:: jest tak samo

---

Tak może zaokrąglenie liczby do iluś tam miejscach po przecnku, biblioteka cmath.

http://cpp0x.pl/kursy/Kurs-C++/Biblioteka-math-h/322

std raczej musi być bo to standard c, dlatego jest std, bynajmiej tak czytałem

---

Radzę przerobić jakiś porządny kurs, w którym dobrze omówiono funkcje bo ich nie rozumiesz. Wpisujesz zmienną f, która jest niezainicjalizowana i ma w sobie śmieci z pamięci. Twoja funkcja zwraca double więc to co zwraca przypisz do zmiennej f i wtedy ją wypisuj(ewentualnie zmienną f przekazuj przez referencje, typ funkcji zmień na void i usuń return).

Answer 2

• W funkcji main brakuje definicji zmiennej f. Powinno być raczej tak:

  double f = zamiana(c);

• Funkcja zamiana powinna wyglądać wg mnie tak (wystarczy tylko jeden argument):

  double zamiana(double c);

  Przyjmuje ona stopnie Celsjusza, które zmienia na stopnie Fahrenheita.

• Parametr f, który był wcześniej parametrem funkcji zamiana przenieś do funkcji:

  double f=1.8*c+32;
  return f;

Answer 3

Zadekleruj "f"

A więc yrobiłęm tak i się kompiluje, czytaj error, bo nie był tylko jeden error.

#include <iostream>
using namespace std;
 
double zamiana();
 
int main()
{
  double c,f;
 
    cout<<"Podaj temperature w celcujszach: ";
    cin>>c;
 
    cout<<endl;
 
    zamiana();
    cout<<"Celcjusze: "<<c<<" = "<<f<<"farenhajta";
 
 return 0;
}
 
double zamiana()
{
	double f, c;
    f=1.8*c+32;
    return f;
}

 

Answer 4

Witam, czytając komentarze zauważyłem, że jest problem z wyświetleniem odpowiedniej liczby liczb po przecinku. Robią to takie instrukcje:

#include <iomanip> //Załaczamy odpowiedną bibliotekę
cout<<fixed; //Zmiana wyświetlenia bez wartości wykładniczej
cout<<setprecision(2); //Ustawiamy liczbę wyświetlanych liczb po przecinku (np. 2)

Cieszę się jeśli pomogłem