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



我正在制作一个主机游戏,我正在制作地图。地图是一组矢量。矢量包含我要打印到控制台的字符。我的代码:

"Window.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();

"Window.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++的新手,所以非常感谢您的帮助。提前感谢!

此函数使用不正确。警告信号是,它是非常数,但用作访问器。它返回向量的副本,而不是引用。

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版本,这样它仍然可以在调用方不想修改的const对象上调用。

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

我只是在这里多放了一点,因为这是一个很好的练习,但你不应该在这个程序中这样做。它会使你的代码变得混乱,而你的代码已经充满了更糟糕的做法=)

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

相关内容

  • 没有找到相关文章

最新更新