プロパティベースのテスト概要
Intro to property based tests
一部の企業は、インタビュープロセスの一環として、ローマ数字のカタを実行するように求めます。この章では、TDDでこれに取り組む方法を示します。
アラビア数字(数字0〜9)をローマ数字に変換する関数を記述します。
ローマ数字について聞いたことがない場合は、ローマ人が数字を書き留めた方法です。
あなたはシンボルを一緒に貼り付けることによってそれらを構築し、それらのシンボルは数字を表します
つまりIは「1」です。IIIは「3」です。
簡単に見えますが、いくつか興味深いルールがあります。 Vは「5」を意味しますが、IVは「4」です( IIIIではありません)。
MCMLXXXIVは「1984」です。それは複雑に見え、これを最初から理解するためのコードをどのように書くことができるか想像することは困難です。
この本で強調しているように、ソフトウェア開発者にとって重要なスキルは、「有用な」機能の「薄い垂直スライス」を特定して特定し、繰り返すことです。 TDDワークフローは、反復的な開発を容易にするのに役立ちます。
したがって、「1984」ではなく、「1」から始めましょう。

最初にテストを書く

1
func TestRomanNumerals(t *testing.T) {
2
got := ConvertToRoman(1)
3
want := "I"
4
5
if got != want {
6
t.Errorf("got %q, want %q", got, want)
7
}
8
}
Copied!
あなたがこの本でこれまでに持っているならば、これはうまくいけばあなたにとって非常に退屈で日常的な感じです。それは良いことです。

テストを実行してみます

./numeral_test.go:6:9: undefined: ConvertToRoman
コンパイラーに道を案内する

テストを実行するための最小限のコードを記述し、失敗したテスト出力を確認します

関数を作成しますが、まだテストに合格しないでください。常に、期待どおりにテストが失敗することを確認してください。
1
func ConvertToRoman(arabic int) string {
2
return ""
3
}
Copied!
今すぐ実行されます。
1
=== RUN TestRomanNumerals
2
--- FAIL: TestRomanNumerals (0.00s)
3
numeral_test.go:10: got '', want 'I'
4
FAIL
Copied!

成功させるのに十分なコードを書く

1
func ConvertToRoman(arabic int) string {
2
return "I"
3
}
Copied!

リファクタリング♪

まだリファクタリングする必要はありません。
結果をハードコーディングするだけでは変だと感じますが、TDDではできるだけ長く「赤字」を避けたいと考えています。あまり達成していないように感じるかもしれませんが、APIを定義し、ルールの1つをキャプチャするテストを取得しました。「実際の」コードがかなりばかげていても。
ここで、その不安な気持ちを使って新しいテストを記述し、少しだけ馬鹿なコードを書くように強制します。

最初にテストを書く

サブテストを使用してテストを適切にグループ化できます
1
func TestRomanNumerals(t *testing.T) {
2
t.Run("1 gets converted to I", func(t *testing.T) {
3
got := ConvertToRoman(1)
4
want := "I"
5
6
if got != want {
7
t.Errorf("got %q, want %q", got, want)
8
}
9
})
10
11
t.Run("2 gets converted to II", func(t *testing.T) {
12
got := ConvertToRoman(2)
13
want := "II"
14
15
if got != want {
16
t.Errorf("got %q, want %q", got, want)
17
}
18
})
19
}
Copied!

テストを実行してみます

1
=== RUN TestRomanNumerals/2_gets_converted_to_II
2
--- FAIL: TestRomanNumerals/2_gets_converted_to_II (0.00s)
3
numeral_test.go:20: got 'I', want 'II'
Copied!
それほど驚きはありません

成功させるのに十分なコードを書く

1
func ConvertToRoman(arabic int) string {
2
if arabic == 2 {
3
return "II"
4
}
5
return "I"
6
}
Copied!
ええ、まだ問題に取り組んでいないようです。したがって、前進させるために、さらに多くのテストを作成する必要があります。

リファクタリング♪

