このコードがセグメンテーションフォルトをもたらす理由を理解できません。私はあなたの下の文が間違っている

Question

#include<cmath> 
#include<cstdio> 
#include<vector> 
#include<iostream> 
#include<algorithm> 
//There were problems in this code. the include wasnt showing up. So i just put them as comments. 

using namespace std; 

int main() 
{ 
    int n, a[n],count[100],temp; 

    cin>>n; 

    if(n<100||n>pow(10,6)) 
     return 0; 

    for(int i=0;i<100;i++) 
    { 
     count[i]=0; 
    } 

    for(int j=0;j<n;j++) 
    { 
     cin>>a[j]; 

     if(a[j]<0||a[j]>=100) 
      return 0; 
    } 

    for(int m=0;m<n;m++) 
    { 
     temp=a[m]; 
     count[temp]++; 
    } 

    for(int s=0;s<100;s++) 
    { 
     cout<<count[s]<<" "; 
    } 

    return 0; 
} 

Answer 1

おっと！

未定義の変数（システムが望むものであれば何でもよい）で可変サイズの配列（存在しない...ちょっと）を作成しようとしているようです。

int n; 
std::cin >> n; 
int* a = new int[n]; 

するか、あなたがそれらを含めていることから、魔法C++ vectorsを使用します。

使用ポインタの代わりに、ユーザーは、アレイを作成する前に、変数を埋めてみましょう。彼らは、LIFOおよびFIFOのような複数のインアウト方法と共に、動的割り当てのような普通のポインタと型安全性のいくつかの新しい方法よりも多くの機能を提供する：

int n; 
std::cin >> n; 
std::vector<int> a(n); 

Answer 2

使用することはできませんメモリを割り当てたところ、誰も私を伝えることができますしてください。 -

int n, a[n],count[100],temp; 

cin>>n; 

固定サイズの配列の宣言は、コンパイル時にサイズを必要とする必要があります。上記の宣言では、nはコンパイル時にサイズがありません。したがって、コンパイラはスタックからメモリを割り当てることができません。

a