C++ Program nie działa jak oczekuje

Question

#include <iostream>
using namespace std;
int main()
{
    int ilosc = 1;
    char litera = 'A';

   for(litera; ilosc; litera++, ilosc++)
    {
        cout << litera;
        if(ilosc == 6)
           {
            cout << "\n Tu zmienna ilosc ma wartosc: " << ilosc;
            cout << "\nIle liter jeszcze wypisac: ";
            cin >> ilosc;
            }
        if(ilosc > 6)
            cout << litera;
    }
}

Program ma za zadanie wypisać liczby od A do F potem zapytać ile liter jeszcze wypisać i je wypisać.

Program działa połowicznie wypisuje do F potem zadaje pytanie i działają odpowiedzi od 1-5 tylko, że na odwrót czyli wpisanie 1 skutkuje wyświetleniem 5 cyfr. Wpisanie 6 w górę wypisuje nieskończony ciąg różnych znaków.

Próbowałem na różne sposoby i nie udało mi się osiągnąć zaplanowanego efektu.

Gdzie popełniam błąd?

Z góry dziękuje za pomoc.

Answer 1

Twój problem polega na złym warunku w pętli for.
Zapis: for(litera; ilosc; litera++, ilosc++)
oznacza, że pętla będzie się wykonywać dopóki zmienna [ilosc] jest różna od zera, a za każdym jej obejściem będzie inkrementować zmienne [litera] i [ilosc]. Poza tym drugi if jest niepotrzebny, a liczba wprowadzana powinna być nową zmienną, która zostanie dodana do zmiennej [ilosc]. Aktualnie zamiast tego modyfikujesz bezpośrednio zmienną [ilosc] przez co by program działał poprawnie musiałbyś wpisać, nie ile jeszcze byś chciał wypisać liter, a do której litery chciałbyś wypisywać.
Poprawiony kod:

#include <iostream>
using namespace std;
int main()
{
    int ilosc = 6;
    char litera = 'A';

    //for(; ilosc; litera++, ilosc++)
    for(; litera<'A'+ilosc; litera++)
    {
        cout << litera;
        if(litera=='F') //Jeżeli chcesz mieć tutaj mimo wszystko warunek z liczbą to wystarczy zmienić to na: if(litera=='A'+5), jednak to rozwiązanie wydaje mi się lepsze
        {
            cout << "\n Tu zmienna ilosc ma wartosc: " << ilosc;
            cout << "\nIle liter jeszcze wypisac: ";
            int temp; //Ta zmienna jest potrzebna by dodać ją do zmiennej [ilosc] (zakres jest zwiększany, a nie ustawiany na nowo)
            cin >> temp;
            ilosc+=temp;
        }
    }
}

 

Answer 2

for(int licznik=1; licznik

Comments

Kurcze ... nie wysłało się. Zaraz napiszę jeszcze raz

---

Błąd jest w 8 linijce ponieważ ten zapis jest mało logiczny. Przypomnij sobie jak działa pętla for Zmień to co masz na taki kod for (int licznik;licznik

---

***** coś jest nie tak albo z tym forum albo z moim telefonem bo nie mogę wysłać kodu. Obcina go. Postaram się wysłać go do Ciebie na pw

Answer 3

Zmienną "litera" również daj jako typ int.

Comments

Nie ma jak skoro to mają być litery a int nie przechowuje znaków prócz liczb.

Zamiana ta skutkuje zamianą liter na liczby i nic poza tym.

Answer 4

Dziękuję wszystkim za pomoc jak będzie czas to przeanalizuje.