in C++期待の一次式の前に ']'

Question

ここはクイックソート用のコードです。私は初心者ですので親切にお手伝いください。

#include<iostream> 
using namespace std; 

class quick 
{ 
    private: 
     int n,left,right,i,j; 
     float a[55]; 
    public: 
     void getdata(); 
     void sort(float[],int,int); 
     void putdata(); 

}; 
void quick::getdata() 
{ 
    cout<<"Enter how many elements you want to enter:"; 
    cin>>n; 

    for(int k=0;k<n;k++) 
    { 
     cout<<"Enter percentage of students:"<<k+1<<":"; 
     cin>>a[k]; 
    } 

    left=0; 
    right=n-1; 
} 

void quick::putdata() 
{ 

    for(int k=0;k<5;k++) 
    { 
     cout<<"\nSorted marks are:"<<a[k]<<endl; 
    } 
} 

void quick::sort(float a[],int left,int right) 
{ 
    if(left<right) 
    { 
     int i=left; 
     int j=right+1; 
     float pivot=a[left]; 
     do{ 
      do{ 
       i++; 
      }while((a[i]<pivot)&& left<right); 

      do{ 
       j--; 
      }while(a[j]>pivot); 

      if(i<j) 
       swap(a[i],a[j]); 
     }while(i<j); 

     a[left]=a[j]; 
     a[j]=pivot; 
     sort(a,left,j-1); 
     sort(a,j+1,right); 
    } 
} 

int main() 
{ 

    quick obj; 
    obj.getdata(); 
    obj.sort(a[],left,right); 
    obj.putdata(); 
return (0); 
} 

それは私int main()機能でエラー与えている：このスコープ内で宣言されていない

1. を。
2. ']'の前に一次式が必要です。

Answer 1

それはあなたがint main()内部変数としてa宣言されていない言及したよう。むしろそれは迅速な目的です。関数では、のような配列をaとして渡すことはありません。a、left、rightは、オブジェクトから直接アクセスできないクラスのプライベート変数です。 publicと宣言し、obj.a、obj.left、obj.rightのソート機能として使用します。

完全なコード：答えとして

#include<iostream> 
using namespace std; 

class quick 
{ 
    public: 
     int n,left,right,i,j; 
     float a[55]; 

     void getdata(); 
     void sort(float[],int,int); 
     void putdata(); 

}; 
void quick::getdata() 
{ 
    cout<<"Enter how many elements you want to enter:"; 
    cin>>n; 

    for(int k=0;k<n;k++) 
    { 
     cout<<"Enter percentage of students:"<<k+1<<":"; 
     cin>>a[k]; 
    } 

    left=0; 
    right=n-1; 
} 

void quick::putdata() 
{ 

    for(int k=0;k<5;k++) 
    { 
     cout<<"\nSorted marks are:"<<a[k]<<endl; 
    } 
} 

void quick::sort(float a[],int left,int right) 
{ 
    if(left<right) 
    { 
     int i=left; 
     int j=right+1; 
     float pivot=a[left]; 
     do{ 
      do{ 
       i++; 
      }while((a[i]<pivot)&& left<right); 

      do{ 
       j--; 
      }while(a[j]>pivot); 

      if(i<j) 
       swap(a[i],a[j]); 
     }while(i<j); 

     a[left]=a[j]; 
     a[j]=pivot; 
     sort(a,left,j-1); 
     sort(a,j+1,right); 
    } 
} 

int main() 
{ 

    quick obj; 
    obj.getdata(); 
    obj.sort(obj.a,obj.left,obj.right); 
    obj.putdata(); 
return (0); 
} 

Answer 2

は@Shubhamカトリで与えられます。ここに訂正されたコードがあります。

#include<iostream> 
using namespace std; 
class quick 
{ 
    public: int n,left,right,i,j; 
     float a[55]; 
    public: 
     void getdata(); 
     void sort(float[],int,int); 
     void putdata(); 

}; 
void quick::getdata() 
{ 
    cout<<"Enter how many elements you want to enter:"; 
     cin>>n; 
    for(int k=0;k<n;k++) 
    { 
     cout<<"Enter percentage of students:"<<k+1<<":"; 
     cin>>a[k]; 
    } 

    left=0; 
    right=n-1; 
} 
void quick::putdata() 
{ 

    for(int k=0;k<n;k++) 
    { 
     cout<<"\nSorted marks are:"<<a[k]<<endl; 

    } 

} 
void quick::sort(float a[],int left,int right) 
{ 
    if(left<right) 
     { 
      int i=left; 
      int j=right+1; 
      float pivot=a[left]; 
    do{ 
     do{ 
      i++; 
       }while((a[i]<pivot)&& left<right); 

     do{ 
      j--; 
      }while(a[j]>pivot); 

     if(i<j) 
      swap(a[i],a[j]); 
    } 

     while(i<j); 
    a[left]=a[j]; 
    a[j]=pivot; 
    sort(a,left,j-1); 
    sort(a,j+1,right); 
} 
} 
int main() 
{ 

    quick obj; 
    obj.getdata(); 
    obj.sort(obj.a,obj.left,obj.right); 
    obj.putdata(); 
return (0); 
}