[c++新手问题] redefinition of default argument 报错应该如何解决

extern 那行去掉等号后面的东西？

@Valyrian 如果去掉的话在没有参数对函数进行调用的时候，如`print_tree();`则会显示 No matching function for call to 'private tree'

那再加一行 extern string print_tree(); 这样？

@Valyrian 还是会报错，

Undefined symbols for architecture x86_64:
"print_tree()", referenced from:
_main in main.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

definition 里面不应该有默认参数值吧

查了一下，好像是你原版的基础上把 cpp 文件里的等号后面删掉

default argument 应该写在函数声明里，而不是函数定义里。

>> 改成 > >

默认参数只能出现在声明
定义中不能加

7L +1

@ipoh 新版本无所谓了

extern 后面有初始化就算是定义了， extern 失去意义了。再从其它地方定义 print_tree 当然就算 redefine 了

7 楼和 9 楼说了答案。你在定义处不要写 default argument

顺手给你加个引用"The distinction between a declaration and a definition may seem obscure at this
point but is actually important. To use a variable in more than one file requires
declarations that are separate from the variable ’ s definition. To use the same variable
in multiple files, we must define that variable in one and only one file. Other files
that use that variable must declare but not define that variable."

example:
"extern double pi = 3.1416; // definition"

详见<C++ Primer 5th> "2.2.2. Variable Declarations and Definitions"

实现的 cpp 不能加上默认参数，你把 cpp 文件中的默认参数删除掉看看，

@zonyitoo
@lazyhare
@fgcmaster

谢谢大家的帮助，我现在把代码修改了一下，把 extern 关键字去掉了， default argument 也只在函数的头文件里面定义了。但是出现了另一个报错，说

duplicate symbol _root in:
/Users/Boris/Library/Developer/Xcode/DerivedData/Algorithems_ToolBox-duheuacdjorkbdensozspjevuypq/Build/Intermediates/Algorithems ToolBox.build/Debug/Algorithems ToolBox.build/Objects-normal/x86_64/binary_tree.o
/Users/Boris/Library/Developer/Xcode/DerivedData/Algorithems_ToolBox-duheuacdjorkbdensozspjevuypq/Build/Intermediates/Algorithems ToolBox.build/Debug/Algorithems ToolBox.build/Objects-normal/x86_64/main.o

这个是 main.cpp 的代码：

#include <iostream>
#include "binary_tree.hpp"

int main(){
print_tree();
}

这个是 binary_tree.hpp 的代码，我觉得并没有重复申明 root 变量啊，只在头文件里面生成了。

#include "binary_tree.hpp"
using namespace std;

Node* get_min(Node* node){
if (node->left)
return get_min(node->left);
return node;
}

Node* add(int key, int value){
Node* node;
node->key = key;
node->value = value;
return add(node);
}

Node* add(Node * node){
Node* current_node = root;
while(1){
if (!current_node) {
current_node = node;
return current_node;
}
if (node->key == current_node->key) {
return current_node;
}
else if (node->key > current_node->key){
current_node = current_node->right;
}
else{
current_node = current_node-> left;
}
}


}

string print_tree(stack<Node*, vector<Node*>> nodes){
cout<<nodes.top()<<endl<<root<<endl;
return "";
}

@borischenc 你把 root 写在头文件里然后重复包含了吧。。

@nifury 因为头文件里面有

#ifndef binary_tree_hpp
#define binary_tree_hpp

所以照理来说是不会出现重复包含的情况的。

@borischenc
不，你在 binary_tree.cpp 里包含一次，在 main 里包含一次，所以重复了
只要在一个 CPP 里定义，然后 hpp 里 extern 就行

@nifury 嗯嗯解决了！！谢谢！！