访问数组中的对象重新创建对象？

Question

我正在制作一个主机游戏，我正在制作一张地图。地图是一个向量数组。向量包含我打印到控制台的字符。我的代码：

“窗口.h”

#include <string>
#include <vector>

class Row {
public:
    std::vector<char> row;
    int id;
    Row();
    void printRow();
    void resetRow();
    void insertStringIntoRow(std::string toInsert, int startIndex);
    std::vector<char> getRow() {
        return row;
    }
};

class Window {
public:
    void showMap();
    void writeToMap(std::string stringToInsert, int rowNum, int startIndex);
    void writeInRectangle(std::string stringToWrite, int rowNum, int startIndex);
    void setCursorToPosition(int x, int y);
    void resetMap();
    Row getRowAt(int index);
};

void initColors();
void setWindow(Window windowToSet);
Window getGameWindow();

“窗口.cpp”

#include "Window.h"
#include <iostream>
#include <Windows.h>
#include <WinBase.h>
#include <stdlib.h>
#include "color.h"

using namespace eku;

Row map[25];

//Class Window
void Window::showMap() {
    setCursorToPosition(0, 0);
    for (int i = 0; i < 25; i++) {
        getRowAt(i).printRow();
    }
}

void Window::writeToMap(std::string stringToInsert, int rowNum, int startIndex) {
    Row r = getRowAt(rowNum);
    r.insertStringIntoRow(stringToInsert, startIndex);
}

void Window::writeInRectangle(std::string stringToWrite, int rowNum, int startIndex) {
    if (startIndex != 0) {
        std::string topbar = "~";
        for (int i = 0; i < stringToWrite.length() + 2; i++) {
            topbar += ' ';
        }
        topbar += '^';
        getRowAt(rowNum - 1).insertStringIntoRow(topbar, startIndex - 1);
    }

    std::string toInsert = "~ ^" + stringToWrite + "~ ^";
    getRowAt(rowNum).insertStringIntoRow(toInsert, startIndex - 1);

    if (startIndex != 25) {
        std::string bottombar = "~";
        for (int i = 0; i < stringToWrite.length() + 2; i++) {
            bottombar += ' ';
        }
        bottombar += '^';
        getRowAt(rowNum + 1).insertStringIntoRow(bottombar, startIndex - 1);
    }
}

void Window::setCursorToPosition(int x, int y) {
    HANDLE hOut;
    COORD Position;
    hOut = GetStdHandle(STD_OUTPUT_HANDLE);

    Position.X = x;
    Position.Y = y;

    SetConsoleCursorPosition(hOut, Position);
}

void Window::resetMap() {
    for (Row row : map) {
        row.resetRow();
    }
}

Row Window::getRowAt(int index)
{
    return map[index];
}

//Class Row
const char WHITEBACKCOL = '~';
const char DEFCOL = '^';

int i = 0;

Row::Row() {
    row.resize(80, ' ');
    id = i;
    i++;
}

void Row::printRow() {
    for (int i = 0; i < row.size(); i++) {
        switch (row[i]) {
        case WHITEBACKCOL:
            setcolor(black, white);
        case DEFCOL:
            setcolor(white, black);
        default:
            std::cout << row[i];
        }
    }
}

void Row::resetRow() {
    row.resize(80);
    for (int i = 0; i < 80; i++) {
        row[i] = ' ';
    }
}

void Row::insertStringIntoRow(std::string toInsert, int startIndex) {
    int endIndex = (startIndex + toInsert.length());
    int stringPos = 0;
    for (int i = startIndex; i < endIndex; i++) {
        if (i < row.size() - 1) {
            row.at(i) = toInsert[stringPos];
        }
        else {
            row.push_back(toInsert[stringPos]);
        }
        stringPos++;
    }
}
Window defWindow;

void initColors() {
    concolinit();
    setcolor(white, black);
}

void setWindow(Window windowToSet) {
    defWindow = windowToSet;
}

Window getGameWindow() {
    return defWindow;
}

“Main.cpp”

#include <iostream>
#include "GameEngine.h"
#include "Window.h"

int main() {
    setWindow(Window());
    initColors();

    getGameWindow().writeInRectangle("Hia", 1, 10);

    getGameWindow().showMap();
}

每当我调用showMap() 时，我得到的只是一个空白控制台。它似乎只打印默认的空格映射，而不是我输入的文本。我还尝试仅使用 printRow() 打印我编辑的单行，但它们也只显示空格。 我能够在insertStringIntoRow() 方法中查看对向量行的更改，但是即使更改应该在那里，它们也没有显示在其他任何地方。几乎每次我访问它时都会创建我的Row 对象。我是 C++ 新手，因此不胜感激。提前致谢！

Answer 1

此功能使用不正确。警告标志是它是非常量的，但用作访问器。它返回向量的副本，而不是引用。

std::vector<char> getRow() {
    return row;
}

按照您的调用方式，您希望它就地修改行，但您实际上所做的只是修改行的副本，然后立即丢弃该副本。示例：

getRowAt(rowNum).insertStringIntoRow(toInsert, startIndex - 1);

解决此问题的简单方法是返回来自getRow() 的引用：

std::vector<char> & getRow() {
    return row;
}

要解决的另一件事是将其分配给临时变量而不是内联使用它。在这种情况下，您可以将临时作为参考：

Row & r = getRowAt(rowNum);
r.insertStringIntoRow(stringToInsert, startIndex);

---

实现这一点的正确方法是也提供一个常量版本，这样它仍然可以在调用者不想修改的常量对象上调用它。

const std::vector<char> & getRow() const {
    return row;
}

我只在此处添加了这个额外的部分，因为这是一种很好的做法，但您不应该在这个程序中这样做。它会使您的代码变得混乱，这些代码已经充满了更糟糕的做法 =)

Answer 2

发送对 writeToMap() 的引用而不是变量。 这将更改原始地图对象而不是复制变量。