使用Golang编写的Project Euler#22问题;每次返回不同的结果。

huangapple go评论149阅读模式
英文:

Project Euler #22 with Golang; returns different result every time

问题

我正在使用Go解决Project Euler的第22题,但我对Go还不熟悉,我的代码给出了不一致的结果,每次运行程序都会显示不同的结果。它的结果总是非常接近我所认为的正确答案,但在几百个点之间波动。我最初认为可能是浮点数舍入不准确导致的,但我已经检查过了,不是这个问题(我认为)。如果有人能指出可能导致这种情况的原因,我将非常感激。我已经努力寻找代码问题几天了,但没有能够解决它,甚至在论坛上也没有找到类似的问题。

另外,我最初使用了Go语言的math/big包,但得到的结果也是不断变化的。

package main

import (
	"fmt"
	"io/ioutil"
	"log"
	"math"
	"strings"
)

func openFileReturnSlice(f string) []string {
	bytes, err := ioutil.ReadFile(f)
	if err != nil {
		log.Fatal("Failed to read file: p022_names.txt")
	}
	s2s := strings.Split(string(bytes), "\"")
	s22 := strings.Join(s2s, "")
	names := strings.Split(s22, ",")
	return names
}

func alphabetize(n []string) ([]string, map[string]int) {
	wordsValues := make(map[string]float64)
	wordLetterVal := make(map[string]int)
	for _, s := range n {
		loop := -1
		var wordValue float64
		alpha := int(0)
		for _, l := range s {
			ell := int(l) - 64
			mvDec := math.Pow(100, math.Abs(float64(loop)))
			wordValue += float64(l) / mvDec
			alpha += ell
			loop--
		}
		wordsValues[s] = wordValue
		wordLetterVal[s] = alpha
	}
	var sortedNames []string
	lenWordValues := len(wordsValues)
	for i := 0; i < lenWordValues; i++ {
		var lowValName string
		lowVal := float64(10)
		for k, v := range wordsValues {
			if v < lowVal {
				lowVal = v
				lowValName = k
			}
		}
		delete(wordsValues, lowValName)
		sortedNames = append(sortedNames, lowValName)
	}
	return sortedNames, wordLetterVal
}

func main() {
	names := openFileReturnSlice("p022_names.txt")
	alphabetical, sumAlphaValues := alphabetize(names)
	var total int
	for k := 0; k < len(alphabetical); k++ {
		var temp int
		key := k + 1
		temp = sumAlphaValues[alphabetical[k]] * key
		total += temp
	}
	fmt.Println("The total is: ", total)
}
英文:

I am doing problem 22 of Project Euler with Go, which I am new to, and my code gives me inconsistent results meaning every time I run the program it shows a different result. It is always very close to what I have seen is the correct answer but ranges around within several hundred points. I originally thought it may have been due to float rounding inaccuracy but I have checked and that is not the case(I think).
I would really appreciate it if someone could point out what may be going on that would cause this. I have struggled to find the problem with my code for several days and have not been able to solve it or even find similar issues on forums.
As a side note I originally wrote this using the golang math/big package and was getting the same changing results.

package main
import (
&quot;fmt&quot;
&quot;io/ioutil&quot;
&quot;log&quot;
&quot;math&quot;
&quot;strings&quot;
)
func openFileReturnSlice(f string) []string {
bytes, err := ioutil.ReadFile(f)
if err != nil {
log.Fatal(&quot;Failed to read file: p022_names.txt&quot;)
}
s2s := strings.Split(string(bytes), &quot;\&quot;&quot;)
s22 := strings.Join(s2s, &quot;&quot;)
names := strings.Split(s22, &quot;,&quot;)
return names
}
func alphabetize(n []string) ([]string, map[string]int) {
wordsValues := make(map[string]float64)
wordLetterVal := make(map[string]int)
for _, s := range n {
loop := -1
var wordValue float64
alpha := int(0)
for _, l := range s {
ell := int(l) - 64
mvDec := math.Pow(100, math.Abs(float64(loop)))
wordValue += float64(l) / mvDec
alpha += ell
loop--
}
wordsValues
展开收缩
= wordValue wordLetterVal
展开收缩
= alpha } var sortedNames []string lenWordValues := len(wordsValues) for i := 0; i &lt; lenWordValues; i++ { var lowValName string lowVal := float64(10) for k, v := range wordsValues { if v &lt; lowVal { lowVal = v lowValName = k } } delete(wordsValues, lowValName) sortedNames = append(sortedNames, lowValName) } return sortedNames, wordLetterVal } func main() { names := openFileReturnSlice(&quot;p022_names.txt&quot;) alphabetical, sumAlphaValues := alphabetize(names) var total int for k := 0; k &lt; len(alphabetical); k++ { var temp int key := k + 1 temp = sumAlphaValues[alphabetical[k]] * key total += temp } fmt.Println(&quot;The total is: &quot;, total) }

