'iloraz' was not declared in this scope

Question

#include <iostream>

using namespace std;

int main ()
{
	float a;
	float b;
	
	cout<<"podaj liczbe"<<endl;
	cout<<"a= ";
	cin>>a;
	cout<<"b= ";
	cin>>b;
	if (b= 0)
	{
		iloraz= a/b;
		cout<<"a/b= "<<iloraz;
	}
	else
	    cout<<"nie dzielimy przez zero";
	return 0;
}

co mam zrobić żeby nie było tego błędu bo wymiękam

Answer 1

Odpowiadając na Twoje pytanie, to nie zadeklarowałeś nigdzie zmiennej iloraz.

Najlepiej byłoby ją zadeklarować tam, gdzie jest potrzebna (a nie w zakresie globalnym, jak to w komentarzu do swojej odpowiedzi napisał zkff56). Zadeklaruj ją w funkcji main "obok" deklaracji zmiennej a oraz b i będzie dobrze :)

---

W Twoim programie jest jeszcze inny ważny błąd:

if (b= 0)

powinno być zamienione na:

if (b != 0)

ponieważ operator = jest operatorem przypisania i (zazwyczaj) nie chcesz używać go w wyrażeniach warunkowych, często jest mylony z operatorem porównania ==. Oczywiście w tym wypadku chcesz sprawdzić czy możesz dzielić, czyli czy b jest różne (!=) od zera.

Edit: uwzględnienie uwagi z komentarza.

Comments

if (b= 0)
    {
        iloraz= a/b;
        cout<<"a/b= "<<iloraz;
    }
    else
        cout<<"nie dzielimy przez zero";

Jesteś pewny, że powinno być ==?

Poza tym 

if (b= 0)

To będzie zawsze prawdą.

---

A dlaczego nie w zakresie globalnym? Jak będzie chciał robić kiedyś inne funkcje i w nich wykorzystać tę zmienną?

---

@zkkf56: to wtedy sobie przekaże wartość zmiennej iloraz jako parametr funkcji . Nie ma co nadużywać zmiennych globalnych.

@tkz: pierwsza uwaga trafna, poprawiłem odpowiedź. Dzięki!

Co do drugiej, to o ile pamiętam, to wartość wyrażenia "b = 0" będzie równa wartości b po tym przypisaniu, czyli w tym wypadku 0. Zero jest tożsame z fałszem, czyli w rezultacie zawsze powinna być wykonana gałąź druga, z wiadomością o niedzieleniu przez zero.

---

Fakt, operator przypisania, zwraca wartość przypisaną, w tym przypadku faktycznie masz rację.

Answer 2

Nie zadeklarowałeś zmiennej iloraz.

Comments

Uzupełnię twój kod:
 

#include <iostream>
 
using namespace std;

//Musisz zadeklarować zmienną
float iloraz;

int main ()
{
    float a;
    float b;
     
    cout<<"podaj liczbe"<<endl;
    cout<<"a= ";
    cin>>a;
    cout<<"b= ";
    cin>>b;
    if (b= 0)
    {
        iloraz= a/b;
        cout<<"a/b= "<<iloraz;
    }
    else
        cout<<"nie dzielimy przez zero";
    return 0;
}

Myślę, że to wszystko i możesz zamykać temat, pozdrawiam :)

---

Nie wszystko, tego if'a też należy poprawić, bo wątpię by był taki zamysł. Zważając na to, że dzielenie przez 0 w c++ jest ub.

---

Racja.