sallam-ahmed / Polynomial-Calculator

Data Structure 2015 Course Project
Apache License 2.0
0 stars 3 forks source link

PolynimalSolver wrong assumptions #6

Closed khaledkee closed 8 years ago

khaledkee commented 8 years ago

PolynomialSolver class wrongly assumes first term has degree zero, second term has degree one, etc.

khaledkee commented 8 years ago

I have this fix, put I think it can be improved: Change:

Complex[] _cn = new Complex[p.Count];
for (int i = 0; i < p.Count; i++)
    _cn[i] = p[i].Coefficient;

To:

Complex[] _cn = new Complex[p[p.Count-1].Degree+1];
for (int i = 0; i < p.Count; i++)
    _cn[p[i].Degree] = p[i].Coefficient;

@faroukm15