答案1

得分: 4

浮点数的使用是可疑的,它是不精确的。对映射的迭代顺序没有指定,并且不能保证从一次迭代到下一次迭代是相同的。你对映射的使用很可能是看似随机行为的最有可能的解释。

第一个问题要问的是:什么是正确的答案?

package main

import (
	"bytes"
	"fmt"
	"io/ioutil"
	"log"
	"strings"
)

func readNames(f string) []string {
	b, err := ioutil.ReadFile(f)
	if err != nil {
		log.Fatal(err)
	}
	s := string(bytes.Replace(b, []byte(`"`), []byte(``), -1))
	return strings.Split(s, ",")
}

func totalScores(names []string) int64 {
	for i := 0; i < len(names); i++ {
		for j := i + 1; j < len(names); j++ {
			if names[i] > names[j] {
				names[i], names[j] = names[j], names[i]
			}
		}
	}

	total := int64(0)
	for i, name := range names {
		score := int64(0)
		for _, char := range name {
			score += int64(char - 'A' + 1)
		}
		score *= int64(i) + 1
		total += score
	}
	return total
}

func main() {
	names := readNames("p022_names.txt")
	total := totalScores(names)
	fmt.Println("The total is: ", total)
}

输出:

The total is:  871198282

这是 Project Euler 期望你编写的第一个版本解决方案的方式。你的第二个版本应该用更快的排序算法(如快速排序)替换简单的选择排序。

例如,你的程序花费很长时间才能产生错误的结果:

The total is:  871197995
real	0m1.945s
user	0m1.944s
sys	    0m0.004s

我的程序产生了正确的结果并且更快:

The total is:  871198282
real	0m0.771s
user	0m0.768s
sys	    0m0.004s

如果我们用 Go 的排序函数替换我的选择排序:

sort.StringSlice(names).Sort()

修改后的程序产生了正确的结果,并且速度更快:

The total is:  871198282
real	0m0.014s
user	0m0.012s
sys	    0m0.000s

Project Euler 的第22题是关于快速排序算法,而不是关于简单计算分数。

要使你的代码得到稳定的结果,请注释掉 delete 语句:

// delete(wordsValues, lowValName)

现在你只剩下浮点数错误:浮点数算术IEEE 浮点数

你的算法创建了近似的、非唯一的浮点数 wordValue。因此,对映射的随机迭代可能会选择一些不同的 lowVallowValName 对,并且可能会删除不同的映射条目。

非唯一的 wordValue

0.657666698284738
0.6576697465786885
0.6576697465786885
0.6576698865786884
0.6578786577658275
0.6578847978698485
0.658571858384738
0.6669827865826875
0.6765847269827377
0.676976698384738
0.677282738384698
0.6772827383847367
0.6772827383847367
0.677282738384738
0.677282738384738
0.6772827383847982
0.6776697769788476
0.6982786983847379
0.6986657871697675
0.7076798269786776
0.7076798269788476
0.7082657867698368
0.7082657867738368
0.7082696869827366
0.7082696869827366
0.7165668273697676
0.716979827169658
0.7169798271698486
0.716979827173658
0.716979827173658
0.716979827173658
0.7185737676698277
0.7269788273698486
0.7273766869716582
0.7465678185697674
0.746567818569769
0.746567818569769
0.7469657878698486
0.7479836980727379
0.7565847265827377
0.7565847269827377
0.7573776669827669
0.7765716865766978
0.7765826769767379
0.7765826769767379
0.7765827165826985
0.7765827165826985
0.7765827165826985
0.7765827165826985
0.7765827165827385
0.7765827165827385
0.7765827185698275
0.7773677269767378
0.827983657673787
0.8384698072657875
0.8472797765837379
0.8665766978847378
英文:

The use of floating point is suspicious, it's imprecise. The iteration order over maps is not specified and is not guaranteed to be the same from one iteration to the next. Your use of maps is the most likely explanation for seemingly random behavior.


The first question to ask is: What is the right answer?

