CODEWITHSUNDEEP
goslicestrconv
user3015224
user3015224

Reputation: 27

Golang parse array

I am trying to figure out why my code is not working. I wish to take a slice of numbers and strings, and separate it into three slices. For each element in the slice, if it is a string, append it to the strings slice, and if it is a positive number, append it to the positive numbers, and likewise with negative. Yet, here is the output

Names: EvTremblay 45.39934611083154 -75.71148292845268

[Crestview -75.73795670904249 BellevueManor -75.73886856878032 Dutchie'sHole -75.66809864107668 ...

Positives:[45.344387632924054 45.37223315413918 ... ] Negatives: []

Here is my code. Can someone tell me what is causing the Negatives array to not have any values?

func main() {
    fmt.Printf("%q\n", strings.Split("a,b,c", ","))
    var names []string
    var positives, negatives []float64
    bs, err := ioutil.ReadFile("poolss.txt")
    if err != nil {
        return
    }
    str := string(bs)
    fmt.Println(str)
    tokens := strings.Split(str, ",")
    for _, token := range tokens {
        if num, err := strconv.ParseFloat(token, 64); err == nil {
            if num > 0 {
                positives = append(positives, num)
            } else {
                negatives = append(negatives, num)
            }

        } else {
            names = append(names, token)
        }

    fmt.Println(token)
    }

    fmt.Println(fmt.Sprintf("Strings: %v",names))
    fmt.Println(fmt.Sprintf("Positives: %v", positives))
    fmt.Println(fmt.Sprintf("Negatives: %v",negatives))
    for i := range names{
        fmt.Println(names[i])

        fmt.Println(positives[i])

        fmt.Println(negatives[i])
    }
}

Upvotes: 0

Views: 4194

Answers (3)

user3015224
user3015224

Reputation: 27

Here's what I tried, it works now! Thanks for the help, everyone!

func main() {
findRoute("poolss.csv", 5)
 }

func findRoute( filename string, num int) []Edge {
var route []Edge
csvFile, err := os.Open(filename)
if err != nil {
    return route
}
reader := csv.NewReader(bufio.NewReader(csvFile))
var pools []Pool
for {
    line, error := reader.Read()
    if error == io.EOF {
        break
    } else if error != nil {
        log.Fatal(error)
    }
    lat, err := strconv.ParseFloat(line[1], 64)
    long, err := strconv.ParseFloat(line[2], 64)
    if err == nil {
        pools = append(pools, Pool{
            name: line[0],
            latitude:  lat,
            longitude: long,
        })
    }
}
return route
}

Upvotes: 0

Kenny Grant
Kenny Grant

Reputation: 9623

Your problem above I think lies with the extra \n attached to each float probably - you get no negative entries if you end in a linefeed or you would get one if you have no linefeed at the end. So insert a printf so that you can see the errors you're getting from strconv.ParseFloat and all will become clear.

Some small points which may help:

  • Check errors, and don't depend on an error to be of only one type (this is what is confusing you here) - always print the error if it arrives, particularly when debugging
  • Don't use the name of a package for a variable (strings), it won't end well
  • Use a datastructure which reflects your data
  • Use the CSV package to read CSV data

So for example for storing the data you might want:

type Place struct {
   Name string 
   Latitude int64
   Longitude int64
}

Then read the data into that, depending on the fact that cols are in a given order, and store it in a []Place.

Upvotes: 1

peterSO
peterSO

Reputation: 166569

Your code has strings as a variable name:

var strings []string

and strings as a package name:

tokens := strings.Split(str, ",")

Don't do that!

strings.Split undefined (type []string has no field or method Split)

Playground: https://play.golang.org/p/HfZGj0jOT-P

Upvotes: 2

Related Questions