テストでいくつかの繰り返しがあります。 「与えられた入力X、Yを期待する」の問題のように感じる何かをテストしているときは、おそらくテーブルベースのテストを使用する必要があります。
1
func TestRomanNumerals(t *testing.T) {
2
cases := []struct {
3
Description string
4
Arabic int
5
Want string
6
}{
7
{"1 gets converted to I", 1, "I"},
8
{"2 gets converted to II", 2, "II"},
9
}
10
11
for _, test := range cases {
12
t.Run(test.Description, func(t *testing.T) {
13
got := ConvertToRoman(test.Arabic)
14
if got != test.Want {
15
t.Errorf("got %q, want %q", got, test.Want)
16
}
17
})
18
}
19
}
Copied!
これ以上テストのボイラプレートを書かなくても、簡単にケースを追加できるようになりました。
頑張って「3」に行きましょう

最初にテストを書く

以下をケースに追加してください
1
{"3 gets converted to III", 3, "III"},
Copied!

テストを実行してみます

1
=== RUN TestRomanNumerals/3_gets_converted_to_III
2
--- FAIL: TestRomanNumerals/3_gets_converted_to_III (0.00s)
3
numeral_test.go:20: got 'I', want 'III'
Copied!

成功させるのに十分なコードを書く

1
func ConvertToRoman(arabic int) string {
2
if arabic == 3 {
3
return "III"
4
}
5
if arabic == 2 {
6
return "II"
7
}
8
return "I"
9
}
Copied!

リファクタリング♪

わかりましたので、これらのifステートメントを楽しんでいないようになり、コードを十分に見てみると、arabicのサイズに基づいてIの文字列を構築していることがわかります。
より複雑な数値については、ある種の算術および文字列連結を行うことを "知っています"。
これらの考えを念頭に置いてリファクタリングを試してみましょう。 それは最終的なソリューションには適さないかもしれませんが、それは問題ありません。 私たちはいつでもコードを捨てて、私たちをガイドする必要のあるテストからやり直すことができます。
1
func ConvertToRoman(arabic int) string {
2
3
var result strings.Builder
4
5
for i:=0; i<arabic; i++ {
6
result.WriteString("I")
7
}
8
9
return result.String()
10
}
Copied!
これまでにstrings.Builderを使用したことがない可能性があります
Builderは、Writeメソッドを使用して文字列を効率的に構築するために使用されます。メモリのコピーを最小限に抑えます。
通常、実際のパフォーマンスの問題が発生するまで、このような最適化に悩まされることはありませんが、コードの量は文字列に追加する「手動」よりも大きくないため、より高速なアプローチを使用することもできます。
コードは私にはよく見え、ドメインを 私たちが今知っているように 記述しています。

ローマ人もDRYに夢中になりました...

物事は今より複雑になり始めています。ローマ人はその知恵の中で、繰り返し登場する人物は読みにくく、数え難くなるだろうと考えていました。したがって、ローマ数字のルールでは、同じ文字を3回以上続けて繰り返すことはできません。
代わりに、次に高いシンボルを取り、その左側にシンボルを置くことで「減算」します。すべてのシンボルを減算器として使用できるわけではありません。I(1)、X(10)、C(100)のみ。
たとえば、ローマ数字の「5」はVです。 「4」を作成するには、 IIIIではなく、 IVを実行します。

最初にテストを書く

1
{"4 gets converted to IV (can't repeat more than 3 times)", 4, "IV"},
Copied!

テストを実行してみます

1
=== RUN TestRomanNumerals/4_gets_converted_to_IV_(cant_repeat_more_than_3_times)
2
--- FAIL: TestRomanNumerals/4_gets_converted_to_IV_(cant_repeat_more_than_3_times) (0.00s)
3
numeral_test.go:24: got 'IIII', want 'IV'
Copied!

成功させるのに十分なコードを書く

1
func ConvertToRoman(arabic int) string {
2
3
if arabic == 4 {
4
return "IV"
5
}
6
7
var result strings.Builder
8
9
for i:=0; i<arabic; i++ {
10
result.WriteString("I")
11
}
12
13
return result.String()
14
}
Copied!

リファクタリング♪

文字列の構築パターンを壊したことが「好き」ではないので、それを続けたいと思います。
1
func ConvertToRoman(arabic int) string {
2
3
var result strings.Builder
4
5
for i := arabic; i > 0; i-- {
6
if i == 4 {
7
result.WriteString("IV")
8
break
9
}
10
result.WriteString("I")
11
}
12
13
return result.String()
14
}
Copied!
「4」を現在の考えに「合わせる」ために、アラビア数字からカウントダウンし、進行中に文字列に記号を追加します。これが長期的に機能するかどうかはわかりませんが、見てみましょう!
「5」を作成しましょう

