fix: 사전거병 후 군주가 사라지는 경우에 대한 수정 #247

2023-12-29T12:11:15+09:00

jyp9 commented

2023-12-29 12:11:15 +09:00

사전거병 후 군주가 사라지는 경우에 대해 몇 가지 수정을 제안합니다.

사전거병 성공 시 삭턴을 리셋합니다.
사전거병 후 군주가 6턴만에 삭턴사하지 않게끔 수정하여, 거병자에 대한 책임을 강화합니다.
자율턴 로직을 일부 수정하여, 방랑군 소속 장수가 좀 더 의미 있는 턴을 수행하게끔 합니다.
종전에는 물자조달을 마지막 선택지로 삼았으나, 방랑군은 물자조달을 수행할 수 없습니다.
이에 마지막 옵션인 물자조달 수행에 실패하는 경우, 최종 옵션으로 견문을 선택하게끔 수정합니다.
(cf. 견문은 수행에 필요한 조건이 없습니다.)

사전거병 후 군주가 사라지는 경우에 대해 몇 가지 수정을 제안합니다. 1) 사전거병 성공 시 삭턴을 리셋합니다. 사전거병 후 군주가 6턴만에 삭턴사하지 않게끔 수정하여, 거병자에 대한 책임을 강화합니다. 2) 자율턴 로직을 일부 수정하여, 방랑군 소속 장수가 좀 더 의미 있는 턴을 수행하게끔 합니다. 종전에는 물자조달을 마지막 선택지로 삼았으나, 방랑군은 물자조달을 수행할 수 없습니다. 이에 마지막 옵션인 물자조달 수행에 실패하는 경우, 최종 옵션으로 견문을 선택하게끔 수정합니다. (cf. 견문은 수행에 필요한 조건이 없습니다.)

jyp9 added 1 commit 2023-12-29 12:11:16 +09:00

fix: 사전거병 후 군주가 사라지는 경우에 대한 수정 69f4d8e2c7

jyp9 reviewed 2023-12-29 12:20:19 +09:00

hwe/sammo/API/General/BuildNationCandidate.php Outdated

						
				@@ -90,6 +90,10 @@ class BuildNationCandidate extends \sammo\BaseAPI

				      return '거병을 실패했습니다.';

				    }

				    if ($general->getNPCType() < 2){

jyp9 commented

2023-12-29 12:20:19 +09:00

이 위치가 맞을까요? 거병에 성공한 시점이 이 시점인거 같아서 여기에 넣었습니다만..

Hide_D commented

2023-12-31 11:28:46 +09:00

API를 호출하는건 항상 유저이므로, NPCType 자체는 검사할 필요가 없는것 같고,
$general->setVar()를 했기 때문에, $general->applyDB($db)는 호출해줘야 할 것 같습니다.

API를 호출하는건 항상 유저이므로, NPCType 자체는 검사할 필요가 없는것 같고, `$general->setVar()`를 했기 때문에, `$general->applyDB($db)`는 호출해줘야 할 것 같습니다.

jyp9 marked this conversation as resolved

Hide_D requested changes 2023-12-31 11:37:24 +09:00

hwe/sammo/GeneralAI.php Outdated

						
				@@ -3442,3 +3441,1 @@

				            return buildGeneralCommandClass('che_물자조달', $this->general, $this->env);

				        }

				        return $cmd;

				        if ($cmd->hasFullConditionMet()) return $cmd;

Hide_D commented

2023-12-31 11:31:03 +09:00

이번 코드 한정으로 제 스타일은

if (!$cmd->hasFullConditionMet()) {
   $cmd = buildGeneralCommandClass(딴거);
}
if (!$cmd->hasFullConditionMet()) {
   $cmd = buildGeneralCommandClass(또딴거);
}

에 가깝긴 하겠네요.

물론 조기 탈출은 좋아합니다:)

이번 코드 한정으로 제 스타일은 ``` if (!$cmd->hasFullConditionMet()) { $cmd = buildGeneralCommandClass(딴거); } if (!$cmd->hasFullConditionMet()) { $cmd = buildGeneralCommandClass(또딴거); } ``` 에 가깝긴 하겠네요. 물론 조기 탈출은 좋아합니다:)

❤️ 1

jyp9 commented

2024-01-26 22:16:08 +09:00