Ошибка c6262 c

При запуске процесса система выделяет в его адресном пространстве
специальную область под стек. В стеке хранятся локальные переменные,
адреса возвратов и прочая «сиюминутная» информация.

Например, когда вы в main размещаете массив int-ов из 100 элементов,
съедается, как минимум, «sizeof (int) * 100» байт стека. Когда вызывается
функция, адрес возврата размером «sizeof (ptr)» тоже сохраняется в стеке.
Чем больше вложенных вызовов и чем больше в каждой функции локальных
переменных, тем больше потребляется стековой памяти. Если пренебрегать
этим, можно нарваться на переполнение стека и тогда программа будет
аварийно завершена. Часто это происходит в алгоритмах с использованием
рекурсии или когда создаются стековые объекты большого размера.

Характерной особенностью стека является то, что он растет «вниз»,
от больших адресов к меньшим.

По умолчанию Windows резервирует под стек потока 1 мегабайт, причем
лишь часть этой памяти закоммичена, то есть, обеспечена реальными
физическими страницами памяти, остальное находится в состоянии «reserved».
При «поедании» стека система коммитит все новые и новые страницы для
него, пока не достигает конца области, которая изначально была
выделена под стек, и тогда наступает stack overflow…

For practice purposes, I tried creating a console app that will store basic information of employees and store it in a text file called employee-info.txt. I was able to write data in the text file but everytime the program writes new data, the C6262 warning appears. The warning can be seen in the function called employeeDataChecker() and employeeWriteData() under Employee.cpp, which I will indicate the codes below.

For employeeDataChecker(), the warning states that function uses ‘22800’ bytes of stack: exceeds/analyze:stacksize’16384′. Consider moving some data to heap. For employeeWriteData, it has the same warning but larger bytes of stack ‘23264’. I even tried adding pointers in the arrays I integrated in the code, but it mess up my code even more.

Kindly advice and some guidance on how to manage this or provide me with a better solution in achieving my goal. You can also pinpoint bad practices that I made, so that I can learn from it to improve my code. The code I provided below is long but I’m still working on it everyday to minimize the code.

Currently, this is the data stored in employee-info.txt:

       ID           Firstname            Lastname                 Sales
        1              Dwyane             Anthony             250000.00
        2              Joseph            Cardinal             450000.00
        4               Bruno                Mars             250000.00

This is the code for Employee.h:

#pragma once
#include<string>
#include<iostream>

class Employee
{
public:
struct EmployeeRecord {
    static const int recordSize = 100;
    static const int fieldSize = 4;
    std::string record[recordSize][fieldSize];
};

public:
Employee();
~Employee();
void employeeDataChecker();
void employeeWriteData(int recordCount, std::string recordCopy[EmployeeRecord::recordSize][EmployeeRecord::fieldSize]);
void employeeDisplayData();
EmployeeRecord& employeeReturnRecordArray();

private:
EmployeeRecord emp_record;

};

And here is the code in Employee.cpp:

void Employee::employeeDataChecker() {
//Check if there are data in the employee-info.txt
EmployeeRecord emp;
std::ifstream inFile, inFile2;
int recordCount = 0;
inFile.open("C:\\Users\\RJ\\Desktop\\employee-info.txt"); // use to get the number of values stored in record array
inFile2.open("C:\\Users\\RJ\\Desktop\\employee-info.txt"); // use to get all contents in record array

for (int index = 0; index < emp.recordSize; index++) {
    for (int index2 = 0; index2 < emp.fieldSize; index2++) {
        while (inFile >> emp.record[index][index2]) {
            recordCount++;
        }
    }
}

for (int index = 0; index < emp.recordSize; index++) {
    for (int index2 = 0; index2 < emp.fieldSize; index2++) {
        inFile2 >> emp.record[index][index2];
    }
}
//used as a dummy array to hold the values in record array and pass as an argument 
std::string recordCopy[emp.recordSize][emp.fieldSize];


    for (int index = 0; index < emp.recordSize; index++) {
        for (int index2 = 0; index2 < emp.fieldSize; index2++) {
            recordCopy[index][index2] = emp.record[index][index2];
        }
    }



inFile.close();
inFile2.close();

employeeWriteData(recordCount, recordCopy);
}//end of employeeDataChecker

