DecisionTree icon indicating copy to clipboard operation
DecisionTree copied to clipboard

Published 20 hours ago • verified publisher icon Linzecong

Bugs and improvement suggestions

Open Astrojia opened this issue 5 years ago • 1 comments
trafficstars

很感谢您分享的代码，

在README提供的例子中您希望通过年龄和压力大小来判断幸福等级，这里幸福等级不止两个，但在程序DecisionTree.h Line71 您定义了bool型的aaa，会使第三类幸福等级出现时产生bug。定义为int型应该更合适。

DecisionTree.h Line 207计算entropy若移到循环外可减少额外计算。

类的函数成员定义写在.h文件中，函数成员不是内联的，一旦文件被两个或以上cpp包含，该函数将被重新定义。似乎影响了其扩展性。

感谢，祝好

Astrojia avatar Feb 29 '20 10:02 Astrojia

十分感谢你的建议！

很感谢您分享的代码，

在README提供的例子中您希望通过年龄和压力大小来判断幸福等级，这里幸福等级不止两个，但在程序DecisionTree.h Line71 您定义了bool型的aaa，会使第三类幸福等级出现时产生bug。定义为int型应该更合适。

DecisionTree.h Line 207计算entropy若移到循环外可减少额外计算。

类的函数成员定义写在.h文件中，函数成员不是内联的，一旦文件被两个或以上cpp包含，该函数将被重新定义。似乎影响了其扩展性。

感谢，祝好

Linzecong avatar Mar 03 '20 02:03 Linzecong