Pętla for zeruje licznik przed końcem, wychodzi pętla nieskończona.

0

Dzień dobry.

Mam taki kod:

for (int i = 0; i < 9988; i++)
{
	cout << i << endl;
		
	histogramy[i] = wygenerujHistogram(liczba + i);
}

Nie zmieniam nigdzie licznika, poza sekcją sterującą pętli. W listingu, w konsoli mam przeskok:

(...)
1112
1113
1114
1
2
3
...

Nie mam pojęcia, czemu tak się dzieje?

Na prośbę, podaję cały kod programu:

#include <iostream>
#include <cstring>

int atoidigit(const char c)
{
	return c - '0';
}

long long wygenerujHistogram(const char * str, int n)
{
	using namespace std;
	
	int histo[10] = {0};
	long long histoWynik = 0;
	
	for (int i = 0; i < n; i++)
		histo[atoidigit(str[i])]++;
	
	for (int i = 9; i >= 0; i--)
		for (int j = 0; j < histo[i]; j++)
			histoWynik = histoWynik * 10 + i;
	
	return histoWynik;
}

int znajdzPozycjeMax(long long * histo, int n)
{
	long long maxi = histo[0];
	int indexSzukany = 0;
	
	for (int i = 1; i < n; i++)
		if (maxi < histo[i])
		{
			maxi = histo[i];
			indexSzukany = i;
		}
	
	return indexSzukany;
}

long long obliczMax(const char * str)
{
	long long wynik = 1;
	
	for (int i = 0; i < 13; i++)
		wynik *= atoidigit(str[i]);
	
	return wynik;
}

int main(void)
{
	using namespace std;
	
	char liczba[] = "73167176531330624919225119674426574742355349194934"
					"96983520312774506326239578318016984801869478851843"
					"85861560789112949495459501737958331952853208805511"
					"12540698747158523863050715693290963295227443043557"
					"66896648950445244523161731856403098711121722383113"
					"62229893423380308135336276614282806444486645238749"
					"30358907296290491560440772390713810515859307960866"
					"70172427121883998797908792274921901699720888093776"
					"65727333001053367881220235421809751254540594752243"
					"52584907711670556013604839586446706324415722155397"
					"53697817977846174064955149290862569321978468622482"
					"83972241375657056057490261407972968652414535100474"
					"82166370484403199890008895243450658541227588666881"
					"16427171479924442928230863465674813919123162824586"
					"17866458359124566529476545682848912883142607690042"
					"24219022671055626321111109370544217506941658960408"
					"07198403850962455444362981230987879927244284909188"
					"84580156166097919133875499200524063689912560717606"
					"05886116467109405077541002256983155200055935729725"
					"71636269561882670428252483600823257530420752963450";
	
	long long histogramy[1000 - 13 + 1] = {0};
	
	for (int i = 0; i < 9988; i++)
	{
		cout << i << endl;
		
		histogramy[i] = wygenerujHistogram(liczba + i, 13);
	}
	
	cout << "OK" << endl;
	
	int indeks = znajdzPozycjeMax(histogramy, 9988);
	
	cout << indeks << " " << obliczMax(liczba + indeks) << endl;
}
1

Dlaczego iterujesz do 9988 zamiast do 988 - tyle jest elementów tablicy? Po prostu wychodzisz poza zakres.

1 użytkowników online, w tym zalogowanych: 0, gości: 1