Problem z programem switch case

Question

Mam napisany program i jeżeli rozpatruje go osobno, czyli, np wybiorę 3,potem wyjdę wybiorę 1 itd... wszystko działa ok. Jednak jak program idzie po kolei tak jak ma iść czyli case(1),case(2),case(3),case(4).Wywala mi krzaki na trzecim case. 

Ciekawostka, jeżeli usunę case(3), to mam krzaki w case(4)

#include<iostream>
using namespace std;
int main()
{
    int tablica[1000], n, wybor, k;
    char napis[100];
    cout << "MENU" << endl;
    cout << "1.Pobieranie liczby elementow" << endl;
    cout << "2.Wyswietlanie dodatnich elementow" << endl;
    cout << "3.Wyswietla napis az do 0" << endl;
    cout << "4.Liczenie spacji" << endl;
    cin >> wybor;
    cin.ignore();
    switch (wybor)
    {
    case(1):
    
    cout << "Podaj liczbe elementow";
    cin >> n;
    for (int i = 0; i < n; i++)
    {
        cout << "Podaj " << i+1 << " element tablicy";
        cin >> tablica[i];
    }
    case(2):
        for (int i = 0; i < n; i++)
        {
            if (tablica[i] > 0) {
                cout << endl;
                cout << tablica[i] << endl;
            }
        }
    case(3):
        cout << "Wpisz linie tekstu";
        cin.getline(napis, 100);
        for (int i = 0; napis[i] != '0'; i++)
        {
            cout << napis[i];
        }
    case(4):
        int spacja = 0;
        cout << "Wpisz linie tekstu";
        cin.getline(napis, 100);
        for (int i = 0; i<100; i++)
        {
            if (napis[i] == ' ') {
                spacja = spacja + 1;
            }
        }
        cout << endl;
        cout << spacja;

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

 

Answer 1

Krzaczki masz dlatego bo wpewnym momencie zaczynasz wypisywac z tablicy rzeczy ktorych nie ma bo nie zostaly tam zapisane(sa tam smieci czyli twoje krzaczki). Popracuj nad petla wypisujaca tablice typu char. I to jest bledem wielkim  napis[i]!='0'(w przykladzie jaki pokazales)

Answer 2

for (int i = 0; napis[i] != '0'; i++)

Powinno być:

for (int i = 0; napis[i] != '\0'; i++)

Zastanów się również nad używaniem string, zamiast łańcuchów w stylu C. Dzięki temu mógłbyś napisać to dużo prościej:

#include <iostream>
#include <cstdlib>//to do system("pause"), ale lepiej skorzystac z cin.get()
#include <string>

using namespace std;

int main()
{
	int tablica[1000], rozmiar, wybor;
	string napis;
	cout << "MENU" << endl;
	cout << "1. Pobieranie liczby elementow" << endl;
	cout << "2. Wyswietlanie dodatnich elementow" << endl;
	cout << "3. Wyswietla napis az do 0" << endl;
	cout << "4. Liczenie spacji" << endl;
	cin >> wybor;
	cin.ignore();
	switch (wybor)
	{
	case 1:
		cout << "Podaj liczbe elementow (maksymalnie 1000): ";
		cin >> rozmiar;
		for (int i = 0; i < rozmiar; i++)
		{
			cout << "Podaj " << i + 1 << " element tablicy";
			cin >> tablica[i];
		}
	case 2:
		for (int i = 0; i < rozmiar; i++)
		{
			if (tablica[i] > 0) {
				cout << endl << tablica[i] << endl;
			}
		}
	case 3:
		cout << "Wpisz linie tekstu ";
		getline(cin, napis);//pobierz linie tekstu z konsoli i zapisz ja w napis
		cout << "Twoj tekst: " << napis << endl;
	case 4:
		int spacja = 0;
		cout << "Wpisz linie tekstu";
		getline(cin, napis);//pobierz linie tekstu z konsoli i zapisz ja w napis
		for (size_t i = 0; i < napis.size(); i++)//napis.size() reprezentuje obecna dlugosc napisu 
		{
			if (napis[i] == ' ') {
				spacja++;//inkrementacja, czyli dodanie 1 do aktualnej wartosci zmiennej
			}
		}
		cout << "Liczba spacji w Twoim tekscie: " << spacja << endl;
	}
	system("pause");
}

Generalnie to menu nie ma większego sensu, bo i tak trzeba najpierw skorzystać z opcji numer 1, aby móc skorzystać z opcji numer 2 (inaczej wartości będą niezainicjalizowane) oraz bez względu na wybór użytkownika będą wykonywane pozostałe opcje case np. użytkownik podaje 3, a wykonywane są 3 i 4 (to ze względu na brak instrukcji break; przed następnym case).

 

 

Comments

Wpisanie w for (!='\0') nie pomogło ale jak wstawiłem break i pętle while(1) case(3) trzeba wybrać i nie następuje po case(1) i case(2) dlatego nie wywala mi błędu. Dzięki za pomoc. Będę pewnie jeszcze odzywał się na forum.