Outfile이 메모리 위반으로 인해 충돌을 일으킴

Question

숫자를 10 진수에서 2 진수로 변환하는 프로그램을 작성 중입니다. 나는 이미 그것을위한 알고리즘을 가지고 있고 프로그램은 괜찮아요 cout을 사용하는 동안 작동합니다. 그러나, 루프에서 outfile을 사용하자마자 프로그램은 오류 코드 (0xC0000005)와 충돌합니다. 아웃 오브 바운드 요소에 액세스하여

#include <iostream> 
#include <iomanip> 
#include <fstream> 
#include <cmath> 
using namespace std; 

int main() 
{ 
    int num, remainder_count; 
    ifstream infile; //define new input file stream 
    ofstream outfile; //define new output file stream 

    infile.open("C:\\Users\\Arctic-Gaming\\CLionProjects\\working\\Source\\Binary Conversion (Loop w File)\\Binary Input.txt"); //connect the stream to an actual file 
    if (!infile) 
    { 
     cout << "Cannot open input file! Program aborted" << endl; 
     return 1; 
    } 

    outfile.open("C:\\Users\\Arctic-Gaming\\CLionProjects\\working\\Source\\Binary Conversion (Loop w File)\\Decimal Output.txt"); //connect the stream to an actual file 

    do 
    { 
     int remainder [15] = {0}; 
     remainder_count = 15; 

     infile >> num; 
     outfile << "\n" << num << endl; 
     if (num > 0 && num <= 65535) 
     { 
      while (num > 0) 
      { 
       remainder[remainder_count] = num % 2; 
       num /= 2; 
       remainder_count--; 
      } 

      remainder_count = 0; 
      while (remainder_count < 16) 
      { 
       if (remainder_count % 4 == 0) 
       { 
        outfile << " "; 
       } 
       outfile << remainder[remainder_count]; 
       remainder_count++; 
      } 
     } 
     else if (num == 0) 
      outfile << "0000 0000 0000 0000" << endl; 

     else 
      cout << "Error! Invalid Input." << endl; 
    } 
    while (!infile.eof()); 
} 

Answer 1

프로그램이 보증되지 행동 : 여기 내 소스 코드입니다. 동작이 정의되지 않았으므로이 문제는 파일 스트림을 사용하는 것과는 반대로 std::cout을 사용하는 것과 아무런 관련이 없습니다.

int remainder [15] = {0}; 
//... 
remainder_count = 15; 
//... 
remainder[remainder_count] = num % 2; // out-of-bounds access (remainder[15] is out-of-bounds) 

위의 행이 실행되면 프로그램이 어떻게 작동할지에 대한 모든 배팅이 해제됩니다. 배열의 유효한 색인 범위는 0에서 n-1까지이며, 여기서 n은 배열의 요소 수입니다. 유효한 유효 색인은 remainder 배열의 경우 0, 1, 2, 최대 14입니다.

는 대신 정의되지 않은 동작, 당신은 당신이 at()을 사용하여 해당 요소에 액세스하는 즉시 던져진 std::out_of_range 예외를 얻을 것 대신 일반 C++ 배열의 std::array를 사용으로 전환 한 경우. 당신이 볼 당신이있어서

 std::array<int, 15> remainder= {{0}}; 
    remainder_count = 15; 
    //...  
    if (num > 0 && num <= 65535) 
    { 
     while (num > 0) 
     { 
      remainder.at(remainder_count) = num % 2; //... exception thrown 

Live Example

그래서, 프로그램은 "잘 실행하지"결코, 당신이 범위를 벗어날 배열의 예정되지 않도록 당신은 당신의 프로그램을 수정해야합니다 .