mastervlad Posted February 27, 2015 Report Share Posted February 27, 2015 Am facut o clasa vector in c++ si am incercat sa definesc operatia de adunare pe componente, dar imi da o eroare.class vector{ private: int *p; int n; public: vector(){}; void add(vector other); vector(int size); ~vector(){delete [] p;} vector (const vector &v); int operator[](int i) {return p[i];} vector operator=(const vector&a);};vector vector::add(vector v){ vector vi; n=v.n; for (int i=0;i<n;i++) vi.p[i]=p[i]+v.p[i]; return vi;}Care ar putea fi problema? Mersi mult. Quote Link to comment Share on other sites More sharing options...
nedo Posted February 27, 2015 Report Share Posted February 27, 2015 Problema principala este in functia de adunare. Tu creezi un vector gol, ce are membrul p, neinitializat. p nu indica catre nici o adresa/variabila. Va trebui sa faci in asa fel ca p sa aiba o valoare atunci cand il folosesti, faci un constructor de genul vector(int marime); si initializezi n cu marime si p cu new int*[n]. De asemenea in cazul de fata, cand vi este sters vei primi o eroare pentru ca incerci sa stergi un pointer gol. Quote Link to comment Share on other sites More sharing options...
mastervlad Posted February 27, 2015 Author Report Share Posted February 27, 2015 Problema principala este in functia de adunare. Tu creezi un vector gol, ce are membrul p, neinitializat. p nu indica catre nici o adresa/variabila. Va trebui sa faci in asa fel ca p sa aiba o valoare atunci cand il folosesti, faci un constructor de genul vector(int marime); si initializezi n cu marime si p cu new int*[n]. De asemenea in cazul de fata, cand vi este sters vei primi o eroare pentru ca incerci sa stergi un pointer gol.Facusem initial un constructor de genul:vector::vector(int size){ n=size; p=new int[n]; for (int i=0;i<n;i++){ cout<<"Elementul"; cin>>p[i];}}Cand declar vi, il declar vi(x) unde x=v.n? Quote Link to comment Share on other sites More sharing options...
nedo Posted February 27, 2015 Report Share Posted February 27, 2015 Exact. Quote Link to comment Share on other sites More sharing options...