最初にテストを書く

1
{"5 gets converted to V", 5, "V"},
Copied!

テストを実行してみます

1
=== RUN TestRomanNumerals/5_gets_converted_to_V
2
--- FAIL: TestRomanNumerals/5_gets_converted_to_V (0.00s)
3
numeral_test.go:25: got 'IIV', want 'V'
Copied!

成功させるのに十分なコードを書く

「4」で行ったアプローチをコピーするだけです
1
func ConvertToRoman(arabic int) string {
2
3
var result strings.Builder
4
5
for i := arabic; i > 0; i-- {
6
if i == 5 {
7
result.WriteString("V")
8
break
9
}
10
if i == 4 {
11
result.WriteString("IV")
12
break
13
}
14
result.WriteString("I")
15
}
16
17
return result.String()
18
}
Copied!

リファクタリング♪

このようなループでの繰り返しは、通常、呼び出されるのを待っている抽象化の兆候です。ループを短絡することは読みやすさのための効果的なツールであるかもしれませんが、それはまたあなたに何か他のものを伝えているかもしれません。
アラビア数字をループしていて、特定の記号にぶつかった場合はbreakと呼びますが、実際に行っているのは、iを手間をかけて減算することです。
1
func ConvertToRoman(arabic int) string {
2
3
var result strings.Builder
4
5
for arabic > 0 {
6
switch {
7
case arabic > 4:
8
result.WriteString("V")
9
arabic -= 5
10
case arabic > 3:
11
result.WriteString("IV")
12
arabic -= 4
13
default:
14
result.WriteString("I")
15
arabic--
16
}
17
}
18
19
return result.String()
20
}
Copied!
  • いくつかの非常に基本的なシナリオのテストから得られたコードから読み取っている信号を考えると、ローマ数字を作成するには、シンボルを適用するときに「アラビア語」から減算する必要があることがわかります
  • forループはもはやiに依存せず、代わりに、arabicから十分な数のシンボルを減算するまで文字列を構築し続けます。
このアプローチが「6」(VI)、「7」(VII)、「8」(VIII)にも有効であると確信しています。 それでも、ケースをテストスイートに追加し、(簡潔にするためにコードは含めません。不明な場合はgithubのサンプルを確認してください)を確認してください。
「9」は、次の数の表現から「I」を減算する必要があるという点で、「4」と同じルールに従います。 「10」はローマ数字で「X」で表されます。したがって、「9」は「IX」になります。

最初にテストを書く

1
{"9 gets converted to IX", 9, "IX"}
Copied!

テストを実行してみます

1
=== RUN TestRomanNumerals/9_gets_converted_to_IX
2
--- FAIL: TestRomanNumerals/9_gets_converted_to_IX (0.00s)
3
numeral_test.go:29: got 'VIV', want 'IX'
Copied!

成功させるのに十分なコードを書く

以前と同じアプローチを採用できるはずです
1
case arabic > 8:
2
result.WriteString("IX")
3
arabic -= 9
Copied!

リファクタリング♪

