PG0001.Question1

[jack81722]

Introduction

寫出一個計算爆擊的函式

Answer

解答

需要注意的是爆擊率如果等於0，理論上不會有任何可能發生爆擊，
但在產出rand的時候可能有0的值，所以題目假設區間為[0, 1) (不等式0 <= rand < x)，
因此沒有寫rand <= critRate

int CriticalAtk(int atk, float critRate, float rand)
{
    if (atk < 0)
        return 0;
    if (rand < critRate)
        return (int)(atk * 1.5f);
    return atk;
}

[tommyboys0107]

Cliff Lee CL (程式、專案管理)

我的答案

public class Question1 
{
    const float CRITICAL_MULTIPLIER = 1.5f;
    
    /// <summary>
    /// Calculate the atk if critical occured or not
    /// </summary>
    /// <param name="atk">The attack point</param>
    /// <param name="critRate">The critical rate</param>
    /// <param name="rand">The random value</param>
    /// <returns>Modified attack point</returns>
    public int CriticalAtk(int atk, float critRate, float rand)
    {
        if (atk <= 0)
            return 0;
        if (critRate > rand)
            return (int)(atk * CRITICAL_MULTIPLIER);
        return atk;
    }
}

[A24575255]

JIA（程式新手）

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
{
    bool isAtkLessThenZero = atk <= 0;
    if (isAtkLessThenZero) return 0;

    const float criticalHitMultiplier = 1.5f;
    bool isCritRateGreaterThanZero = critRate > 0;
    bool isCritRate = rand <= critRate;
    if (isCritRate && isCritRateGreaterThanZero) return (int)(atk * criticalHitMultiplier);

    return atk;
}

[YouSyuan]

Syuan(新人)

我的答案

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk <= 0) return 0;
            return critRate > rand ? (int)(atk * 1.5) : atk;
        }

[BoLingCan]

罐頭（新人）

