[英]C++ Operator Overload >>
即使存在于任何地方,我也没有找到答案。 我有课,我想要重载 >> 运算符
该代码可以编译,但不能按我想要的方式工作。 我想把值放在数组内的所有单元格中,但它只适用于第一个单元格。 看起来我的循环不起作用。
编辑:下面的完整代码。 顺便说一句,抱歉语言不通,但这是我的作业,老师想要那个名字。
#include <cstdlib>
#include <iostream>
#include <string>
#include <cmath>
template <class T>
class Wielomian {
private:
int stopien;
T *wspolczynniki;
public:
friend std::ostream & operator << (std::ostream &output, Wielomian &w)
{
output << "Wielomian: ";
for (int i = w.stopien-1; i >= 0; i--)
{
output << w.wspolczynniki[i] << "x^" << i << " ";
if (i)
output << "+ ";
}
return output;
}
friend std::istream & operator >> (std::istream &input, Wielomian &w)
{
int i = 0;
do {
input >> w.wspolczynniki[i++];
} while (w.stopien < i);
return input;
}
T operator () (T x)
{
T wynik = 0;
for (int i = 0; i < this->stopien; i++)
{
wynik += this->wspolczynniki[i] * pow(x,i);
}
return wynik;
}
T& operator[](const int index)
{
return wspolczynniki[index];
}
Wielomian operator + (const Wielomian &w)
{
const Wielomian *wiekszy;
const Wielomian *mniejszy;
if (w.stopien > this->stopien)
{
wiekszy = &w;
mniejszy = this;
}
else
{
wiekszy = this;
mniejszy = &w;
}
for (int i = 0; i < mniejszy->stopien; i++)
wiekszy->wspolczynniki[i] += mniejszy->wspolczynniki[i];
return *wiekszy;
}
Wielomian operator - (const Wielomian &w)
{
const Wielomian *wiekszy;
const Wielomian *mniejszy;
if (w.stopien > this->stopien)
{
wiekszy = &w;
mniejszy = this;
}
else
{
wiekszy = this;
mniejszy = &w;
}
for (int i = 0; i < mniejszy->stopien; i++)
wiekszy->wspolczynniki[i] -= mniejszy->wspolczynniki[i];
return *wiekszy;
}
Wielomian operator = (const Wielomian &w)
{
this->stopien = w.stopien;
this->wspolczynniki = new float[this->stopien];
memcpy(this->wspolczynniki, w.wspolczynniki, w.stopien * sizeof(double));
}
Wielomian(const Wielomian &w)
{
this->stopien = w.stopien;
this->wspolczynniki = new float[this->stopien];
memcpy(this->wspolczynniki, w.wspolczynniki, w.stopien * sizeof(double));
}
Wielomian(int stopien = 0, T wspolczynik[] = { 3 })
{
this->stopien = stopien;
wspolczynniki = new T[this->stopien];
for (int i = 0; i < stopien; i++)
{
this->wspolczynniki[i] = wspolczynik[i];
}
}
~Wielomian()
{
free(this->wspolczynniki);
}
};
int main()
{
double tab[4] = {3,4,5,6};
Wielomian<double> w1(4,tab);
std::cin >> w1;
std::cout << w1;
std::cin.get();
std::cin.get();
return 0;
}
您在operator >>
while
条件是错误的。 在您的代码中,您有:
int i = 0;
do {
input >> w.wspolczynniki[i++];
} while (w.stopien < i);
i
开始时为0
,然后input >> w.wspolczynniki[i++]
后为1
。 while
条件是(w.stopien < i)
所以如果w.stopien
(在你的例子中是4
)那么小,那么i
在第一次迭代中是1
,你将继续循环。 但是4 < 1
是false
您将始终只读取一个值。
因此,让您的工作do-while
开始工作,您需要将其更改为(w.stopien > i)
。 但是当您测试索引i
是否在正确的范围内时,您根本不应该使用do-while
,而是使用while
循环。
int i = 0;
while (i < w.stopien) {
input >> w.wspolczynniki[i++];
}
甚至是一个for
循环,它会让你更清楚你在做什么:
for(int i=0; i< w.stopien; i++) {
input >> w.wspolczynniki[i];
}
除此之外 - 以及评论中已经提到的 - 永远不要将不属于一起的内存分配和释放结合起来。 如果您使用new[]
那么您必须使用delete[]
来释放内存而不是free
。
并且不要使用有符号数 ( int
) 作为索引,所以stopien
和i
应该是无符号的(例如size_t
)。 对于stopien
您应该确保在构造时它是1
或更大。
如果你被允许,你应该将形式T* wspolczynniki
切换到std::vector<T> wspolczynniki
这将允许你摆脱复制构造函数、赋值运算符、析构函数,你不需要 int stopien , and you could simplify other parts of the code using [algorithm](https://en.cppreference.com/w/cpp/algorithm), or at least keep that you normally would use a
std::vector`(或其他容器)然后自己做分配和复制。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.