それはコードがまだどこかにリファクタリングがあることを私たちに伝えているように感じますが、私には完全に明白ではないので、続けましょう。
これもコードはスキップしますが、テストケースに「10」のテストを追加します。「10」は「X」である必要があり、先に進む前に合格にします。
「39」までのコードが機能すると確信しているので、ここに追加したいくつかのテストがあります。
1
{"10 gets converted to X", 10, "X"},
2
{"14 gets converted to XIV", 14, "XIV"},
3
{"18 gets converted to XVIII", 18, "XVIII"},
4
{"20 gets converted to XX", 20, "XX"},
5
{"39 gets converted to XXXIX", 39, "XXXIX"},
Copied!
これまでにオブジェクト指向プログラミングを行ったことがある場合は、少し疑惑を抱いてswitchステートメントを表示する必要があることがわかります。通常、実際には代わりにクラス構造でキャプチャできる場合でも、一部の命令コード内でコンセプトまたはデータをキャプチャしています。
Goは厳密にはオブジェクト指向ではありませんが、オブジェクト指向が提供するレッスンを完全に無視することを意味するわけではありません(あなたが伝えたいだけのことです)。
私たちの切り替えステートメントは、動作とともにローマ数字に関するいくつかの真実を説明しています。
データを動作から分離することで、これをリファクタリングできます。
1
type RomanNumeral struct {
2
Value int
3
Symbol string
4
}
5
6
var allRomanNumerals = []RomanNumeral {
7
{10, "X"},
8
{9, "IX"},
9
{5, "V"},
10
{4, "IV"},
11
{1, "I"},
12
}
13
14
func ConvertToRoman(arabic int) string {
15
16
var result strings.Builder
17
18
for _, numeral := range allRomanNumerals {
19
for arabic >= numeral.Value {
20
result.WriteString(numeral.Symbol)
21
arabic -= numeral.Value
22
}
23
}
24
25
return result.String()
26
}
Copied!
これはずっと気分が良いです。数値に関連するいくつかのルールをアルゴリズムで非表示にするのではなくデータとして宣言しました。アラビア数字を処理して、適合する場合は結果に記号を追加する方法を確認できます。
この抽象化はより大きな数で機能しますか? 「50」のローマ数字(L)で機能するようにテストスイートを拡張します。
ここにいくつかのテストケースがあります。
1
{"40 gets converted to XL", 40, "XL"},
2
{"47 gets converted to XLVII", 47, "XLVII"},
3
{"49 gets converted to XLIX", 49, "XLIX"},
4
{"50 gets converted to L", 50, "L"},
Copied!
助けが必要?
追加するシンボルはこの要点で確認できます。

そして残りの部分!

残りの記号は次のとおりです
Arabic
Roman
100
C
500
D
1000
M
残りのシンボルについても同じ方法を使用します。テストとシンボルの配列の両方にデータを追加するだけです。
あなたのコードは1984:MCMLXXXIVで動作しますか?
これが私の最後のテストスイートです
1
func TestRomanNumerals(t *testing.T) {
2
cases := []struct {
3
Arabic int
4
Roman string
5
}{
6
{Arabic: 1, Roman: "I"},
7
{Arabic: 2, Roman: "II"},
8
{Arabic: 3, Roman: "III"},
9
{Arabic: 4, Roman: "IV"},
10
{Arabic: 5, Roman: "V"},
11
{Arabic: 6, Roman: "VI"},
12
{Arabic: 7, Roman: "VII"},
13
{Arabic: 8, Roman: "VIII"},
14
{Arabic: 9, Roman: "IX"},
15
{Arabic: 10, Roman: "X"},
16
{Arabic: 14, Roman: "XIV"},
17
{Arabic: 18, Roman: "XVIII"},
18
{Arabic: 20, Roman: "XX"},
19
{Arabic: 39, Roman: "XXXIX"},
20
{Arabic: 40, Roman: "XL"},
21
{Arabic: 47, Roman: "XLVII"},
22
{Arabic: 49, Roman: "XLIX"},
23
{Arabic: 50, Roman: "L"},
24
{Arabic: 100, Roman: "C"},
25
{Arabic: 90, Roman: "XC"},
26
{Arabic: 400, Roman: "CD"},
27
{Arabic: 500, Roman: "D"},
28
{Arabic: 900, Roman: "CM"},
29
{Arabic: 1000, Roman: "M"},
30
{Arabic: 1984, Roman: "MCMLXXXIV"},
31
{Arabic: 3999, Roman: "MMMCMXCIX"},
32
{Arabic: 2014, Roman: "MMXIV"},
33
{Arabic: 1006, Roman: "MVI"},
34
{Arabic: 798, Roman: "DCCXCVIII"},
35
}
36
for _, test := range cases {
37
t.Run(fmt.Sprintf("%d gets converted to %q", test.Arabic, test.Roman), func(t *testing.T) {
38
got := ConvertToRoman(test.Arabic)
39
if got != test.Roman {
40
t.Errorf("got %q, want %q", got, test.Roman)
41
}
42
})
43
}
44
}
Copied!
  • データに十分な情報が記載されていると感じたため、「コメント」を削除しました。
  • もう少し自信を与えるために見つけた他のエッジケースをいくつか追加しました。テーブルベースのテストでは、これは非常に安価です。
