Pętla w c++ nieskończona

Question

Witam

Dlaczego ta pętla jest nieskończona?

#include <iostream>

using namespace std;
int a,i,j=0;
int main()
{
    cout << "podaj liczbe a" << endl;
    cin>>a;
    for(i=0;i<=a;)
    {i++;
        if(i%13==0)
        {
            continue;
        }
while(i<=a)
        {cout<<"*"<<endl;
    }}
    return 0;
}

 

Comments

#include <iostream>
#include <string>
 
int main() {
    int number;
 
    std::cin >> number;
    for(auto i = 0; i < number; ++i) {
        std::cout << std::string(i + 1, '*') << '\n';
    }
}

 

Answer 1

Dałeś pętlę while - będzie się ona wykonywać dopóki i jest mniejsze lub równe a. W pętli nie zmieniasz wartości i czyli przy pierwszym obiegu cały czas i ma wartość 1 - stąd pętla nieskończona. Zmień while na else (co 13 linijka będzie wtedy bez gwiazdki).

Comments

Ja chcę żeby każda linijka miała tyle gwiazdek, jaki jest numer linijki, ale mi nie wychodzi. :(

---

#include <iostream>
 
using namespace std;

int main()
{
	int a,i,j=0;
    cout << "podaj liczbe a" << endl;
    cin>>a;
    for(i=1;i<=a;i++)
    {
        if(i%13==0)
        {
        	
            continue;
        }
	else
	{
		for(int x = 0; x < i; x++)
       {
       		cout<<"*";
	   }
	   
	   cout<<endl;
	}
       
    }
    return 0;
}

To może coś takiego? Wtedy każda linijka (oprócz co 13) ma tyle gwiazdek ile numer linijki (licząc od 1).

Jeśli nie zależy ci na tej co 13 linijce wyrzuć z kodu powyżej if  oraz else.

---

ok, już zgadłam, działa.

Answer 2

#include <iostream>

using namespace std;
int a,i,j;
int main()
{
    cout << "podaj liczbe a" << endl;
    cin>>a;
    for(i=1;i<=a;i++)
    {
        if(i%13==0)
        {
            continue;
        }
for (j=1;j<i;j++)
{
    cout<<"*";
}cout<<endl;}
    return 0;
}

Jak mam tak to działa z continue

Comments

Dlaczego a, i i j są globalne?

---

a dlaczego nie

---

Dlatego, że tworzenie ponad miarę zmiennych globalnych jest złym stylem zmniejszającym czytelność kodu.

Nie lepiej tak:

int main()
{
    int a;

    cout << "podaj liczbe a" << endl;
    cin >> a;

    for (int i = 1; i <= a; i++) {
        if (i % 13 == 0) continue;
        for (int j = 1; j < i; j++) cout << '*';
        cout << '\n';
    }

    return 0;
}

?