devsam/core
3
6
Fork 1
Code Issues 60 Pull Requests Releases 1 Wiki Activity

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

Merged
Hide_D merged 3 commits from jyp9/core:nomore_wandering_lord_run into devel 2024-02-17 00:01:35 +09:00
Conversation 3 Commits 3 Files Changed 2
+3 -3
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2f00aee947 - Show all commits
hwe/sammo/GeneralAI.php
+3 -3
View File
@@ -3440,9 +3440,9 @@ class GeneralAI
$cmd = buildGeneralCommandClass($this->rng->choice($candidate), $this->general, $this->env);
if (!$cmd->hasFullConditionMet()) {
jyp9 marked this conversation as resolved Outdated
Hide_D commented 2023-12-31 11:31:03 +09:00
Outdated
Review
Copy Link
Copy Source

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

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
Outdated
Review
Copy Link
Copy Source

아 이거 맨 처음 $cmd가 잘 되어 있으면 hasFullConditionMet()을 두 번이나 검사하게 되긴 하지만.. 그래도 스타일 맞추는 게 좋은게 좋은거지 싶어서 일단 따라가봄미다..

아 이거 맨 처음 $cmd가 잘 되어 있으면 hasFullConditionMet()을 두 번이나 검사하게 되긴 하지만.. 그래도 스타일 맞추는 게 좋은게 좋은거지 싶어서 일단 따라가봄미다..
jyp9 commented 2024-01-26 22:17:29 +09:00
Outdated
Review
Copy Link
Copy Source

는 아무래도 아닌 거 같아서 약간 더 수정했습니다.

는 아무래도 아닌 거 같아서 약간 더 수정했습니다.
$cmd = buildGeneralCommandClass('che_물자조달', $this->general, $this->env);
}
if (!$cmd->hasFullConditionMet()) {
$cmd = buildGeneralCommandClass('che_견문', $this->general, $this->env);
if (!$cmd->hasFullConditionMet()) {
$cmd = buildGeneralCommandClass('che_견문', $this->general, $this->env);
}
}
return $cmd;