배열 메모리를 어떻게 할당 해제 할 수 있습니까?

Question

이 출력으로 인해 프로그램이 작동을 멈 춥니 다. 어떻게 할당을 해제 할 수 있습니까? 나는 내 코드에서 가지고있는 것을 시도했지만 그것은 도움이되지 않는다. 전체 코드는 여기에서 찾을 수 있습니다 :이 코드를 실행하면 full code

for (i = 0 ; i < n ; i++) { 
    if (
     !((x[i] >= 0) && (x[i] <= a) & 
      (y[i] >= 0) && (y[i] <= b) 
     ) 
    ) { 
     cout << x[i] << ' ' << y[i] << ' ' << s[i] << ' ' << "SPADOL" << endl; 
    } else { 
     cout << x[i] << ' ' << y[i] << ' ' << s[i] << endl; 
    } // ALLOCATION IS DONE AT THIS POINT 

    delete [] x; 
    delete [] y; 
    delete [] s; 
} 

, 내가 입력 x와 y를 시작하고 내가 y 값을 입력 한 직후, 프로그램이 작동을 멈 춥니 다. 한 행만 입력하면 프로그램이 완벽하게 실행되지만 두 개 이상의 행을 입력하려고 할 때 문제가 발생합니다 .. 첫 번째 출력이 인쇄 된 직후 프로그램이 충돌합니다. 벡터와 내 전체 코드 : x, y 및 s 동적 new[] 및 n > 1를 통해 할당 할 수있는 배열을 가정

#include <iostream> 
#include <string> 
#include <vector> 
#include <stdlib.h> 
#include <string> 

using namespace std; 
// vector<string> inputs; 
int n; // pocet uloh 
int a; // rozmer obdlznika a 
int b; // rozmer obdlznika b 
vector<int> x; 
vector<int> y; 
vector<string> s; 
int i; 
int d; 

static const char alpha[] = {'D', 'L', 'P'}; 
char genRandom() 
{ 
    return alpha[rand() % strlen(alpha)]; 
} 
// end of generator 


int main() { 

    cin >> n; 

    while(!((n >= 1)&&(n <=15000))) 
    { 
      cout << "max 15000" << flush; 
      cin >> n; 
      } 


    cin >> a >> b; 

    while(!((a >= 1)&&(a <=100) & (b >= 1)&&(b <= 100)&&(a!=b))) 
    { 
      cout << "chyba max 100 alebo a!=b" << endl; 
      cin >> a >> b; 
      } 


     for (i = 0; i < n; i++) 
       {  
        cout << "Uloha " << i+1 << ":" << endl; 

        cin >> x[i]; 
        cin >> y[i]; 
        cin >> s[i]; 

        while(!((x[i]>=0)&&(x[i]<=a))) { 
        cout << "Try Again x: " << flush; 
        cin >> x[i];} 
        while(!((y[i]>=0)&&(y[i]<=b))) { 
        cout << "Try Again y: " << flush; 
        cin >> y[i];} 


        if (s[i] == "S"){ 
          y[i] = (y[i]+1); 
          }else if (s[i] == "J"){ 
           y[i] = (y[i]-1); 
           }else if (s[i] == "V"){ 
             x[i] = (x[i]+1); 
             }else if (s[i] == "Z"){ 
              x[i] = (x[i]-1); 
              } 
        cin >> d; 
        while(!((d>=1)&& (d<=200))) { 
        cout << "Try Again d: " << flush; 
        cin >> d;} 


        for (int counter=0; counter<d; counter++) 
        { 
        cout << genRandom(); 
        } 
        cout << endl; 

      } // koniec for 

      for (i = 0 ; i < n ; i++) 
            { 
             if(!(((x[i]>=0)&&(x[i]<=a)) | ((y[i]>=0)&&(y[i]<=b)))){ 
                cout << x[i] << ' ' << y[i] << ' ' << s[i] << ' ' << "SPADOL" << endl; 
                }else{ 
                cout << x[i] << ' ' << y[i] << ' ' << s[i] << endl; 

                } 

            } 




system("pause"); 

} 

Answer 1

은, 루프 주위, xy, s 처음 할당을 해제합니다 다음에 시도합니다 다음 반복을 x[i] 읽으십시오.

당신은 루프의 delete[] 외부를 이동해야합니다, 또는 더 나은, 당신은 std::vector처럼 자동적으로 자신을 정리 컨테이너를 사용해야합니다

std::vector<int> x; 
std::vector<int> y; 
std::vector<int> s; 

for (i = 0 ; i < n ; i++) { 
    if (
     !((x[i] >= 0) && (x[i] <= a) & 
      (y[i] >= 0) && (y[i] <= b) 
     ) 
    ) { 
     cout << x[i] << ' ' << y[i] << ' ' << s[i] << ' ' << "SPADOL" << endl; 
    } else { 
     cout << x[i] << ' ' << y[i] << ' ' << s[i] << endl; 
    } 
} 

Answer 2

당신이 키워드 "새"로 할당하지는 경우 , 삭제를 호출 할 필요가 없습니다. 힙에 할당하는 경우에만 할당을 해제해야합니다. 표준 lib 클래스를 사용하여 내부적으로 힙에 대한 새/삭제 작업을 수행 할 수 있습니다.