アルゴリズムは変更しませんでした。allRomanNumerals配列を更新するだけで済みました。
1
var allRomanNumerals = []RomanNumeral{
2
{1000, "M"},
3
{900, "CM"},
4
{500, "D"},
5
{400, "CD"},
6
{100, "C"},
7
{90, "XC"},
8
{50, "L"},
9
{40, "XL"},
10
{10, "X"},
11
{9, "IX"},
12
{5, "V"},
13
{4, "IV"},
14
{1, "I"},
15
}
Copied!

ローマ数字の解析

まだ終わっていません。次に、ローマ数字の fromintに変換する関数を書きます

最初にテストを書く

テストケースを少しリファクタリングして再利用できます
case変数をvarブロックのパッケージ変数としてテストの外に移動します。
1
func TestConvertingToArabic(t *testing.T) {
2
for _, test := range cases[:1] {
3
t.Run(fmt.Sprintf("%q gets converted to %d", test.Roman, test.Arabic), func(t *testing.T) {
4
got := ConvertToArabic(test.Roman)
5
if got != test.Arabic {
6
t.Errorf("got %d, want %d", got, test.Arabic)
7
}
8
})
9
}
10
}
Copied!
スライス機能を使用して、今のところテストの1つだけを実行していることに注意してください。(cases [:1])これらのテストを一度にすべて合格にしようとすると、飛躍的に大きくなります。

テストを実行してみます

1
./numeral_test.go:60:11: undefined: ConvertToArabic
Copied!

テストを実行するための最小限のコードを記述し、失敗したテスト出力を確認します

新しい関数定義を追加します
1
func ConvertToArabic(roman string) int {
2
return 0
3
}
Copied!
テストが実行され、失敗するはずです
1
--- FAIL: TestConvertingToArabic (0.00s)
2
--- FAIL: TestConvertingToArabic/'I'_gets_converted_to_1 (0.00s)
3
numeral_test.go:62: got 0, want 1
Copied!

成功させるのに十分なコードを書く

あなたは何をするべきか知っています
1
func ConvertToArabic(roman string) int {
2
return 1
3
}
Copied!
次に、テストのスライスインデックスを変更して、次のテストケースに移動します(例:cases [:2])。 3つ目のケースについても、考えられる最もおかしなコードを理解し、(最高の本で間違いありませんか?)これが私のばかげたコードです。
1
func ConvertToArabic(roman string) int {
2
if roman == "III" {
3
return 3
4
}
5
if roman == "II" {
6
return 2
7
}
8
return 1
9
}
Copied!
機能する実際のコードの愚かさを通して、以前のようなパターンを見ることができます。入力を反復処理して something を構築する必要があります。この場合は合計です。
1
func ConvertToArabic(roman string) int {
2
total := 0
3
for range roman {
4
total++
5
}
6
return total
7
}
Copied!

最初にテストを書く

次に、cases[:4]IV)に移動します。これは、文字列の長さである2を返すため、失敗します。

成功させるのに十分なコードを書く

1
// earlier..
2
type RomanNumerals []RomanNumeral
3
4
func (r RomanNumerals) ValueOf(symbol string) int {
5
for _, s := range r {
6
if s.Symbol == symbol {
7
return s.Value
8
}
9
}
10
11
return 0
12
}
13
14
// later..
15
func ConvertToArabic(roman string) int {
16
total := 0
17
18
for i := 0; i < len(roman); i++ {
19
symbol := roman[i]
20
21
// look ahead to next symbol if we can and, the current symbol is base 10 (only valid subtractors)
22
if i+1 < len(roman) && symbol == 'I' {
23
nextSymbol := roman[i+1]
24
25
// build the two character string
26
potentialNumber := string([]byte{symbol, nextSymbol})
27
28
// get the value of the two character string
29
value := allRomanNumerals.ValueOf(potentialNumber)
30
31
if value != 0 {
32
total += value
33
i++ // move past this character too for the next loop
34
} else {
35
total++
36
}
37
} else {
38
total++
39
}
40
}
41
return total
42
}
Copied!
これは恐ろしいことですが、機能します。コメントを追加する必要があると感じたのは残念です。
  • 与えられたローマ数字の整数値を検索できるようにしたいので、RomanNumeralの配列から型を作成し、それにメソッドValueOfを追加しました
  • 次にループの中では、文字列が十分に大きくて、現在の記号が有効な減算器であるかどうか、先を見る必要があります。現時点では、 I(1)ですが、X(10)または C(100)にすることもできます。
    • これらの両方の条件を満たす場合、値を検索して合計に追加する必要があります if 特別な減算器の1つです。それ以外の場合は無視します
    • 次に、このシンボルを2回カウントしないように、iをさらにインクリメントする必要があります。

