C++传递vector<InvoiceItem*>作为函数参数报类型不匹配错误如何解决
错误原因
编译报错的核心是checkout函数的前置声明与实际定义的参数类型不匹配:
- 调用
checkout的位置位于main函数中,编译器处理调用代码时只读取到了提前写的checkout前置声明,该声明的参数类型为std::vector<InvoiceItem>,和你传入的vector<InvoiceItem*>类型不匹配; - 你写的参数为
vector<InvoiceItem*>的checkout实现位于调用位置之后,编译器处理调用时还没有读到该实现,因此会基于前置声明抛出类型转换失败的错误。
修复方案
找到main函数之前的checkout函数前置声明,将参数类型从vector<InvoiceItem>修改为vector<InvoiceItem*>,和实际函数定义的签名保持一致,即可解决该编译错误。
其他代码问题优化建议
- 参数传递优化:当前
vector采用值传递会触发全容器拷贝,性能开销大,建议改为const引用传递,修改函数签名为void checkout(const vector<InvoiceItem*>& order)。 - 内存泄漏修复:通过
new创建的InvoiceItem对象没有对应的delete释放,会产生内存泄漏。可以在使用完order后遍历释放所有指针指向的内存,也可以直接改用vector<InvoiceItem>存储对象实例,无需手动管理内存。 - 未初始化变量修复:
checkout函数中的double total未初始化就直接累加,会触发未定义行为,需要修改为double total = 0.0;。 - 逻辑错误修复:累加完所有商品总价后,代码用
cin >> total覆盖了计算结果,不符合结账逻辑,删除该输入语句,直接输出计算得到的total即可。
内容的提问来源于stack exchange,提问作者user17609908
相关产品推荐
相关产品推荐

