Błąd w Visual Studio 2019

Mam taki kod:

#include <iostream>
using namespace std;

int main()
{
    int const Wiersz = 1000;
    int const Kolumna = 5;
    int ile = 0;
    int WynikiMiniLotka[Wiersz][Kolumna] =
    {
        {5,23,26,27,37},
        {2,12,15,23,40},
        {22,23,25,32,41},
        {5,10,12,13,17},
        {9,32,34,38,40},
        {1,3,4,11,29},
        {4,10,21,33,35},
        {5,9,11,16,42},
        {18,21,26,28,36},
        {1,2,20,28,36},
        {5,19,25,34,39},
        {12,13,21,27,28},
        {15,17,26,31,38},
        {8,20,29,37,42},
        {3,4,6,34,38},
        {3,28,31,32,40},
        {9,21,29,32,35},
        {28,36,37,38,41},
        {13,15,18,23,24},
        {3,20,26,34,42},
        {7,9,10,11,28},
        {19,25,29,38,39},
        {10,15,21,37,42},
        {3,26,34,35,42},
        {5,6,32,41,42},
        {3,12,22,32,41},
        {10,13,19,34,41},
        {4,10,14,30,32},
        {11,25,31,34,41},
        {1,9,13,31,36},
        {3,10,15,16,20},
        {11,20,23,26,36},
        {5,12,25,28,38},
    };

    for (int i = 0; i < 30; i++)
    {
        for (int j = 0;  j < 1;  j++)
            if (WynikiMiniLotka[i][j] == 3)
            ile++;
    }

    cout << ile;
    return 0;
}

Wyświetla się błąd:
Ostrzeżenie C6262 Liczba bajtów stosu używana przez funkcję („20020”) przekracza wartość /analyze:stacksize „16384”. Rozważ przeniesienie części danych na stertę. ConsoleApplication1
Błąd wskazuje na funkcję main.

Co trzeba zmienić w kodzie programu?

Ta tablica jest najwyraźniej za duża aby trzymać ją na stosie. Użyj std::vector

@kq:

kq napisał(a):

Ta tablica jest najwyraźniej za duża aby trzymać ją na stosie. Użyj std::vector

Czy wektory są mniej wydajne od tablic przy większej ilości danych?

Nie

static int WynikiMiniLotka[Wiersz][Kolumna] (zakładam jednowątkowość) pomoże. Ale lepiej zrobi to po ludzku, jak zasugerował kq.

Ja bym dał po prostu constexpr lub static const.
Jednak najlepsze rozwiązanie to czytać te dane z pliku.

MarekR22 napisał(a):

Ja bym dał po prostu constexpr lub static const.

Jednak najlepsze rozwiązanie to czytać te dane z pliku.

Dlaczego lepiej z pliku niż używać vector?

Wczytywanie z pliku jest niezależne od tego jak w pamięci będziesz to trzymał (domyślnie: użyj std::vector). To tak jakbyś pytał czemu lepiej nosić buty niż skarpety ;​)

kq napisał(a):

Wczytywanie z pliku jest niezależne od tego jak w pamięci będziesz to trzymał (domyślnie: użyj std::vector). To tak jakbyś pytał czemu lepiej nosić buty niż skarpety ;​)

Czego muszę się nauczyć, aby to co robi ten program z tablicą mógł robić z plikiem? Może jakaś mała podpowiedź w postaci kodu lub pseudo kodu?

To ja zadam pytanie pomocnicze: co takiego robisz, że pytasz o wydajność? Bo pytanie z pierwszego posta sugeruje, że raczej jesteś początkujący.

alagner napisał(a):

To ja zadam pytanie pomocnicze: co takiego robisz, że pytasz o wydajność? Bo pytanie z pierwszego posta sugeruje, że raczej jesteś początkujący.

Nic konkretnego. W ramach nauki taki program do statystyk minilotka sobie wymyśliłem. Gdzieś wyczytałem że tablice są wydajniejsze od vector przy wielu operacjach na dużej ilości danych. dlatego pytałem o wydajność.

Sylwester Sajdak napisał(a):

Wyświetla się błąd:
Ostrzeżenie C6262 Liczba bajtów stosu używana przez funkcję („20020”) przekracza wartość /analyze:stacksize „16384”. Rozważ przeniesienie części danych na stertę. ConsoleApplication1
Błąd wskazuje na funkcję main.

To nie jest błąd, to jest ostrzeżenie (warning). Kompilator cię informuje że funkcja zjada ponad 16 kB stosu, czyli „dużo”. W tym jednak przypadku nie ma zagrożenia przekroczenia rozmiaru stosu (którego rozmiar pod Visualem domyślnie wynosi 1 MB).

Azarien napisał(a):

Sylwester Sajdak napisał(a):

Wyświetla się błąd:
Ostrzeżenie C6262 Liczba bajtów stosu używana przez funkcję („20020”) przekracza wartość /analyze:stacksize „16384”. Rozważ przeniesienie części danych na stertę. ConsoleApplication1
Błąd wskazuje na funkcję main.

To nie jest błąd, to jest ostrzeżenie (warning). Kompilator cię informuje że funkcja zjada ponad 16 kB stosu, czyli „dużo”. W tym jednak przypadku nie ma zagrożenia przekroczenia rozmiaru stosu (którego rozmiar pod Visualem domyślnie wynosi 1 MB).

Wolę dmuchać na zimne. Bo co by było jak by doszło z milion nowych liczb do tablicy.

Jak to mówi dziadek Knuth "Premature optimization is the root of all evil". Btw coś ostatnio dużo szamanów amatorów lotka na 4p.

No na pewno lepiej je ładować z osobnego pliku a nie za każdym razem jak zmieniasz dane zmieniać i kąpilować program na nowo.

pragmaticdev napisał(a):

Jak to mówi dziadek Knuth "Premature optimization is the root of all evil". Btw coś ostatnio dużo szamanów amatorów lotka na 4p.

Dziadek też mawiał: "Yet we should not pass up our opportunities in that critical 3%". Jak bym uczył się programować grę to byłbym nałogowym graczem?

Najpierw zidentyfikuj problem - potem optymalizuj. Inaczej to jest bez sensu. Z drugiej strony, serio chcesz ładować dane statycznie w kodzie programu? Jeśli tak to problem leży zupełnie gdzie indziej ;)

Sylwester Sajdak napisał(a):

alagner napisał(a):

To ja zadam pytanie pomocnicze: co takiego robisz, że pytasz o wydajność? Bo pytanie z pierwszego posta sugeruje, że raczej jesteś początkujący.

Nic konkretnego. W ramach nauki taki program do statystyk minilotka sobie wymyśliłem. Gdzieś wyczytałem że tablice są wydajniejsze od vector przy wielu operacjach na dużej ilości danych. dlatego pytałem o wydajność.

A pokaż link. Vectory to też tablice, tylko że na sterydach. Także to muszą być jakieś konkretne przypadki. Jedyną zaletą tablic jest ich mały rozmiar. Wydajnościowo nie odczujesz różnicy.

@Czitels: Nie pamiętam już jaka to była strona więc nie podam do niej linku. Za to pamiętam jeden z ostatnio przeglądanych: https://www.educba.com/c-plus-plus-vector-vs-array/. Temat można zamknąć bo i tak muszę nauczyć się pracować z plikami w C++.