'funkcja_dodawania' was not declared in this scope

Question

Mam problem, zrobiłem funkcje:

#include <iostreame>

using namespace std;

int main()
{
    funkcja_dodawania(1,2,3)
}

float funkcja_dodawania(float x,float y,float suma)
    {
        cout<<"Podaj 1 liczbe: ";
        cin>> x;
        cout<<"Podaj 2 liczbe: ";
        cin>> y;
        suma=x+y;
        cout<<"Suma wynosi "<<suma;
    }

 

i wyskakuje mi error w 7 linijce: 'funkcja_dodawania' was not declared in this scope

nie mam pojęcia jak to naprawić

Answer 1

#include <iostreamE>? Mała literówka w kodzie zawsze boli :( Zmień to na #include <iostream> i powinno śmigać :)

Comments

to nie pomoże :)

---

To powinno już na 100% działać. Skopiuj :)

#include <iostream>

using namespace std;

void funkcja_dodawania(float x,float y)
{
        float suma;
cout<<"Podaj 1 liczbe: ";
cin>> x;
cout<<"Podaj 2 liczbe: ";
cin>> y;
suma=x+y;
cout<<suma;
}

int main()
{
funkcja_dodawania(1, 2);
}

 

---

Dawno nie pisałem w C++, ale wydaje mi się, że wiem w czym leżał problem. Funkcja była poniżej funkcji main. Jeśli w C++ chcesz użyć jakiejś funkcji (np. "funkcja_dodawania" w funkcji "main"), to powinieneś napisać funkcję, którą masz zamiar użyć nad funkcją, w której chcesz jej użyć :)

---

wielkie dzięki, już wiem co źle robiłem

---

mam do ciebie jeszcze jedną prośbę, mógłbyś mi powiedzieć co zrobić żeby ta funkcja nie wyświetlała się w tym pliku, i żebym mógł ją wywołać w innym pliku

---

Chodzi ci o to, żeby wywołać funkcję, która znajduje się w innym pliku?

---

tak, i też jak ją trzeba napisać w tym pliku

---

Podrozdział 7.4 ładnie wszystko wyjaśnia: http://shebang.pl/kursy/programowanie-cpp/r7-funkcje Osobiście polecam zapoznać się z całością :)

Answer 2

Pisz prototypy funkcji , angielskie nazwy zmiennych i funkcji, jedna zmienna jest niepotrzebna , nie używaj przestrzeni nazw std.

#include <iostream>
 
float TotalSum(float x, float y);   // prototyp! 
 
// zamiast przestrzeni nazw ewentualnie ...
// using std::cout;
// using std::cin;
// using std::endl;
 
 
int main(){
   
    std::cout << TotalSum(1,2) << std::endl;  
}
 
float TotalSum(float x,float y){

        std::cout<<"Podaj 1 liczbe: " << std::endl;
        std::cin>> x;
        std::cout<<"Podaj 2 liczbe: " << std::endl;
        std::cin>> y;
        return x+y;
    }

 

Comments

a jeśli bym dał nazwę prototypa funkcji, to też by działało?

---

Tzn? Jeżeli chodzi o to czy nalezy wpisywać tylko  nazwę prototypu na początku to nie. Musisz wypisać jaki typ mają argumenty przekazywane (float) i co zwraca funkcja (też float), oraz musi być  jej nazwa. Nazwy zmiennych są opcjonalne, ale pomagają potem się rozeznać w programie.

float TotalSum (float, float);

Jeżeli zaś chodzi Ci o to czy w ogóle pominąć prototyp to można, wtedy całą funkcje umieszczasz ponad funkcją która ją wywołuje (w tym przypadku main) ale musisz liczyć się z tym że sporo na tym tracisz, może nie w takim małym programie ale w większych owszem. Głownie działa tutaj kontrola błędów, ewentualna konwersja jesli chodzi o zły typ zwracany, oraz kontrola poprawności typów argumentów. Czytelniejszy program itd. Na razie tyle mi przychodzi do głowy.

---

a jak mam napisać to żeby ciało funkcji było  w jednym pliku, a jej wywołanie w innym?

---

Musisz dołączyć plik .h do .cpp. Zapewne używasz IDE, a ja uzywam do tego linuksowego terminala więc ciężko mi cokolwiek mówić jak masz je dołączyć. Ale podaje te 2 pliki 

SUMMING.CPP

#include "Summing.h"
#include <iostream>
 
int main(){
   
    std::cout << Summing(1,2) << std::endl;  
}
 

 

SUMMING.H

#include <iostream>


 static float Summing(float x,float y){

std::cout<<"Podaj 1 liczbe: " << std::endl;
std::cin>> x;
std::cout<<"Podaj 2 liczbe: " << std::endl;
std::cin>> y;
return x+y;
}

    

 

 

 

Answer 3

Zmień float na void :) Jeżeli funkcja nie zwraca żadnej wartości, a służy do wypisania czegoś, to powinna być "rodzaje" void. Jeśli chciałbyś, żeby zwracała sumę to musiałbyś napisać return suma; w funkcji i w main napisać cout<<funkcja_dodawania(1, 2); :)

EDIT: Dopiero zauważyłem, że w do metody przekazujesz sume. Dziwna praktyka, ale tak to jest na początku :D Nie wszystko się rozumie. Funkcja ma obliczać wartość sumy, a ty jej nie znasz (po co byłaby ci funkcja do liczenia, jakbyś znał wynik?), więc nie przekazujesz wartości sumy.

Ciężko mi to wytłumaczyć pisząć. Jak chcesz, to mogę z chęcia wytłumaczyć Ci o co w tym biega np. przez skype :) Napisz mi PW, jeśli jesteś zainteresowany :)

Comments

#include <iostreame>
 
using namespace std;
 
int main()
{
    funkcja_dodawania(1,2);
}
 
void funkcja_dodawania(float x,float y)
    {
float suma;
        cout<<"Podaj 1 liczbe: ";
        cin>> x;
        cout<<"Podaj 2 liczbe: ";
        cin>> y;
        suma=x+y;
        cout<<"Suma wynosi "<<suma;
    }

 

---

Lub tak:

#include <iostreame>
 
using namespace std;
 
int main()
{
    cout<<funkcja_dodawania(1,2);
}
 
float funkcja_dodawania(float x,float y)
    {
float suma;
        cout<<"Podaj 1 liczbe: ";
        cin>> x;
        cout<<"Podaj 2 liczbe: ";
        cin>> y;
        suma=x+y;
        return suma;
    }

---

wielkie dzięki

---

napisałem to sobie, i nadal mam ten błąd xd

---

Jak jest możliwość to polecam łączyć deklaracje z przypisaniem (to znaczy razem inicjacja) czyli od razu float suma = x+y;. I jest błąd w dyrektywie preprocesora.