🐛 (roman-numerals): fix sixth-test and refactor

yuanyu90221 · yuanyu90221 · commit 66f44f2981c4 · 2025-07-12T23:38:56.000+08:00
diff --git a/property-base-test/roman_number.go b/property-base-test/roman_number.go
@@ -18,9 +18,9 @@ var allRomanNumerals = []RomanNumerals{
 func ConvertToRoman(arabic int) string {
 	var result strings.Builder
 	for _, romanNumeral := range allRomanNumerals {
-		if arabic >= romanNumeral.Value {
-			arabic -= romanNumeral.Value
+		for arabic >= romanNumeral.Value {
 			result.WriteString(romanNumeral.Symbol)
+			arabic -= romanNumeral.Value
 		}
 	}
 	return result.String()

Original file line number	Diff line number	Diff line change
`@@ -18,9 +18,9 @@ var allRomanNumerals = []RomanNumerals{`
`18`	`18`	`func ConvertToRoman(arabic int) string {`
`19`	`19`	`var result strings.Builder`
`20`	`20`	`for _, romanNumeral := range allRomanNumerals {`
`21`		`- if arabic >= romanNumeral.Value {`
`22`		`- arabic -= romanNumeral.Value`
	`21`	`+ for arabic >= romanNumeral.Value {`
`23`	`22`	`result.WriteString(romanNumeral.Symbol)`
	`23`	`+ arabic -= romanNumeral.Value`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`	`return result.String()`