이 특정 기능을 실행 한 후 내 프로그램이 중단되는 이유는 무엇입니까?

Question

프로그램에서이 기능을 실행하기위한 명령을 입력 할 때마다 실행되고 다음과 같이 충돌합니다.

"응용 프로그램이 비정상적으로 종료되도록 런타임을 요청했습니다."

왜 이렇게합니까?

void showInventory(player& obj) { 
    std::cout << "\nINVENTORY:\n"; 
    for(int i = 0; i < 20; i++) { 
     std::cout << obj.getItem(i); 
     i++; 
     std::cout << "\t\t\t" << obj.getItem(i) << "\n"; 
    } 
} 

std::string getItem(int i) { 
     return inventory[i]; 
    } 

Answer 1

:

std::string toDo(player& obj) //BY KEATON 
{ 
    std::string commands[5] = // This is the valid list of commands. 
    {"help", "inv"}; 

    std::string ans; 
    std::cout << "\nWhat do you wish to do?\n>> "; 
    std::cin >> ans; 

    if(ans == commands[0]) { 
     helpMenu(); 
     return NULL; 
    } 
    else if(ans == commands[1]) { 
     showInventory(obj); 
     return NULL; 
    } 
} 

개

요구가되게합니다 :

std::string toDo(player& obj) //BY KEATON 
{ 
    std::string commands[5] = // This is the valid list of commands. 
    {"help", "inv"}; 

    std::string ans; 
    std::cout << "\nWhat do you wish to do?\n>> "; 
    std::cin >> ans; 

    if(ans == commands[0]) { 
     helpMenu(); 
     return ""; 
    } 
    else if(ans == commands[1]) { 
     showInventory(obj); 
     return "";   // Needs to be '""' 
    } 
} 

신용이 스타크 프로토 타입!

Answer 2

전 = 19, 당신은 내가 20이되는 후 배열의 마지막 항목을 얻을 경계 예외

Answer 3

for(int i = 0; i < 20; i++) { 
    std::cout << obj.getItem(i); 

이 아주 제대로되지에서 야기한다 또 다른의 getItem이있는 경우. 마법의 숫자를 사용하지 마십시오. 대신 이런 식으로 (당신에 의해 구현 될 것입니다) 20 사용 INT listSize = obj.ListSize()

listSize = obj.ListSize(); 
    for(int i = 0; i <listSize ; i++) { 
     std::cout << obj.getItem(i); 

의 당신은 당신이 범위를 벗어난되지 않도록 할 것이다.

class player{ 
public: 
//--whatever it defines 
int ListSize() 
{ 
    return (sizeof(inventory)/sizeof(inventory[0])); 
} 
}; 

그런 다음

를 사용

void showInventory(player& obj) { // By Johnny :D 
    std::cout << "\nINVENTORY:\n"; 
    int listSize = obj.ListSize()/2; //if you are sure that is odd number 
    for(int i = 0; i < listSize; ++i) { 
     std::cout << obj.getItem(i); 
     i++; 
     std::cout << "\t\t\t" + obj.getItem(i) + "\n"; 
    } 
} 

Answer 4

이 함수를 작성 : 또한 당신이 (내가 왜 이유를 얻을하지 않습니다) 당신이 모시 한 루프에서이 개 항목을 인쇄하려면

이 코드에서

void showInventory(player& obj) { // By Johnny :D 
    int length = obj.ListSize(); 
    std::cout << "\nINVENTORY:\n"; 
    for(int i = 0; i < length; i++) { 
     std::cout << obj.getItem(i); 
     i++; 
     std::cout << "\t\t\t" << obj.getItem(i) << "\n"; 
    } 
}