Sigill 신호가있는 이유는 무엇입니까?

Question

나는 두명의 전투기 사이의 싸움에서 승자의 이름을 돌려줘야하는 문제가있다. 다음과 같이 전투기의 클래스는 다음과 같습니다

class Fighter 
{ 
private: 
std::string name; 

int health; 

int damagePerAttack; 

public: 
Fighter(std::string name, int health, int damagePerAttack) 
{ 
    this->name = name; 
    this->health = health; 
    this->damagePerAttack = damagePerAttack; 
} 

~Fighter() { }; 

std::string getName() 
{ 
    return name; 
} 

int getHealth() 
{ 
    return health; 
} 

int getDamagePerAttack() 
{ 
    return damagePerAttack; 
} 

void setHealth(int value) 
{ 
    health = value; 
} 
}; 

내가 승자의 이름을 반환해야하는 함수를 썼다.

std::string declareWinner(Fighter* fighter1, Fighter* fighter2, 
    std::string firstAttacker) 
    { 
    // Your code goes here. Have fun! 
    if(firstAttacker==fighter1->getName()) 
    { 
     while(fighter1->getHealth()!=0&&fighter2->getHealth()!=0) 
     { 
     fighter2->setHealth(fighter2->getHealth()-fighter1->getDamagePerAttack()); 
     if(fighter2->getHealth()<=0) 
     { 
      return fighter1->getName(); 
     } 
     fighter1->setHealth(fighter1->getHealth()-fighter2->getDamagePerAttack()); 
     if(fighter1->getHealth()<=0) 
     { 
      return fighter2->getName(); 
     } 
     } 
    } 
    else if(firstAttacker==fighter2->getName()) 
    { 
     while(fighter1->getHealth()!=0&&fighter2->getHealth()!=0) 
     { 
     fighter1->setHealth(fighter1->getHealth()-fighter2->getDamagePerAttack()); 
     if(fighter1->getHealth()<=0) 
     { 
      return fighter2->getName(); 
     } 
     fighter2->setHealth(fighter2->getHealth()-fighter1->getDamagePerAttack()); 
     if(fighter2->getHealth()<=0) 
     { 
      return fighter1->getName(); 
     } 
     } 
    } 
    } 

이것은 내 모든 필요를 충족 시키지만, 신호가 울리고, 내가 뭘 잘못했는지 모른다. 어떻게 처리해야합니까?

Answer 1

일부 조건에서는 함수가 끝까지 실행되고 값을 반환하지 않고 종료 될 수 있으며 스택을 손상시키고 SIGILL로 이어질 수 있습니다. 안전한 측정 값으로 함수의 끝 부분에 return 문을 추가 할 수 있습니다.

std::string declareWinner(Fighter* fighter1, Fighter* fighter2, 
    std::string firstAttacker) 
{ 
    // Your code goes here. Have fun! 
     if(firstAttacker==fighter1->getName()) 
     { 
      while(fighter1->getHealth()!=0&&fighter2->getHealth()!=0) 
      { 
      fighter2->setHealth(fighter2->getHealth()-fighter1->getDamagePerAttack()); 
      if(fighter2->getHealth()<=0) 
      { 
       return fighter1->getName(); 
      } 
      fighter1->setHealth(fighter1->getHealth()-fighter2->getDamagePerAttack()); 
      if(fighter1->getHealth()<=0) 
      { 
       return fighter2->getName(); 
      } 
      } 
     } 
     else if(firstAttacker==fighter2->getName()) 
     { 
      while(fighter1->getHealth()!=0&&fighter2->getHealth()!=0) 
      { 
      fighter1->setHealth(fighter1->getHealth()-fighter2->getDamagePerAttack()); 
      if(fighter1->getHealth()<=0) 
      { 
       return fighter2->getName(); 
      } 
      fighter2->setHealth(fighter2->getHealth()-fighter1->getDamagePerAttack()); 
      if(fighter2->getHealth()<=0) 
      { 
       return fighter1->getName(); 
      } 
      } 
     } 
     return "No winner"; <= Add before exiting function 
} 

코드에 중복 및 가능한 논리 오류가 있음을 확인했습니다. 나는 (함수 서명을 변경하지 않고) 이런 식으로 재 작성합니다 :

std::string declareWinner(Fighter* fighter1, Fighter* fighter2, 
     std::string firstAttacker) 
{ 
    Fighter *first; 
    Fighter *second; 
    if(firstAttacker == fighter1->getName()) { 
     first = fighter2; 
     second = fighter1; 
    } else if (firstAttacker == fighter2->getName()) { 
     first = fighter1; 
     second = fighter2; 
    } else { 
     // Bad call parameters 
     return "Bad call"; // Throw exception maybe? 
    } 
    // Simulating fighting 
    do { 
     std::swap(second,first); 
     second->setHealth(second->getHealth() - first->getDamagePerAttack()); 
    } while (second->getHealth() > 0); 
    return first->getName(); 

}