C++は可変サイズの配列を禁止しています

Question

私は他の誰かが書いた既存のコードをいくつか使用していますが、コンパイルできません（ここではCの経験は限られていますが、ここで

#include "utilities.h" 
FILE *open_file(char *filename, const char*extension, const char *access) 
{ 
    char string[MAX_STR_LEN]; 
    FILE *strm = NULL; 

    if(filename[0]=='\0') 
    { 
     printf("\n INPUT FILENAME (%s) > ",access); 
     fgets(string,MAX_STR_LEN,stdin); 
     sscanf(string,"%s",filename); 
     printf(" FILE %s opened \n", filename); 
    } 
    int len=strlen(filename); 

    if(len + strlen(extension) >= MAX_STR_LEN) 
    { 
     printf("\n ERROR: String Length of %s.%s Exceeds Maximum", 
       filename, extension); 
     return(NULL); 
    } 

    // char *filename1 = new(char[len+strlen(extension)+1]); 

    const int filenameLength = len+strlen(extension)+1; 
    char *filename1 = new(char[filenameLength]); 

    strcpy(filename1,filename); // temp filename for appending extension 

    /* check if file name has .extension */ 
    /* if it does not, add .extension to it */ 
    int i=len-1; 
    while(i > 0 && filename[i--] != '.'); 
    // printf("\n Comparing %s to %s", extension, filename+i+1); 
    if(strcmp(extension, filename+i+1) ) 
     strcat(filename1,extension); 
    if((strm = fopen(filename1, access)) == NULL) 
    { 
     printf("\n ERROR OPENING FILE %s (mode %s)", filename1,access); 
    } 
    delete(filename1); 
    return(strm); 
} 

utilities.ccはエラーです。あなたが私にしてください知っている任意の追加情報が必要な場合はライン251上の

Compiling utilities.cc ... 
src/utilities.cc: In function ‘FILE* open_file(char*, const char*, const char*)’: 
src/utilities.cc:251: error: ISO C++ forbids variable-size array 
gmake: *** [/home/landon/geant4/work/tmp/Linux-g++/exampleN01/utilities.o] Error 1 

エラー は

char *filename1 = new(char[filenameLength]); 

を指します。

Answer 1

これを試してみてください代わりに

char *filename1 = new char[filenameLength]; 

filenamelengthがconstとして宣言されていない限り、あなたはこの

char filename1[filenamelength]; 

のようなスタック上のローカル可変長配列として配列を作成することはできません。あなたは配列にメモリを割り当てたとして

また、あなたはそうしないと、メモリリークを持っています

delete [] filename1; 

を使用してメモリを解放する必要があります。また、値の末尾にかっこを付けることは必須ではありません。

Answer 2

エラーは正しいです。 VLA（可変サイズ配列）はC++では禁止されています。これはVLAです：VLAない

char *filename1 = new char[filenameLength]; 

が、charの配列はヒープに割り当て：

char filename1char[filenameLength]; 

は何あなたはおそらく意味することは、このです。あなたは演算子delete[]を使用して、このポインタを削除する必要があることに注意してください：

delete[] filename1; 

Answer 3

彼らは禁じられますが、回避策は、たとえば、スタックアロケータを使用することですされている：あなたは、スタックアロケータを使用することができます

http://howardhinnant.github.io/stack_alloc.html

::std::vector（または他のコンテナと一緒に、あるいは直接に）VLAがあなた自身です。