我的答案ヾ(•ω•`)o

public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk >= 0)
            {
                if (rand < critRate)
                    return (int)(atk * 1.5f);
                return atk;
            }

            return 0;
        }

連假結束了，我被爆擊了இ௰இ

[crazyjimmy0623]

吉米（不知道要學什麼但是喜歡創作的新人）

我的答案在這

 public int CriticalAtk(int atk, float critRate, float rand)
{
    if(atk > 0)
        return critRate > rand ? (int)(atk * 1.5) : atk;
    else
        return 0;       
}

我.需要..更多的..知識!

[TWEdward]

TWEdward(新人/程式)

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
        {
            //throw new NotImplementedException("code something here ...");
            if(atk < 0)
            {
                return 0;
            }

            if(rand < critRate)  //
            {
                return (int)(atk * 1.5);
            }
            else
            {
                return atk;
            }
        }

第一次碰到單元測試，測試時過不了程式碼再重寫一次，第二次過不了再再重寫一次，第三次過不了再再再重寫一次，終於給它過了

[jack81722]

測試不是絕對的，因為他只測試了工程師對邏輯的理解
如果沒有相關領域的知識，測試程式碼就會有問題
當然還有很多零零總總的問題，這些會在測試的課堂上再說明
敬請期待

[riveRobot]

riverRobot (新人/企劃/程式)

我的答案

public class Question1 
{
	/// <summary>
	/// Calculate the atk if critical occured or not
	/// </summary>
	/// <param name="atk">The attack point</param>
	/// <param name="critRate">The critical rate</param>
	/// <param name="rand">The random value</param>
	/// <returns>Modified attack point</returns>
	public int CriticalAtk(int atk, float critRate, float rand)
	{
		if(atk < 0)
			return 0;
		if(critRate > rand)
			return (int)(atk * 1.75f);
		return atk;
	}
}

單元測試結果

課程/作業心得

好耶爆擊

[louise-bwsx]

小4 ( 程式 )

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            atk = atk <= 0 ? 0 : atk;
            bool isCrit = critRate > rand;
            atk = isCrit ? (int)(atk * 1.5f) : atk;
            return atk;
        }

[jefflin0424]

肥羊(程式)

我的答案

public class Question1
    {
        const float critRateMultiplier = 1.5f;
        /// <summary>
        /// Calculate the atk if critical occured or not
        /// </summary>
        /// <param name="atk">The attack point</param>
        /// <param name="critRate">The critical rate</param>
        /// <param name="rand">The random value</param>
        /// <returns>Modified attack point</returns>
        public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk <= 0) return 0;

            if (critRate > rand)
            {
                return (int)(atk * critRateMultiplier);
            }
            else
            {
                return atk;
            }
        }
    }

[Suncacao]

Suncacao(程式)

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
        {
            if(atk < 0){
                return 0;
            }else if(critRate > rand){
                return (int)(atk * 1.5f);
            }
            return atk;
        }

[angus945]

Angus 小呈 （企劃程式技美圖形工程師什麼的，窩都想當）

答案

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            float damageRate = 1;

            damageRate *= (rand < critRate) ? 1.5f : 1;

            int modifiedATK = Mathf.RoundToInt(atk * damageRate);

            return Mathf.Max(0, modifiedATK);
        }

[OLDShaw1022]

老蕭OLDShaw（程式）

我的答案

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            if(atk < 0) return 0;

            if(rand < critRate)
            {
                atk = (int)(atk * 1.5f);
            }
            return atk;
        }

測試截圖

[ISnoweve]

Snoweve (程式)

我的答案

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk <= 0) return 0;
            return critRate > rand ? (int)(atk * 1.5) : atk;
        }

<<反饋>>
'w'

[jack81722]

^w^

[s32436515]

肉鬆（程式 / 音樂）

我的答案

　　public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk <= 0)
                return 0;
            else if (rand < critRate)
                return (int)(atk * 1.5f);
            return atk;
        }

勞贖

[jack81722]

什麼事？

[ChiaBurn]

ChiaBurn(程式 | 企劃)

作業程式碼

public int CriticalAtk(int atk, float critRate, float rand)
        {
            // 如果atk小於0，回傳0
            if (atk <= 0)
            {
                return 0;
            }

            // 若爆擊率大於隨機值，代表爆擊成功。(爆擊率越高越容易發生爆擊效果)
            bool isCriticalSuccess = critRate > rand;
            if (isCriticalSuccess)
            {
                // 爆擊時攻擊力會變為基礎攻擊力的1.5倍，回傳結果指定為整數。
                return (int)(atk * 1.5);
            }
            return atk;
        }

程式碼測試結果

想法與反饋

之前寫作業時我是利用LinQPad執行程式，並另外寫了for迴圈來產生隨機input後call CriticalAtk進行測試：

    // 進行5次測試
    for(int i = 0; i <5; i++)
    {
        Random randGenerator = new Random();
        // 基礎攻擊力：隨機取-1~5之間的整數
        int atk = randGenerator.Next(-5, 10);
        // 爆擊率：介於0~1之間的浮點數(0~100%)
        float critRate = (float)randGenerator.Next(0, 100) / 100;
        // 隨機值：介於0~1之間的浮點數(0~100%)。
        float rand = (float)randGenerator.Next(0, 100) / 100;
        $"atk = {atk}, critRate = {critRate}, rand = {rand}，爆擊攻擊力計算結果為：".Dump();
        CriticalAtk(atk, critRate, rand).Dump();    
    }

這樣的測試方式有幾個不方便的地方：

1. 由於input為隨機產生，我不一定能測到當下我想測的特定情境。
   例如：因應需求修改的關係，程式裡atk <= 0情境下return值要改成別的數字，我修改完後跑測試時在運氣極差的情況下有可能跑一小時都遇不到input剛好atk <= 0的情況。
2. 這只會把數次測試的input、output印出來，不會幫我驗證結果。
   程式跑完後我得用自己貧弱的小眼睛盯Log、再用貧乏的小腦袋比對題目來檢查結果正不正確。

相比之下寫unit test來跑真的方便許多！

[jack81722]

有種測試叫作模糊測試（fuzzy test)
可以用來測試一些數值範圍內的隨機狀況
但僅僅是檢查多數情況是否正確

在單元測試的情況下
我們只是假設這種情況出現時
程式怎麼反應
所以可以選擇一些極端狀況來驗證

用log和人眼來檢查是不太可能做得完整
但我們能盡可能推理哪些狀況應該測試
其實就是一個“驗算”的過程

[jane901201]

Tina( 程式 )

我的答案

using System;

namespace PG0001.Questions
{
    public class Question1 
    {
        /// <summary>
        /// Calculate the atk if critical occured or not
        /// </summary>
        /// <param name="atk">The attack point</param>
        /// <param name="critRate">The critical rate</param>
        /// <param name="rand">The random value</param>
        /// <returns>Modified attack point</returns>
        public int CriticalAtk(int atk, float critRate, float rand)
        {
            int result = 0;
            
            if(atk < 0){
                Console.WriteLine("Wrong atk value.");

                return result;
            }

            result = atk;

            if(critRate > rand){
                Console.WriteLine("Have cirtical.");
                result = (int) (result * 1.5f);
            }
            else
                Console.WriteLine("Don't have cirtical.");
            
            return result;        
        }

    }
}

[chopin890612]

阿邦(企劃)

我的解答

public int CriticalAtk(int atk, float critRate, float rand)
{
    if(atk <= 0) return 0;
    if(rand < critRate) return (int)(atk*1.5f);
    else return atk;
}

謝謝老師指教!

[TMLyuiga]

芋圓（美術/新人）

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
		{
			if(atk>0)
			{
				if(rand>=critRate)	//隨機值高於等於暴擊率=>沒有暴擊
				{
					atk=atk*1;
				}else if(rand<critRate)	 //隨機值小於暴擊率=>判定暴擊
				{
					atk=(int)(atk*1.5);
				}
			}
			else if(atk<=0)
			{
				return 0;
			}
			return atk;			 
		}

想法與反饋

第一次測試是使用上次寫過的答案 但在NoCrit的單元測試那出了點問題 後來看了測試文件的程式碼後 更改了隨機值與暴率間的判定關係就通過測試了 不過有點好奇為甚麼兩者相等時是要判定成沒有暴擊?

[jack81722]

你爆擊率如果是0
且rand又剛好出現0的話
就會出現爆擊
看起來是不合理的
之後講到rand的時候
我再特別講一下這件事情好了

[qs4635463]

JSheng（程式）

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
{
	float value = atk;
	
	if(rand < critRate)
		value *= 1.5f;
	
	return (int)Mathf.Max(0, value);
}

[fillInlovewithmyself3]

鮪魚腹排(新人)

我的答案

反饋

快鼠了，感覺我好像前面打了一些多餘的程式OWQ

[jack81722]

也沒什麼問題啦
錯誤處理的部分本來就還沒講到
倒不如說你想得算細心了

[vre343434]

七七(程式)

我的答案

public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (atk < 0)
                return 0;
            // if (rand >= critRate)
            if (rand < critRate)
            {
                return atk = (int)(atk * 1.5);
            }
            return atk;
        }

回頭補作業 ‹‹( ˙▿˙　)/››‹‹(　˙▿˙ )/››
第一次測沒搞懂rand、critRate在題目間的關係所以偷看了解答 ( ˘･з･)

[DustEvan]

Dust（程式）

回答

public int CriticalAtk(int atk, float critRate, float rand)
{
            rand = Math.Clamp(rand, 0, 1.0f);
            critRate = Math.Clamp(critRate, 0, 1.0f);
            if (rand < critRate)
            {
                atk = (int)(atk*1.5f);
            }
            if (atk <= 0)
            {
                atk = 0;
            }
            return atk;
}

疑問：
不太清楚為什麼，if (rand < critRate)這邊如果加等號的話會報錯呢...

[jack81722]

你爆擊率如果是0
且rand又剛好出現0的話
就會出現爆擊
看起來是不合理的
之後講到rand的時候
我再特別講一下這件事情好了

[DustEvan]

感謝先生的解答，我了解了。

[SoonYu97]

小鲨 (程式)

答案

public class Question1 
{
    /// <summary>
    /// Calculate the atk if critical occured or not
    /// </summary>
    /// <param name="atk">The attack point</param>
    /// <param name="critRate">The critical rate</param>
    /// <param name="rand">The random value</param>
    /// <returns>Modified attack point</returns>
    public int CriticalAtk(int atk, float critRate, float rand)
    {
        if (atk <= 0)
        {
            return 0;
        }
        return rand < critRate ? (int)(atk * 1.5f) : atk;
    }
}

[dun047]

歐雷(程式)

答案

        public int CriticalAtk(int atk, float critRate, float rand)
        {
            if(atk < 0) return atk = 0;
            if(critRate > rand) atk = (int)(atk * 1.5f);
            return atk;
        }

[MurasakiAurora]

極光（程式）

我的答案

 public int CriticalAtk(int atk, float critRate, float rand)
        {
            if (0 > atk)
            {
                return 0;
            }
            if (critRate>rand)
            {
                return (int)(atk * 1.5f);
            }
            return atk;
        }