リファクタリング♪

私はこれが長期的なアプローチになると完全に確信しているわけではなく、私たちができる可能性のある興味深いリファクタリングが潜在的にあるかもしれませんが、私たちのアプローチが完全に間違っている場合に備えて私はそれに抵抗します。私はむしろいくつかのテストに最初に合格して見てもらいたいです。その間、私は最初のifステートメントを少し恐ろしくなくしました。
1
func ConvertToArabic(roman string) int {
2
total := 0
3
4
for i := 0; i < len(roman); i++ {
5
symbol := roman[i]
6
7
if couldBeSubtractive(i, symbol, roman) {
8
nextSymbol := roman[i+1]
9
10
// build the two character string
11
potentialNumber := string([]byte{symbol, nextSymbol})
12
13
// get the value of the two character string
14
value := allRomanNumerals.ValueOf(potentialNumber)
15
16
if value != 0 {
17
total += value
18
i++ // move past this character too for the next loop
19
} else {
20
total++
21
}
22
} else {
23
total++
24
}
25
}
26
return total
27
}
28
29
func couldBeSubtractive(index int, currentSymbol uint8, roman string) bool {
30
return index+1 < len(roman) && currentSymbol == 'I'
31
}
Copied!

最初にテストを書く

cases[:5]に移りましょう
1
=== RUN TestConvertingToArabic/'V'_gets_converted_to_5
2
--- FAIL: TestConvertingToArabic/'V'_gets_converted_to_5 (0.00s)
3
numeral_test.go:62: got 1, want 5
Copied!

成功させるのに十分なコードを書く

それが減算である場合を除いて、コードはすべての文字がIであると想定しているため、値は「1」です。これを修正するには、ValueOfメソッドを再利用できるはずです。
1
func ConvertToArabic(roman string) int {
2
total := 0
3
4
for i := 0; i < len(roman); i++ {
5
symbol := roman[i]
6
7
// look ahead to next symbol if we can and, the current symbol is base 10 (only valid subtractors)
8
if couldBeSubtractive(i, symbol, roman) {
9
nextSymbol := roman[i+1]
10
11
// build the two character string
12
potentialNumber := string([]byte{symbol, nextSymbol})
13
14
if value := allRomanNumerals.ValueOf(potentialNumber); value != 0 {
15
total += value
16
i++ // move past this character too for the next loop
17
} else {
18
total++ // this is fishy...
19
}
20
} else {
21
total+=allRomanNumerals.ValueOf(string([]byte{symbol}))
22
}
23
}
24
return total
25
}
Copied!

リファクタリング♪

