C++ pętla for zakończa program

Question

Robię prostą aplikacje konsolową ale pętla for zakończa program i kolejne skrypty nie są wykonywane i nie wiem co mam zrobić

   for (int i=1; i<=liczba; ++i){
           system( "cls" );
            cout << "podaj nick gracza " << i << ":" << endl;
            cin >> nick[i];}

 

Comments

Wstaw więcej kodu i dokładny opis co się dzieje. Co to nick? Jaki ma rozmiar? W jaki sposób kończy się działanie programu?

---

#include <iostream>
#include <windows.h>
#include <time.h>

using namespace std;

int main ()
{
            int liczba = 1, team, pro, i = 1;
        string tak;


        cout << "Podaj liczbe graczy:" << endl;
        cin >> liczba;
        system( "cls" );
        cout << "Podaj ilosc timow:" << endl;
        cin >> team;
        system( "cls" );
        cout << "Zapisano "<< liczba << " graczy i "<< team << " druzyny" <<endl;
        pro=liczba/team;
        cout << "Bdzie w kazdym timie bedzie po: " << pro << " osob" << endl;
        Sleep(3000);




    string nick[liczba];

void wpis();
{
   for (int i=1; i<=liczba; ++i){
           system( "cls" );
            cout << "podaj nick gracza " << i << ":" << endl;
            cin >> nick[i];}
}
           /* while(i == liczba)
            {
            system( "cls" );
            cout << "podaj nick gracza " << i << ":" << endl;
            cin >> nick[i];
            i = i+1;
            }*/
        system( "cls" );
        cout << "Zapisani gracze:" <<endl;
       for (int a=1; a<=liczba; ++a)
        {
            cout << a << ". " << nick[a] << endl;
        }
    cout << "Rozpoczac losowanie? [T/N]" << endl;
    cin >> tak;
    system( "cls" );

    if (tak == "T")
    {}
 else
    {
        system( "cls" );
        cout << "Ok, anulowanie losowania";
    }

}

Tu daje cały program

---

To Ci się kompilować nie powinno. Prześlij se te zmienne w jakiejś strukturze lub osobno do tej funkcji.

 

//EDIT

To zagnieżdżona funkcja, która nie jest wywoływana.

---

I BŁAGAM zmień timow na teamow.

Answer 1

Wychodzisz poza zakres tablicy. Trochę poprawiony kod:

#include <ctime> // zamiast time.h
#include <iostream>
#include <string> // brakujący nagłówek
#include <windows.h>

using namespace std;

int main()
{
	int liczba = 1, team, pro; // okropne nazwy - praktycznie nie wiadomo do czego zmienne będą używane
	string tak; 

	cout << "Podaj liczbe graczy:" << endl;
	cin >> liczba;
	system("cls");
	cout << "Podaj ilosc timow:" << endl;
	cin >> team;
	system("cls");
	cout << "Zapisano " << liczba << " graczy i " << team << " druzyny" << endl;
	pro = liczba / team;
	cout << "Bdzie w kazdym timie bedzie po: " << pro << " osob" << endl; // conajmniej!
	Sleep(3000);

	//string nick[liczba]; // VLA nie jest częścią standardu C++
	string* nick = new string[liczba];

	for (int i = 0; i < liczba; ++i) { // tablice o rozmiarze 'liczba' indeksujemy od 0 do liczba - 1, inaczej wychodzisz poza tablicę
		system("cls");
		cout << "podaj nick gracza " << i + 1 << ":" << endl; // i + 1
		cin >> nick[i];
	}
	
	system("cls");
	cout << "Zapisani gracze:" << endl;
	for (int a = 0; a < liczba; ++a) // to samo co wyżej
	{
		cout << a + 1 << ". " << nick[a] << endl;
	}
	cout << "Rozpoczac losowanie? [T/N]" << endl;
	cin >> tak;
	system("cls");

	if (tak == "T")
	{
	}
	else
	{
		system("cls");
		cout << "Ok, anulowanie losowania";
	}


	delete[] nick; 
}

Comments

Dzięki!

Answer 2

Dałeś średnik przed '}', powinien być za klamrą.

Edit. przepraszam, jednak nie powinno tak być.

Comments

nie, też nie działa

---

wstaw cały kod.

---

@Karol_Tesla, już

 

Answer 3

#include <iostream>
#include <windows.h>
#include <time.h>
 
using namespace std;
 
int main ()
{
            int liczba = 1, team, pro, i = 1;
        string tak;
 
 
        cout << "Podaj liczbe graczy:" << endl;
        cin >> liczba;
        system( "cls" );
        cout << "Podaj ilosc timow:" << endl;
        cin >> team;
        system( "cls" );
        cout << "Zapisano "<< liczba << " graczy i "<< team << " druzyny" <<endl;
        pro=liczba/team;
        cout << "Bdzie w kazdym timie bedzie po: " << pro << " osob" << endl;
        Sleep(3000);
 
 
 
 
    string nick[liczba]; //a ta alokacja to co?
wpis() //może byś wywołał tą pętle?
//Komputer: nie ma nic dalej. Skończę program. Ale coś mi tutaj nie pasuje. Nie było rozkazu return

 
void wpis() //tutaj był nie potrzebny średnik
{
   for (int i=1; i<=liczba; ++i)
      {
           system( "cls" );
            cout << "podaj nick gracza " << i << ":" << endl;
            cin >> nick[i];
       }
//} // A to co?
           /* while(i == liczba)
            {
            system( "cls" );
            cout << "podaj nick gracza " << i << ":" << endl;
            cin >> nick[i];
            i = i+1;
            }*/
        system( "cls" );
        cout << "Zapisani gracze:" <<endl;
       for (int a=1; a<=liczba; ++a)
        {
            cout << a << ". " << nick[a] << endl;
        }
    cout << "Rozpoczac losowanie? [T/N]" << endl;
    cin >> tak;
    system( "cls" );
 
    if (tak == "T")
    {}
 else
    {
        system( "cls" );
        cout << "Ok, anulowanie losowania";
    }
 return 0; //polecam.
}//tutaj jakaś kropa była.