[Początkujący] Jeśli liczba jest nieparzysta chcę ją wyświetlić.

Question

Witam.
Polecenie zadania: Stwórz program sprawdzający, czy wśród liczb x,y i z wprowadzonych z klawiatury znajduje się liczba nieparzysta. Jeśli chociaż jedna z tych liczb spełnia podany warunek, ma pojawić się komunikat "TAK" i wyświetlić tą (te) liczby, w przeciwnym wypadku - komunikat "NIE".

Program działa świetnie, tylko gdy liczby są parzyste to wyskakują randomowe cyfry (przypuszczam, że numery komórek w RAM-ie).

Pytania:
1. Dlaczego one wyskakują?
2. Jak zrobić, żeby nie wyskakiwały?
3. Czy można jakoś ten kod udoskonalić/zmienić na lepszy? (zawsze można :D)
4. Czy warunek sprawdzający nieparzystość jest dobrze? (x%2==1)

Pozdrawiam i z góry dziękuję za odpowiedź.

Kod:

//algorytm sprawdzajacy czy znajduje sie liczba nieparzysta v2
#include <iostream>

using namespace std;

int main()
{
    int x,y,z,m,n,o;
    cout<<"Podaj liczby x, y, z: ";
    cin>>x>>y>>z;

    if((x%2==1)||(y%2==1)||(z%2==1)) cout<<"TAK"<<endl;
    else cout<<"NIE!!!"<<endl;
    if(x%2==1) m=x;
    if(y%2==1) n=y;
    if(z%2==1) o=z;
    cout<<"Liczby nieparzyste to: "<<m<<", "<<n<<", "<<o<<endl;

    //system "pause";
    return 0;
}

Answer 1

1. Jeżeli który z ifów się nie spełni to zmienne m, n, o mają przypadkowe wartości z pamięci.

3. Nie potrzebujesz zmiennych m, n, o.

2:

#include <iostream>

using namespace std;

int main()
{
	int x, y, z;
	cout << "Podaj liczby x, y, z: ";
	cin >> x >> y >> z;

	if ((x % 2 == 1) || (y % 2 == 1) || (z % 2 == 1))
	{
		cout << "TAK" << endl;
		cout << "Liczby nieparzyste to: ";

		if (x % 2 == 1) cout << x << " ";
		if (y % 2 == 1) cout << y << " ";
		if (z % 2 == 1) cout << z << " ";
		cout << "\n";
	}
	else cout << "NIE!!!" << endl;

	//system("pause"); // ?
	return 0;
}

4. Dobrze.

Answer 2

Bo używasz niezainicjalizowanych zmiennych. Np. zmienna m zostanie zainicjalizowana tylko wtedy gdy x jest nieparzyste. W przeciwnym wypadku będzie tam losowa wartość.

Answer 3

Dziękuję bardzo!!
God bless you all.