C++类成员函数打印变量值异常，输入未传递至输出阶段

问题现象 #

运行程序接收用户输入并验证后，无法输出预期结果。输入维度为5、颜色为blue时，输出垃圾值：

The length is: 2.97299e-41
The width is: 2.99205e-41
The height is: 2.99205e-41
The color is: 
The volume is: 0

预期输出：

The length is: 5
The width is: 5
The height is: 5
The color is: blue
The volume is: 125

核心问题分析 #

• 全局变量与类成员完全脱节：你定义了全局的length/width/height/color字符串变量，但创建Cube对象后，从未把这些输入值赋值给类的成员变量，导致类成员都是未初始化的垃圾值（float类型未初始化会是极小随机值，string为空）。
• 输入验证逻辑完全失效：
  1. 验证后直接强制重新读取输入，不管之前的输入是否合法，相当于验证步骤白做。比如验证长度后，不管flag是不是1，都会再次提示输入并覆盖之前的值。
  2. 颜色验证逻辑颠倒：你判断!isdigit(color[i])就标记非法，这意味着颜色必须是纯数字？但颜色应该是字母组成，逻辑完全搞反，且循环内重复读取输入会导致数据混乱。
• 类型转换无实际作用：stof(length)只是把字符串转成float，但没有将结果赋值给任何变量，转换后的值直接丢弃，完全没用。
• 体积初始化时机错误：float volume = length * width * height;是在类成员声明时初始化，此时length/width/height都未赋值，计算出的volume自然是垃圾值。
• 类定义位置违规：Cube类定义在main函数之后，main中创建对象时编译器还未识别该类，虽然部分编译器能兼容，但不符合C++标准。

修正后的代码 #

#include <iostream>
#include <string>
#include <cctype>
using namespace std;

// 前置定义Cube类，确保main函数能识别
class Cube {
public:
    float length;
    float width;
    float height;
    string color;

    // 计算体积的方法，在长宽高赋值后调用
    float calculateVolume() {
        return length * width * height;
    }

    // 显示立方体信息
    void display() {
        cout << "The length is: " << length << endl;
        cout << "The width is: " << width << endl;
        cout << "The height is: " << height << endl;
        cout << "The color is: " << color << endl;
        cout << "The volume is: " << calculateVolume() << endl;
    }
};

// 验证字符串是否为正整数
bool isPositiveInteger(const string& s) {
    if (s.empty()) return false;
    for (char c : s) {
        if (!isdigit(c)) return false;
    }
    return true;
}

// 验证字符串是否为纯字母（颜色仅允许字母）
bool isAlphaString(const string& s) {
    if (s.empty()) return false;
    for (char c : s) {
        if (!isalpha(c)) return false;
    }
    return true;
}

int main() {
    string inputLength, inputWidth, inputHeight, inputColor;
    float length, width, height;

    // 处理长度输入，直到合法
    do {
        cout << "Enter the cube length: ";
        cin >> inputLength;
        if (!isPositiveInteger(inputLength)) {
            cout << "Enter a valid positive number." << endl;
        } else {
            length = stof(inputLength);
            break;
        }
    } while (true);

    // 处理宽度输入
    do {
        cout << "Enter the cube width: ";
        cin >> inputWidth;
        if (!isPositiveInteger(inputWidth)) {
            cout << "Enter a valid positive number." << endl;
        } else {
            width = stof(inputWidth);
            break;
        }
    } while (true);

    // 处理高度输入
    do {
        cout << "Enter the cube height: ";
        cin >> inputHeight;
        if (!isPositiveInteger(inputHeight)) {
            cout << "Enter a valid positive number." << endl;
        } else {
            height = stof(inputHeight);
            break;
        }
    } while (true);

    // 处理颜色输入
    do {
        cout << "Enter the cube color: ";
        cin >> inputColor;
        if (!isAlphaString(inputColor)) {
            cout << "Enter a valid color (letters only)." << endl;
        } else {
            break;
        }
    } while (true);

    // 将输入值赋值给Cube对象成员
    Cube access;
    access.length = length;
    access.width = width;
    access.height = height;
    access.color = inputColor;

    // 调用显示方法
    access.display();

    return 0;
}

关键修正说明 #

1. 类定义前置：把Cube类移到main函数之前，符合C++语法规范。
2. 输入逻辑重构：用do-while循环处理输入，直到用户输入合法值，避免无效重复输入。
3. 数据关联：将输入并转换后的数值赋值给Cube对象的成员变量，确保输入数据传递到类中。
4. 体积计算调整：把体积计算改为成员方法，在显示时调用，确保基于已赋值的长宽高计算。
5. 验证逻辑修正：颜色验证改为检查是否纯字母，数字验证确保输入为正整数，符合需求。
6. 移除全局变量：改用局部变量管理输入，减少全局变量带来的混乱。

内容的提问来源于stack exchange，提问作者Islandari