package main
import (
&quot;bytes&quot;
&quot;fmt&quot;
&quot;io/ioutil&quot;
&quot;log&quot;
&quot;strings&quot;
)
func readNames(f string) []string {
b, err := ioutil.ReadFile(f)
if err != nil {
log.Fatal(err)
}
s := string(bytes.Replace(b, []byte(`&quot;`), []byte(``), -1))
return strings.Split(s, &quot;,&quot;)
}
func totalScores(names []string) int64 {
for i := 0; i &lt; len(names); i++ {
for j := i + 1; j &lt; len(names); j++ {
if names[i] &gt; names[j] {
names[i], names[j] = names[j], names[i]
}
}
}
total := int64(0)
for i, name := range names {
score := int64(0)
for _, char := range name {
score += int64(char - &#39;A&#39; + 1)
}
score *= int64(i) + 1
total += score
}
return total
}
func main() {
names := readNames(&quot;p022_names.txt&quot;)
total := totalScores(names)
fmt.Println(&quot;The total is: &quot;, total)
}

Output:

The total is:  871198282

This is how Project Euler expected you to write the first version of your solution. Your second version should replace the simple selection sort with a faster sort, like Quicksort.

For example, your program takes a long time to produce the wrong result:

The total is:  871197995
real	0m1.945s
user	0m1.944s
sys	    0m0.004s

My program produces the correct result and is faster:

The total is:  871198282
real	0m0.771s
user	0m0.768s
sys	    0m0.004s

If we replace my selection sort with a Go sort:

sort.StringSlice(names).Sort()

The revised program produces the correct result and it's much faster:

The total is:  871198282
real	0m0.014s
user	0m0.012s
sys	    0m0.000s

Project Euler, Names scores, Problem 22 is about fast sort algorithms, not about the trivial calculation of scores.


For a stable result from your code, comment out the delete statement:

// delete(wordsValues, lowValName)

Now you are left with the floating-point error: Floating-point arithmetic and IEEE floating point.


Your algorithm creates approximate, non-unique floating-point wordValues. Therefore, the random iteration over the map may choose some different lowVal and lowValName pairs and different map entries may be deleted.

Non-unique wordValues:

0.657666698284738
0.6576697465786885
0.6576697465786885
0.6576698865786884
0.6578786577658275
0.6578847978698485
0.658571858384738
0.6669827865826875
0.6765847269827377
0.676976698384738
0.677282738384698
0.6772827383847367
0.6772827383847367
0.677282738384738
0.677282738384738
0.6772827383847982
0.6776697769788476
0.6982786983847379
0.6986657871697675
0.7076798269786776
0.7076798269788476
0.7082657867698368
0.7082657867738368
0.7082696869827366
0.7082696869827366
0.7165668273697676
0.716979827169658
0.7169798271698486
0.716979827173658
0.716979827173658
0.716979827173658
0.7185737676698277
0.7269788273698486
0.7273766869716582
0.7465678185697674
0.746567818569769
0.746567818569769
0.7469657878698486
0.7479836980727379
0.7565847265827377
0.7565847269827377
0.7573776669827669
0.7765716865766978
0.7765826769767379
0.7765826769767379
0.7765827165826985
0.7765827165826985
0.7765827165826985
0.7765827165826985
0.7765827165827385
0.7765827165827385
0.7765827185698275
0.7773677269767378
0.827983657673787
0.8384698072657875
0.8472797765837379
0.8665766978847378

答案2

得分: 4

你对代码中的错误与浮点数有关是正确的。以这两个名字为例:

["BERNARDINE", "BERNARDINA"]

如果你简单地在这个输入集上运行你的代码,你会得到wordsValues的以下值:

map[BERNARDINE:0.6669827865826875 BERNARDINA:0.6669827865826875]

显然可以看到,由于浮点数精度损失,这两个键映射到相同的值。而且正如已经提到的,Go中的映射迭代顺序是随机的,所以你的排序算法可能会产生错误的结果(某些迭代可能会将BERNARDINE放在BERNARDINA之前,因为它们的值相同)。

由于float64支持最多15个有效数字,长度超过8个字符的名字可能会引发问题。

最好的解决方案是使用已经由@peterSO详细介绍的现有排序过程。

英文:

You are correct about the fault in your code being related to floating point numbers. Take these two names for example:

[&quot;BERNARDINE&quot;, &quot;BERNARDINA&quot;]

If you simply run your code on this input set, you get the following value for wordsValues:

map[BERNARDINE:0.6669827865826875 BERNARDINA:0.6669827865826875]

As you can clearly see, both these keys map to same value due to loss in floating point number precision. And as it has already been mentioned that map iteration order is randomized in Go, it is possible that your sorting algorithm is producing incorrect result (some iteration may put BERNARDINE before BERNARDINA since both will have same values).

Since float64 supports up to 15 significant digits, names having more than 8 characters could spell trouble.

The best solution is to use pre-existing sorting procedure as already detailed by @peterSO above.

huangapple
  • 本文由 发表于 2017年5月7日 14:09:07
  • 转载请务必保留本文链接:https://go.coder-hub.com/43828374.html
匿名

发表评论

匿名网友

:?: :razz: :sad: :evil: :!: :smile: :oops: :grin: :eek: :shock: :???: :cool: :lol: :mad: :twisted: :roll: :wink: :idea: :arrow: :neutral: :cry: :mrgreen:

确定