void Employee::employeeWriteData(int recordCount, std::string recordCopy[EmployeeRecord::recordSize][EmployeeRecord::fieldSize]) {
Employee emp;
EmployeeRecord empRec;
int numEmployees;
std::string firstName, lastName;
std::string fullName = "";
double sales;

std::ofstream outFile;
outFile.open("C:\\Users\\RJ\\Desktop\\employee-info.txt", std::ofstream::app);

//pass all values from recordCopy to record array
for (int index = 0; index < empRec.recordSize; index++) {
    for (int index2 = 0; index2 < empRec.fieldSize; index2++) {
        empRec.record[index][index2] = recordCopy[index][index2];
    }
}

std::cout << "------------------------------------------------" << std::endl;
std::cout << "Enter The Number of Employees to Add: ";
std::cin >> numEmployees;
std::cin.get();

if (recordCount == 0) {

    //If employee-info.txt is empty.
    outFile << std::fixed << std::showpoint << std::setprecision(2);
    outFile << std::setw(5) << "ID";
    outFile << std::setw(20) << "Firstname";
    outFile << std::setw(20) << "Lastname";
    outFile << std::setw(22) << "Sales" << std::endl;

    for (int index = 0; index < numEmployees; index++) {
        int empID = index;
        empID++;
        std::cout << "*****Employee ID No." << empID << "*****" << std::endl;
        std::cout << "Enter First Name: ";
        std::getline(std::cin, firstName);
        std::cout << "Enter Last Name: ";
        std::getline(std::cin, lastName);
        fullName = firstName + " " + lastName;

        std::cout << "Enter Total Sales: ";
        std::cin >> sales;

        std::string empIDConverted = std::to_string(empID);
        std::string salesConverted = std::to_string(sales);

        for (int index2 = 0; index2 < empRec.fieldSize; index2++) {
            if (index2 == 0) {
                empRec.record[index][index2] = empIDConverted;
            }
            else if (index2 == 1) {
                empRec.record[index][index2] = firstName;
            }
            else if (index2 == 2) {
                empRec.record[index][index2] = lastName;
            }
            else if (index2 == 3) {
                empRec.record[index][index2] = salesConverted;
            }
        }

        outFile << std::fixed << std::showpoint << std::setprecision(2);
        int numSetW;
        for (int index2 = 0; index2 < empRec.fieldSize; index2++) {

            if (index2 == 0) {
                numSetW = 5;
            }
            else if(index2 == 3){
                numSetW = 22;
            }
            else {
                numSetW = 20;
            }

            if (index2 == (empRec.fieldSize - 1)) {

                std::string getSales = empRec.record[index][index2];
                double salesPreviousType;
                std::istringstream iss(getSales);
                iss >> salesPreviousType;

                outFile << std::setw(numSetW) << salesPreviousType << std::endl;
            }
            else {
                outFile << std::setw(numSetW) << empRec.record[index][index2];
            }
        }
        std::cin.get();
    }

}
else if (recordCount >= empRec.fieldSize) {
    //If employee-info.txt already has an existing data. It will write new data at the end of record array.

    int rows = recordCount / empRec.fieldSize;
    int increasedFieldSize = empRec.fieldSize * 2;
    int preIndex = rows;

    for (int index = rows; index < (numEmployees + preIndex); index++) {
        int empID = index;
        empID++;
        std::cout << "*****Employee ID No." << empID << "*****" << std::endl;
        std::cout << "Enter First Name: ";
        std::getline(std::cin, firstName);
        std::cout << "Enter Last Name: ";
        std::getline(std::cin, lastName);
        fullName = firstName + " " + lastName;

        std::cout << "Enter Total Sales: ";
        std::cin >> sales;

        std::string empIDConverted = std::to_string(empID);
        std::string salesConverted = std::to_string(sales);

        for (int index2 = 0; index2 < empRec.fieldSize; index2++) {
            if (index2 == 0) {
                empRec.record[index][index2] = empIDConverted;
            }
            else if (index2 == 1) {
                empRec.record[index][index2] = firstName;
            }
            else if (index2 == 2) {
                empRec.record[index][index2] = lastName;
            }
            else if (index2 == 3) {
                empRec.record[index][index2] = salesConverted;
            }
        }

        outFile << std::fixed << std::showpoint << std::setprecision(2);
        int numSetW;
        for (int index2 = 0; index2 < empRec.fieldSize; index2++) {

            if (index2 == 0) {
                numSetW = 5;
            }
            else if (index2 == 3) {
                numSetW = 22;
            }
            else {
                numSetW = 20;
            }

            if (index2 == (empRec.fieldSize - 1)) {

                std::string getSales = empRec.record[index][index2];
                double salesPreviousType;
                std::istringstream iss(getSales);
                iss >> salesPreviousType;

                outFile << std::setw(numSetW) << salesPreviousType << std::endl;
            }
            else {
                outFile << std::setw(numSetW) << empRec.record[index][index2];
            }

        }
        std::cin.get();
    }

}
else {
    std::cout << "Number problem!";
}


outFile.close();
  }//end of employeeWriteData

