错误：转换为非标量类型

Question

我正在为一个赋值创建一组派生类。 我被指示使用char数组（c-strings）。 当我编译时，我不断收到错误：

Homework11.cpp: In function âint main()â:
Homework11.cpp:72: error: conversion from âchar [10]â to non-scalar type âBusinessâ requested
Homework11.cpp:73: error: conversion from âchar [10]â to non-scalar type âBusinessâ requested
Homework11.cpp:74: error: conversion from âchar [10]â to non-scalar type âAccountâ requested
Homework11.cpp:75: error: conversion from âchar [10]â to non-scalar type âAccountâ requested

我相当肯定我的问题源于我尝试将实例变量Name设置为发送的参数。这是我的代码，其中包含注释，我相信问题可能存在。

#include <iomanip>
#include <iostream>
#include <cstdlib>
#include <cstring>
using namespace std;

class Person{
public:
        Person() {}
        Person(char theName[]) {strcpy(name,theName);}
        void getName(char theName[]) // I think the problem may be here or in the line above
                { theName = name;}
private:
        char name[80];
 };

class Account : public Person{
public:
        Account() :accountNum(0),balance(0) {}
        Account(int actNo, char theName[])
                :Person(theName),accountNum(actNo),balance(0) {}
        void setBal(float theBalance)
                {balance = theBalance;}
        void deposit(float numDeposited)
                { balance = balance + numDeposited;}
        float withdraw(float numWithdrawn)
                { balance = balance -numWithdrawn;
                  return numWithdrawn;}
        float getBal() {return balance;}
        void printBal();
private:
        int accountNum;
        float balance;
};

 class Business : public Account{
 public:
        Business() : checkFee(0.0) {}
        Business(int actNo, char theName[])
                : Account(actNo, theName),checkFee(0.0) {}
        float withdraw(float numWithdrawn)
                {float newBalance = getBal()-numWithdrawn-checkFee;
                 setBal(newBalance);
                  return numWithdrawn;}
        void setFee(float fee) {checkFee = fee;}
 private:
        float checkFee;
};

void Account::printBal()
{
        char name[80];
        getName(name);
        cout<<setw(10)<<"Account # "<<accountNum<<setw(10)<<
              name<<setw(10)<<balance<<endl;
}


int main()
{
        char businessName1[10]="Business1";
        char businessName2[10] ="Business2";
        char regularName1[10] = "Regular1";
        char regularName2[10] = "Regular2";

       //The following 4 lines are the ones I am getting the error for
        Business bs1 = (1,businessName1);
        Business bs2 = (2,businessName2);
        Account rg1 = (1, regularName1);
        Account rg2 = (2, regularName2);

        cout<<"Intially: "<<endl;
        rg1.printBal();
        rg2.printBal();
        bs1.printBal();
        bs2.printBal();

        bs1.deposit(1000.00);
        bs2.deposit(1000.00);
        rg1.deposit(1000.00);
        rg2.deposit(1000.00);

        cout<<"----------------------------------------"<<endl;
       cout<<"After adding 1000.00 to all accounts:"<<endl;
        rg1.printBal();
        rg2.printBal();
         bs1.printBal();
        bs2.printBal();

        bs1.setFee(1.00);
        bs1.withdraw(500);
        bs2.withdraw(500);
        bs1.deposit(250);
        bs2.deposit(250);
        rg1.withdraw(500);
        rg2.deposit(500);

        cout<<"---------------------------------------"<<endl;
        cout<<"Finially:"<<endl;
        rg1.printBal();
        rg2.printBal();
        bs1.printBal();
        bs2.printBal();

        return 0;
}

Answer 1

正确的语法是Business bs1(1,businessName1); 。 如果你想使用= ，你也可以使用copy Business bs2 = Business(2,businessName2); 。

前者称为直接初始化。 它们并不完全相同，请参阅复制初始化和直接初始化之间的C ++是否存在差异？ 有关深入的信息。

在Business bs1 = (1,businessName1); 1和数组businessName1由逗号运算符分隔。 逗号运算符计算第一个操作数，即1并抛弃结果并返回第二个操作数的值，这是您的情况下的数组。 换句话说，您的代码相当于Business bs1 = businessName1; 。 这就是错误消息说它无法将char[10]转换为Business对象的原因。

Answer 2

将产生错误的第一行更改为Business bs1(1,businessName1); 其余的类似。 这是用于在堆栈上初始化类实例的C ++习惯用法。

Business bs2 = Business(2,businessName2); 我认为，正如Jesse Good所建议的那样，Java语言习惯在C ++中很糟糕。 它的速度较慢，因为有两个隐式构造函数调用和一个复制构造函数调用，而不是Business bs1(1,businessName1);的单个构造函数调用Business bs1(1,businessName1); 。 在这种情况下，还有另一个陷阱：您尚未为Business类型定义复制构造函数，这意味着编译器将为您创建一个执行浅复制的复制构造函数。 bs2.name将结束指向内存的指针，当bs2超出范围时，该指针不一定正确释放 - 经典的内存泄漏。

相应的C ++习惯用法是在堆上构造一个新对象，然后将其地址分配给一个指针： Business *bs2 = new Business(2,businessName2); 。

您的代码还有另一个问题。 通常，在C或C ++中按名称分配数组也是不好的风格（请记住，像char theName[]这样的静态分配字符串只是一种特殊的数组）。 在Person查看getName()的定义：

void getName(char theName[])
    { theName = name; }

这是分配数组名称（不是指针，但是关闭表兄弟），而不是将一个字符串的内容复制到另一个字符串。 在printBal() ，然后写

char name[80];
getName(name);

当getName()执行时，它将printBal()的局部变量name绑定到参数theName 。 到目前为止，这么好，虽然你选择的变量名称可能会有点混乱。 :)但随后身体getName()执行和指定的私有实例变量的地址name ，以theName （这是一个数组的名字-再次，一种特殊的指针）。 当getName()返回时， printBal()的局部变量name没有永久更改。 编写Person::getName()的正确方法是使用strcpy() ，就像编写第二个Person构造函数一样：

void getName(char theName[])
    { strcpy(theName,name); }