where's mistake ?

[GuruMan94]

Please help

  #include<iostream>

using namespace std;

class OddEvenMergeSort{
public:
	int size;
	int a[];
public:
	void print(){
		for(int i=0;i<size;i++)
			cin>>a[i];
	}
	OddEvenMergeSort(){
		cout<<"shemoitanet masivis ganzomileba"<<endl;
		cin>>size;
		cout<<"shemoitanet masivis elementebi"<<endl;
		for(int i(0);i<size;i++)
			cin>>a[i];
		
	}
	~OddEvenMergeSort(){};
	void Sort(){
		oddEvenMergeSorti(0,size);
	}
	void oddEvenMergeSorti(int l,int n){
		if (n>1)
        {
            int m=n/2;
            oddEvenMergeSorti(l, m);
            oddEvenMergeSorti(l+m, m);
            oddEvenMerge(l, n, 1);
        }
	}
	void oddEvenMerge(int l, int n, int r){
        int m=r*2;
        if (m<n)
        {
            oddEvenMerge(l, n, m);      // even subsequence
            oddEvenMerge(l+r, n, m);    // odd subsequence
            for (int i=l+r; i+r<l+n; i+=m)
                compare(i, i+r);
        }
        else
            compare(l, l+r);
    }
     void compare(int i, int j)
    {
        if (a[i]>a[j])
            swap(i, j);
    }

    
};
int main() {
	OddEvenMergeSort A;
	A.Sort();
	A.print();
	return 0;
}

[GuruMan94]

any idea?

[Zhuge]

Why do you think there is a mistake?

[GuruMan94]

because it doesn't sorting. try to compile and u'll see

[keskiverto]

How do you know?

Your print() reads from cin rather that sending values to cout.

More severely, you never reserve any memory for the array. Therefore, when you do assign to array, you overwrite memory that is not yours.

[GuruMan94]

I've changed cin to cout, but it's not working still, what else i need to change ?