TCHAR someString[40000];

Решение

Другие решения

I’m trying to learn C++ by making a password generator, and when I was almost done, I had a warning (Warning C6262 Function uses ‘20044’ bytes of stack: exceeds /analyze:stacksize ‘16384’. Consider moving some data to heap.)
And since I’m a beginner at C++ and only now the basics, I don’t know what to do.

Since I had 2 arrays in the function, I’ve tried to move them out of the main function, It did lower the number of bytes it was using (20100), it was still too high. I’ve tried to read Microsoft’s errors guide, but I don’t know about heaps or moving said data to the heaps in C++

Here’s the code:

#include <iostream>
#include <random>
using namespace std;

class Randomer {
    // random seed by default
    std::mt19937 gen_;
    std::uniform_int_distribution<size_t> dist_;

public:
    /*  ... some convenient ctors ... */

    Randomer(size_t min, size_t max, unsigned int seed = std::random_device{}())
        : gen_{ seed }, dist_{ min, max } {
    }

    // if you want predictable numbers
    void SetSeed(unsigned int seed) {
        gen_.seed(seed);
    }

    size_t operator()() {
        return dist_(gen_);
    }
};

string alphabet{ 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L',
                          'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X',
                          'Y', 'Z', 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j',
                          'k', 'l', 'm', 'n','o','p', 'q', 'r', 's', 't', 'u', 'v',
                          'w', 'x', 'y', 'z' };
string specialchars{ ' ','!','#','$', '%', '&','(',')','*','+','-','.','/',':',';','<','=','>','?','@' };
Randomer rand1{ 0,2 };
Randomer randint{ 0,9 };
Randomer randalpha{ 0,26 };
Randomer randspecial{ 0,20 };
int main()
{
    while (true) { //This loop is to restart the program if an invaild response is detected.
        int i;
        int lengthofpassword = 8;

        cout << "Do you want to generate a password or to type in a existing password to make it stronger?\n\n(1 for generation, 2 for existing password)\n";
        cin >> i;
        int g = 0;
        if (i == 1) {
            cout << "\nWhat is the length of the password?\n";
            cin >> lengthofpassword;
            while (g <= lengthofpassword -1) {

                //if (rand() == 0) {//numb
                //  cout << randint();
                //}
                //else if (rand() == 1) {//letter
                //  cout << alphabet[randalpha()];
                //}
                switch (rand1())
                {
                case 0:
                    cout << alphabet[randalpha()];
                    break;
                case 1:
                    cout << randint();
                    break;
                case 2:
                    cout << specialchars[randspecial()];
                    break;
                default:
                    break;
                }

                g = g + 1;
                //cout << "\n\n" << g << "\n\n";
            }
            cout << "\n";
            system("pause");
            return 0;

        }
        else if (i == 2) {
            cout << "\n";
            system("pause");
            return 0;
        }
        else {
            cout << "\n";
            cout << "Invaild Response!\n";
        }
    }
//  system("pause");
    return 0;
}

Trying to condense it for stackoverflow didn’t work.
Although it hasn’t broken yet and works as expected, I feel like I should learn this type of stuff now rather than later.