使用2个Overloaded []运算符时添加了额外的[]运算符

Question

作为一个附带项目，我正在编写几个课程，以在线性系统上进行矩阵操作和操作。类线性系统将指针保存到矩阵 std :: map中的类对象。 矩阵类本身将2D数组（“矩阵”）保存在双浮子指针中。我写了2个超载[]运算符，一个是直接从linearsystem对象返回矩阵对象的指针，另一个是从矩阵对象返回行（float *）。

这两个操作员都可以独自工作。我可以使用线性系统[“ KeyString”]从地图获取矩阵对象指针。我可以使用矩阵[row]获取行（float *）和矩阵[row] [col]从矩阵获取特定的float元素对象的2D数组。

当我把它们放在一起时，麻烦就到了。我有限的理解（上升的高级Compsci专业）告诉我，使用线性系统[“ KeyString”] [row] [col]以从线性系统中的特定数组中获取一个元素，我应该没有问题。返回类型看起来像线性系统 - ＆gt; matrix- float * - ＆gt; float。但是由于某种原因，只有在我在重载密钥运算符之后放置一个额外的[0]时才有效，因此呼叫看起来像这样：linearsystem [“键串”] [0] [0] [row] [col] [col] [col]。而且它必须是0，其他任何东西，我segfault。

要注意的另一个有趣的事情是，Clion看到[“ KeyString”]作为Overloaded和[row]，但> [0]，而不是好像它叫标准索引运营商，但在什么问题上让我感到困惑。线性系统[“ KeyString”]肯定是返回matrix *，只有一个过载[] operator。请参阅所附的屏幕快照。

ScreenShot

这是代码，让我知道是否需要更多。

线性系统[]和MAP声明：

Matrix *myNameSpace::LinearSystem::operator[](const std::string &name) {
        return matrices[name];
}

std::map<std::string, Matrix *> matrices;

矩阵[]和数组声明：

inline float *myNameSpace::Matrix::operator[](const int row) {
        return elements[row];
}
float **elements;

请注意，上述功能是内联的，因为我挑战了自己以使代码尽可能快，即使使用编译器优化，超载[]为15％比使用matrix.elements [row]慢30％。

请让我知道是否需要更多信息，这是我的第一篇文章，所以我敢肯定它不是完美的。

谢谢你！

Answer 1

您正在用C ++编写C。您不需要那样做。它增加了复杂性。那些您一直在类型之后放置的明星。这些是原始的指针，几乎应该避免它们。 线性系统[“ foo”]是矩阵*，即指向matrix的指针。它是不是 a 矩阵。和指针索引作为数组，因此linearsystem [“ foo”] [0]获取matrix*的第一个元素，将其视为数组。由于实际上只有一个元素，所以这可以解决，并且似乎可以做您想要的。如果这听起来令人困惑，那是因为。

确保您了解所有权语义。原始指针被皱眉，因为它们没有传达任何信息。如果您希望一个值由数据结构拥有，则应直接是t（或std :: simolor_ptr＆lt; t＆gt;（如果您需要虚拟继承），不是t*。如果您要进行函数参数或返回 想要通过所有权的值，则应使用/返回const t＆amp;或t＆amp;如果您需要修改指南。

我不确定您的数据是什么样子，但是矩阵类的合理表示（如果您要速度速度）是类型std :: vector＆lt的实例变量; double＆gt;。 std :: vector是double的托管数组。您可以将阵列预先放置为所需的任何尺寸，并随时随地更改它。它还可以使用三/五/五个规则。

我不确定要表示的线性系统是什么，但是我将假设矩阵本来是由线性系统拥有的（即当系统为当系统时释放，矩阵也应该释放它们的内存），所以我

std::map<std::string, Matrix> matrices;

再次查看类似的东西，您可以在std :: std :: simolor_ptr中包装矩阵（如果您计划）从中继承并做动态调度。尽管我可能会质疑您的设计选择，如果您的矩阵类旨在将其分类。

没有理由Matrix :: Operator []将原始指针返回float（实际上，“原始指针到float”是一种毫无意义的类型）。我可能建议有两个过载。

float myNameSpace::Matrix::operator[](int row) const;
float& myNameSpace::Matrix::operator[](int row);

同样，线性系统:: operator []可以具有两个过载：一个常数和一个可变的。

const Matrix& myNameSpace::LinearSystem::operator[](const std::string& name) const;
Matrix& myNameSpace::LinearSystem::operator[](const std::string& name);

参考（t＆amp;，而不是t*）很聪明，并且在需要时有效地解释，因此您可以调用matrix :: operator []在矩阵＆amp;上，而您不能在不确认间接层的情况下在矩阵*上调用该>。

那里有一个不好的C ++建议的 lot 。如果您要学习的书 /视频 /老师告诉您到处分配float ** < / code>，那么这是一本不好的书 /视频 /老师。 C ++ - 托管数据将易于错误，并且与原始指针相当（C ++编译器在优化方面都比您或我更聪明，所以让它做它的事情）。

如果您确实发现自己确实感到需要进行低级并分配到各处的原始指针，那么切换到C。C是一种专为低级指针操纵而设计的语言。 C ++是一种高级托管的内存语言；碰巧的是，出于历史原因，C ++在整个标准中偶尔放置了几百个C型分配形式的脚枪。

总结：现代C ++几乎从不使用t*，new或delete。开始对智能指针感到满意（std :: unique_ptr和std :: shared_ptr）和参考。稍后您会感谢自己。