Question

我希望有人能指出我的方向，为什么我的getter函数在声明了setter的函数之外使用时似乎无法正确访问。

我正在读取文件，使用getline函数存储每一行的变量，然后将消息分配给 Message.h 中的私有变量。

当我在m1.getmessage()函数中插入readfile()时，它输出的结果非常好，输出正确（我的文本文件中的消息行），但是j 必须给出空白的输出在main()中。我已经尝试了好几个小时了，我一直在阅读有关局部变量的信息，但据我所知，变量已经设置，并且在公共函数中，因此我看不到我要去哪里了在我凌晨4点到达酒庄之前，任何帮助将不胜感激。

Message.h

#include <iostream>

class Message {
private:
std::string message;
std::string cipher;


public:
    void readFile();

    void setMessage(std::string msg) {
        message = msg;
    }
    void setCipher(std::string ciph) {
        cipher = ciph;
    }

    std::string getMessage() {
        return message;
    }
    std::string getCipher() {
        return cipher;
    }
};

Message.cpp

#include "Message.h"
#include <fstream>
#include <iostream>

void Message::readFile()    {
    std::string fileUsername;
    std::string fileForename;
    std::string fileSurname;
    std::string fileAge;
    std::string fileTime;
    std::string fileHour;
    std::string fileMin;
    std::string fileSec;
    std::string fileCipher;
    std::string fileMessage;
    Message m1;
    std::fstream file;
    std::string filename;
    std::cout << "Please enter file name: " << std::endl;
    getline(std::cin, filename);
    file.open(filename);
    if (file.is_open()) {
        std::cout << "File opened" << std::endl;
    } else {
        std::cout << "Wrong file name" << std::endl;
    }
    while(file.is_open()) {

        getline(file, fileUsername);
        getline(file, fileForename);
        getline(file, fileSurname);
        getline(file, fileAge);
        getline(file, fileHour, ':');
        getline(file, fileMin, ':');
        getline(file, fileSec);
        getline(file, fileCipher);
        getline(file, fileMessage);
        file.close();
    }
    m1.setMessage(fileMessage);
    m1.setCipher(fileCipher);
    m1.getMessage();

};

Main.cpp

#include <iostream>
#include <iomanip>
#include <fstream>
#include "Message.h"
#include "Caesar.h"
#include "XOR.h"


int main() {

    Message m1;
    m1.readFile();
    std::cout << m1.getMessage();


    return 0;
}

主体中的cout不返回任何内容，但是如果我将其传输到m1.readfile（）中，它将完美地输出变量。

这是我第一次尝试面向对象的编程，这绝对是我的弱点。预先感谢您的任何建议。

Answer 1

在Message::readFile()函数中，您不是在当前对象上调用函数setMessage和setCipher，而是在局部变量m1上调用。局部变量在函数末尾被丢弃，消息和密码最终不会得到保存。相反，您应该致电

setMessage(fileMessage);
setCipher(fileCipher);
getMessage();

代替

m1.setMessage(fileMessage);
m1.setCipher(fileCipher);
m1.getMessage();

这将更新当前对象的消息和密码变量，然后可以从getMessage()函数中打印main。

Answer 2

您在readfile中创建一个局部变量m1。然后在函数结束时将其丢弃。可能您想返回对象或设置当前实例

Answer 3

您不必在该类的成员函数中创建对象

我的建议：

省略/删除Message m1;中的readFile()
直接将功能称为

setMessage(fileMessage); setCipher(fileCipher); getMessage();

Answer 4

您可以使用此；指向您当前正在处理的对象的指针。如下所示。

this->setMessage(fileMessage);

或

(*this).setMessage(fileMessage);

C ++类的getter在main中不返回任何值，但是在类中返回吗？

4 个答案: