<div id="readability-page-1" class="page"><div>
                <p><a href="/m/category/%EC%A3%BC%EB%8B%88%EC%96%B4%20%EA%B0%9C%EB%B0%9C%EC%9E%90%EB%A1%9C%20%EC%9D%BC%ED%95%98%EB%A9%B0%20%EB%8A%90%EB%82%80%20%EC%A0%90"><span>주니어 개발자로 일하며 느낀 점</span></a></p><h3>[버그] 231125 야스오 버그 영상을 보며 든 생각</h3>
              </div><p><cite>Berno</cite>
                <span> 2023. 11. 26. 10:33</span>
              </p><div>
                                <h3 data-ke-size="size23"><b>0. 이 글을 작성하는 이유</b></h3>
<p data-ke-size="size16">지방에 내려갔다가 서울로 올라오는 버스에서 야스오 버그 영상을 봤는데 이런 메커니즘이 빠져있어서 발생한 버그가 아닐까 하는 게 생각나서 가볍게 정리해 보기 위함(실은 직업병인 듯)</p>

<h3 data-ke-size="size23"><b>1. 버그</b></h3>
<p data-ke-size="size16"><a href="https://youtu.be/GCuyT0C7bBM?feature=shared">https://youtu.be/GCuyT0C7bBM?feature=shared</a></p>
<figure data-ke-type="video" data-ke-style="alignCenter" data-video-host="youtube" data-video-url="https://www.youtube.com/watch?v=GCuyT0C7bBM" data-video-thumbnail="https://scrap.kakaocdn.net/dn/bSl6wI/hyUFd94SfR/uZQU4iUqYID87fAkAUnsB0/img.jpg?width=1280&amp;height=720&amp;face=0_0_1280_720" data-video-width="860" data-video-height="484" data-video-origin-width="860" data-video-origin-height="484" data-ke-mobilestyle="widthContent" data-video-title="롤 야스오의 궁극기 쿨타임이 5초가 되면 생기는 일 ㅋㅋ" data-original-url=""><iframe src="https://www.youtube.com/embed/GCuyT0C7bBM" width="860" height="484" frameborder="" allowfullscreen=""></iframe>

<img src="https://scrap.kakaocdn.net/dn/bSl6wI/hyUFd94SfR/uZQU4iUqYID87fAkAUnsB0/img.jpg?width=1280&amp;height=720&amp;face=0_0_1280_720"></figure>
<blockquote data-ke-style="style3">요약 :&nbsp;<br>신규 아이템을 사용하면(원래는 가지고 있으면이지만 개발 편의를 위해 사용으로 바꿔보았다. 딱 한번 사용 가능하다고 해보자.) 궁극기 쿨타임이 줄어드는데 문제는 사용하고 상점에 팔고 다시 사고 사용하고 상점에 팔고 사고.. 반복하는 만큼 쿨타임이 계속 줄어드는 것</blockquote>

<h3 data-ke-size="size23"><b>2. 정상 동작은?</b></h3>
<p data-ke-size="size16">실은 팔고 나면 다시 원래대로 쿨타임이 적용되어야 하는데 그러지 못함</p>

<h3 data-ke-size="size23"><b>3. 왠지?</b></h3>
<p data-ke-size="size16">원래대로 돌려주는 로직이 빠져있는 느낌이다.</p>
<p data-ke-size="size16">분명 실제 코드는 더 복잡하고 정교하고 이쁘게 작성되어 있겠지만 사건의 재구성을 위해 대충 작성해 보았다.</p>
<pre id="code_1700962182003" data-ke-language="kotlin" data-ke-type="codeblock"><code>class Champion(
    val name : String,
    var hp : Int,
    var mp : Int,
    var skillQ : Skill,
    var skillW : Skill,
    var skillE : Skill,
    var skillR : UltiSkill
) {
    var money = 0.0
    var itemList = mutableMapOf&lt;Int, Item?&gt;()

    fun sellItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null){
            money += item.price / 2
            itemList[itemIdx] = null
            println("아이템 판매")
        }
    }

    fun buyItem(item: Item){
        if(itemList.size &gt;= 6){
            throw IllegalStateException()
        }

        for(i : Int in 0..5){
            if(itemList[i] == null){
                itemList[i] = item
                money -= item.price
                println("아이템 구매")
                return
            }
        }
    }

    fun useItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null &amp;&amp; item.canUseItem){
            item.activeItem(this)
        }
    }
}

class Skill(val name: String, val description: String, var cooldownTime : Double)

class UltiSkill(val name: String, val description: String, var cooldownTime : Double){
    var defaultCooldownTime = cooldownTime
}


abstract class Item(
    val name : String,
    val description : String,
    val price : Int,
    val canUseItem : Boolean = false
){
    abstract fun activeItem(champion : Champion)

    abstract fun sellAction(champion: Champion)
}

class NewItem375 : Item(name = "버그아이템", "사용 시 궁극기 쿨타임이 줄어듭니다.", 3300, true){
    var increaseAtk = 55
    var increaseAtkSpeed = 0.20
    var increaseHp = 500
    var isUsed = false
   override fun activeItem(champion: Champion){
       if(!isUsed) {
           champion.skillR.cooldownTime = champion.skillR.cooldownTime * 0.85
           println("아이템 사용")
           println("야스오 궁극기 쿨타임 : ${champion.skillR.cooldownTime}")
           isUsed = true
       }
    }

    override fun sellAction(champion: Champion) {
        champion.skillR.cooldownTime = champion.skillR.defaultCooldownTime
    }
}</code></pre>
<pre id="code_1700962194065" data-ke-language="kotlin" data-ke-type="codeblock"><code>class Game {
    fun runGame() {
        val yasuo = createYasuo()
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
    }

    fun createYasuo() : Champion = Champion(
        name = "야스오",
        hp = 550,
        mp = 0,
        skillQ = Skill("Q스킬", "대충 Q스킬", 10.0),
        skillW = Skill("Q스킬", "대충 Q스킬", 10.0),
        skillE = Skill(
            "Q스킬", "대충 Q스킬", 10.0
        ),
        skillR = UltiSkill("궁극기", "궁극기 가즈아", 50.0)
    )
}</code></pre>
<p data-ke-size="size16">갑자기 작성하려니 코드가 좀 이상하다. 대충 넘어가자. data class로 뺄 수 있는 부분도 있고 그러지만 대충 넘어가는 게 우리 서로 윈윈인 것 같다.</p>
<figure data-ke-mobilestyle="widthOrigin" data-origin-width="1280" data-origin-height="881">
    <span data-url="https://blog.kakaocdn.net/dn/co9yvg/btsATAyiBIY/kIX4LUa2BTX0k8yojgp9gk/img.png" data-lightbox="lightbox">
        <img src="https://blog.kakaocdn.net/dn/co9yvg/btsATAyiBIY/kIX4LUa2BTX0k8yojgp9gk/img.png" onerror="this.onerror=null; this.src='//t1.daumcdn.net/tistory_admin/static/images/no-image-v1.png'; this.srcset='//t1.daumcdn.net/tistory_admin/static/images/no-image-v1.png';" width="739" height="509" data-origin-width="1280" data-origin-height="881">
    </span>
    
</figure>
<p data-ke-size="size16">쿨타임이 계속 줄어드는 기적을 보여준다.</p>
<p data-ke-size="size16">이유는 간단하다. 아이템을 팔았을 때 이 쿨타임이 원상복귀가 되어야 한다.</p>

<p data-ke-size="size16">저기 sellAction이라는 함수를 임의로 만들었는데 아이템을 팔았을 때 얻은 능력치, 효과들을 초기화시키기 위해 만들었다. 이 함수를 sellItem함수에 넣어보자.</p>
<pre id="code_1700962291482" data-ke-language="kotlin" data-ke-type="codeblock"><code>fun sellItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null){
            money += item.price / 2
            itemList[itemIdx] = null
            item.sellAction(this)
            println("아이템 판매")
        }
    }</code></pre>
<figure data-ke-mobilestyle="widthOrigin" data-origin-width="1280" data-origin-height="881">
    <span data-url="https://blog.kakaocdn.net/dn/FMwJF/btsAQyuXvaD/LQGQC19VP6CQfWsjvtkEIk/img.png" data-lightbox="lightbox">
        <img src="https://blog.kakaocdn.net/dn/FMwJF/btsAQyuXvaD/LQGQC19VP6CQfWsjvtkEIk/img.png" onerror="this.onerror=null; this.src='//t1.daumcdn.net/tistory_admin/static/images/no-image-v1.png'; this.srcset='//t1.daumcdn.net/tistory_admin/static/images/no-image-v1.png';" width="787" height="542" data-origin-width="1280" data-origin-height="881">
    </span>
    
</figure>

<h3 data-ke-size="size23"><b>4. 여기서 느낀 점은?</b></h3>
<p data-ke-size="size16">버그도 버그지만 생각보다 롤 챔피언, 아이템, 스킬, 능력치 등의 아키텍처 구성이 이쁘게 하려면 어떻게 해야 할지 머리가 좀 아프기 시작했다. 이건 좀 봐봐야 할 것 같다.&nbsp;</p>
            
            
                </div></div>

흔한 컴공출신 개발자

bernoyoun

0. 이 글을 작성하는 이유
지방에 내려갔다가 서울로 올라오는 버스에서 야스오 버그 영상을 봤는데 이런 메커니즘이 빠져있어서 발생한 버그가 아닐까 하는 게 생각나서 가볍게 정리해 보기 위함(실은 직업병인 듯)
 