Goで文字列にインデックスを付けると、byteを取得します。これが、文字列を再度構築するときに、string([]byte{symbol})のようなことをしなければならない理由です。 数回繰り返されますが、その機能を移動して、ValueOfが代わりに数バイトを取るようにします。
1
func (r RomanNumerals) ValueOf(symbols ...byte) int {
2
symbol := string(symbols)
3
for _, s := range r {
4
if s.Symbol == symbol {
5
return s.Value
6
}
7
}
8
9
return 0
10
}
Copied!
次に、バイトをそのまま関数に渡すことができます
1
func ConvertToArabic(roman string) int {
2
total := 0
3
4
for i := 0; i < len(roman); i++ {
5
symbol := roman[i]
6
7
if couldBeSubtractive(i, symbol, roman) {
8
if value := allRomanNumerals.ValueOf(symbol, roman[i+1]); value != 0 {
9
total += value
10
i++ // move past this character too for the next loop
11
} else {
12
total++ // this is fishy...
13
}
14
} else {
15
total+=allRomanNumerals.ValueOf(symbol)
16
}
17
}
18
return total
19
}
Copied!
それはまだかなり厄介ですが、そこに到達しています。
cases[:xx]の数値を移動し始めると、かなりの数が現在通過していることがわかります。スライスオペレーターを完全に削除して、失敗するオペレーターを確認します。これが私のスイートの例です。
1
=== RUN TestConvertingToArabic/'XL'_gets_converted_to_40
2
--- FAIL: TestConvertingToArabic/'XL'_gets_converted_to_40 (0.00s)
3
numeral_test.go:62: got 60, want 40
4
=== RUN TestConvertingToArabic/'XLVII'_gets_converted_to_47
5
--- FAIL: TestConvertingToArabic/'XLVII'_gets_converted_to_47 (0.00s)
6
numeral_test.go:62: got 67, want 47
7
=== RUN TestConvertingToArabic/'XLIX'_gets_converted_to_49
8
--- FAIL: TestConvertingToArabic/'XLIX'_gets_converted_to_49 (0.00s)
9
numeral_test.go:62: got 69, want 49
Copied!
私たちが見逃しているのは、couldBeSubtractiveの更新だけなので、他の種類の減算記号を考慮に入れていると思います
1
func couldBeSubtractive(index int, currentSymbol uint8, roman string) bool {
2
isSubtractiveSymbol := currentSymbol == 'I' || currentSymbol == 'X' || currentSymbol =='C'
3
return index+1 < len(roman) && isSubtractiveSymbol
4
}
Copied!
もう一度試してください。まだ失敗します。しかし、私たちは以前にコメントを残しました...
1
total++ // this is fishy...
Copied!
すべての記号がIであることを意味するので、totalをインクリメントするだけではいけません。それを次のものに置き換えます。
1
total += allRomanNumerals.ValueOf(symbol)
Copied!
そして、すべてのテストに合格しました!
これで完全に機能するソフトウェアができたので、自信を持ってリファクタリングを行うことができます。

リファクタリング♪

ここに私が仕上げたすべてのコードがあります。私はいくつかの失敗した試みをしましたが、強調し続けているように、それは問題ありません、そしてテストは私が自由にコードをいじるのを助けます。
1
import "strings"
2
3
func ConvertToArabic(roman string) (total int) {
4
for _, symbols := range windowedRoman(roman).Symbols() {
5
total += allRomanNumerals.ValueOf(symbols...)
6
}
7
return
8
}
9
10
func ConvertToRoman(arabic int) string {
11
var result strings.Builder
12
13
for _, numeral := range allRomanNumerals {
14
for arabic >= numeral.Value {
15
result.WriteString(numeral.Symbol)
16
arabic -= numeral.Value
17
}
18
}
19
20
return result.String()
21
}
22
23
type romanNumeral struct {
24
Value int
25
Symbol string
26
}
27
28
type romanNumerals []romanNumeral
29
30
func (r romanNumerals) ValueOf(symbols ...byte) int {
31
symbol := string(symbols)
32
for _, s := range r {
33
if s.Symbol == symbol {
34
return s.Value
35
}
36
}
37
38
return 0
39
}
40
41
func (r romanNumerals) Exists(symbols ...byte) bool {
42
symbol := string(symbols)
43
for _, s := range r {
44
if s.Symbol == symbol {
45
return true
46
}
47
}
48
return false
49
}
50
51
var allRomanNumerals = romanNumerals{
52
{1000, "M"},
53
{900, "CM"},
54
{500, "D"},
55
{400, "CD"},
56
{100, "C"},
57
{90, "XC"},
58
{50, "L"},
59
{40, "XL"},
60
{10, "X"},
61
{9, "IX"},
62
{5, "V"},
63
{4, "IV"},
64
{1, "I"},
65
}
66
67
type windowedRoman string
68
69
func (w windowedRoman) Symbols() (symbols [][]byte) {
70
for i := 0; i < len(w); i++ {
71
symbol := w[i]
72
notAtEnd := i+1 < len(w)
73
74
if notAtEnd && isSubtractive(symbol) && allRomanNumerals.Exists(symbol, w[i+1]) {
75
symbols = append(symbols, []byte{byte(symbol), byte(w[i+1])})
76
i++
77
} else {
78
symbols = append(symbols, []byte{byte(symbol)})
79
}
80
}
81
return
82
}
83
84
func isSubtractive(symbol uint8) bool {
85
return symbol == 'I' || symbol == 'X' || symbol == 'C'
86
}
Copied!
以前のコードの私の主な問題は、以前のリファクタリングに似ています。一緒に結合された懸念が多すぎました。文字列からローマ数字を抽出し、それらの値を検索するアルゴリズムを作成しました。
そこで、数値の抽出を処理する新しいタイプのwindowedRomanを作成し、それらをスライスとして取得するためのSymbolsメソッドを提供しました。これは、ConvertToArabic関数が単にシンボルを反復処理してそれらを合計できることを意味しました。
いくつかの関数を抽出することでコードを少し壊しました。特に、現在処理している記号が2文字の減算記号であるかどうかを判断するために、不安定なifステートメントを中心にしています。
おそらくもっとエレガントな方法があるでしょうが、私はそれを気にするつもりはありません。コードはそこにあり、動作し、テストされています。私(または他の誰か)が安全に変更できるより良い方法を見つけた場合-大変な作業は完了です。