1. 버그
https://youtu.be/GCuyT0C7bBM?feature=shared
요약 : 
신규 아이템을 사용하면(원래는 가지고 있으면이지만 개발 편의를 위해 사용으로 바꿔보았다. 딱 한번 사용 가능하다고 해보자.) 궁극기 쿨타임이 줄어드는데 문제는 사용하고 상점에 팔고 다시 사고 사용하고 상점에 팔고 사고.. 반복하는 만큼 쿨타임이 계속 줄어드는 것
2. 정상 동작은?
실은 팔고 나면 다시 원래대로 쿨타임이 적용되어야 하는데 그러지 못함
 
3. 왠지?
원래대로 돌려주는 로직이 빠져있는 느낌이다.
분명 실제 코드는 더 복잡하고 정교하고 이쁘게 작성되어 있겠지만 사건의 재구성을 위해 대충 작성해 보았다.
class Champion(
    val name : String,
    var hp : Int,
    var mp : Int,
    var skillQ : Skill,
    var skillW : Skill,
    var skillE : Skill,
    var skillR : UltiSkill
) {
    var money = 0.0
    var itemList = mutableMapOf<Int, Item?>()
    fun sellItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null){
            money += item.price / 2
            itemList[itemIdx] = null
            println("아이템 판매")
        }
    }
    fun buyItem(item: Item){
        if(itemList.size >= 6){
            throw IllegalStateException()
        }
        for(i : Int in 0..5){
            if(itemList[i] == null){
                itemList[i] = item
                money -= item.price
                println("아이템 구매")
                return
            }
        }
    }
    fun useItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null && item.canUseItem){
            item.activeItem(this)
        }
    }
}
class Skill(val name: String, val description: String, var cooldownTime : Double)
class UltiSkill(val name: String, val description: String, var cooldownTime : Double){
    var defaultCooldownTime = cooldownTime
}
abstract class Item(
    val name : String,
    val description : String,
    val price : Int,
    val canUseItem : Boolean = false
){
    abstract fun activeItem(champion : Champion)
    abstract fun sellAction(champion: Champion)
}
class NewItem375 : Item(name = "버그아이템", "사용 시 궁극기 쿨타임이 줄어듭니다.", 3300, true){
    var increaseAtk = 55
    var increaseAtkSpeed = 0.20
    var increaseHp = 500
    var isUsed = false
   override fun activeItem(champion: Champion){
       if(!isUsed) {
           champion.skillR.cooldownTime = champion.skillR.cooldownTime * 0.85
           println("아이템 사용")
           println("야스오 궁극기 쿨타임 : ${champion.skillR.cooldownTime}")
           isUsed = true
       }
    }
    override fun sellAction(champion: Champion) {
        champion.skillR.cooldownTime = champion.skillR.defaultCooldownTime
    }
}
class Game {
    fun runGame() {
        val yasuo = createYasuo()
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
        yasuo.buyItem(NewItem375())
        yasuo.useItem(0)
        yasuo.sellItem(0)
    }
    fun createYasuo() : Champion = Champion(
        name = "야스오",
        hp = 550,
        mp = 0,
        skillQ = Skill("Q스킬", "대충 Q스킬", 10.0),
        skillW = Skill("Q스킬", "대충 Q스킬", 10.0),
        skillE = Skill(
            "Q스킬", "대충 Q스킬", 10.0
        ),
        skillR = UltiSkill("궁극기", "궁극기 가즈아", 50.0)
    )
}
갑자기 작성하려니 코드가 좀 이상하다. 대충 넘어가자. data class로 뺄 수 있는 부분도 있고 그러지만 대충 넘어가는 게 우리 서로 윈윈인 것 같다.
쿨타임이 계속 줄어드는 기적을 보여준다.
이유는 간단하다. 아이템을 팔았을 때 이 쿨타임이 원상복귀가 되어야 한다.
 
저기 sellAction이라는 함수를 임의로 만들었는데 아이템을 팔았을 때 얻은 능력치, 효과들을 초기화시키기 위해 만들었다. 이 함수를 sellItem함수에 넣어보자.
fun sellItem(itemIdx : Int){
        val item = itemList[itemIdx]
        if(item != null){
            money += item.price / 2
            itemList[itemIdx] = null
            item.sellAction(this)
            println("아이템 판매")
        }
    }
 
4. 여기서 느낀 점은?
버그도 버그지만 생각보다 롤 챔피언, 아이템, 스킬, 능력치 등의 아키텍처 구성이 이쁘게 하려면 어떻게 해야 할지 머리가 좀 아프기 시작했다. 이건 좀 봐봐야 할 것 같다.

[버그] 231125 야스오 버그 영상을 보며 든 생각

댓글