プロパティベースのテストの概要

この章で使用したローマ数字のドメインにはいくつかのルールがあります
  • 3つ以上の連続したシンボルは使用できません
  • I (1)、 X (10) 、 C (100) のみが「減算器」になります
  • ConvertToRoman(N)の結果を取得してConvertToArabicに渡すと、Nが返されます
これまでに作成したテストは、「サンプル」ベースのテストとして説明できます。このテストでは、コード周辺のいくつかの例を検証するためのツールを提供します。
ドメインについて知っているこれらのルールを採用し、コードに対して何らかの方法でそれらを実行できるとしたらどうでしょうか。
プロパティベースのテストは、コードにランダムデータを投げ、記述したルールが常に正しいことを確認することで、これを行うのに役立ちます。多くの人は、プロパティベースのテストは主にランダムデータについてであると考えていますが、それは間違いです。プロパティベースのテストに関する本当の課題は、ドメインをよく理解して、これらのプロパティを記述できるようにすることです。
十分な言葉、いくつかのコードを見てみましょう
1
func TestPropertiesOfConversion(t *testing.T) {
2
assertion := func(arabic int) bool {
3
roman := ConvertToRoman(arabic)
4
fromRoman := ConvertToArabic(roman)
5
return fromRoman == arabic
6
}
7
8
if err := quick.Check(assertion, nil); err != nil {
9
t.Error("failed checks", err)
10
}
11
}
Copied!

財産の根拠

最初のテストでは、数値をローマ数字に変換する場合に、他の関数を使用して、最初に取得した数値に変換するときにチェックします。
  • 与えられた乱数(例えば4)。
  • 乱数でConvertToRomanを呼び出します(4の場合はIVを返す必要があります)。
  • 上記の結果を受け取り、それをConvertToArabicに渡します。
  • 上記により、元の入力(4)が得られます。
これは、どちらかにバグがあると壊れるので、自信をつける良いテストのように感じます。合格できる唯一の方法は、彼らに同じ種類のバグがあるかどうかです。これは不可能ではありませんが、ありそうにありません。

技術的な説明

標準ライブラリのtesting/quickパッケージを使用しています
下から読んで、quick.Check関数をいくつかのランダムな入力に対して実行する関数を提供します。関数がfalseを返す場合、チェックに失敗したと見なされます。
上記のassertion関数は乱数を受け取り、関数を実行してプロパティをテストします。

テストを実行する

実行してみてください。
お使いのコンピュータがしばらくハングアップする可能性があるので、退屈したらそれを殺してください。
どうしたの? 以下をアサーションコードに追加してみてください。
1
assertion := func(arabic int) bool {
2
if arabic <0 || arabic > 3999 {
3
log.Println(arabic)
4
return true
5
}
6
roman := ConvertToRoman(arabic)
7
fromRoman := ConvertToArabic(roman)
8
return fromRoman == arabic
9
}
Copied!
次のようなものが表示されます。