common.StringContains() is exactly equal compare, but used for "contains". about gopsutil

[shirou]

Hmm, I am not so good at English. Could you tell me what kind of Function name is better for the purpose?

[gwind]

You are too modest. :-)

a poor example:

package main

import (
    "fmt"
    "strings"
)

// Check the target string slice containes src or not
func StringsHas(target []string, src string) bool {
    for _, t := range target {
        if strings.TrimSpace(t) == src {
            return true
        }
    }
    return false
}

// Does src in any string of the target string slice
func StringsFind(target []string, src string) bool {
    for _, t := range target {
        if strings.Index(t, src) >= 0 {
            return true
        }
    }
    return false
}

func main() {
    v := []string{
        "hello, c",
        "hello, unix",
        "hello, gnu",
        "hello, linux",
        "hello, go",
    }

    test_data := []string{
        "hello, c",
        "unix",
        "Java",
    }
    for _, s := range test_data {
        fmt.Printf("StringsHas(\"%s\", v) = %t\n", s, StringsHas(v, s))
        fmt.Printf("StringsFind(\"%s\", v) = %t\n", s, StringsFind(v, s))
    }
}

[shirou]

Thank you for the suggestion!

golang has the method called strings.Contains to check strings has the substr or not.

If I change your StringsFind to use strings.Contains, changing the name to StringsContains is acceptable?

Sorry for annoying you. But I can not understand well such a nuances. Really.

[gwind]

Yes, it's good. I've missed the strings.Contains .

[shirou]

I commit the change at b5ffc22. If I have miss understanding, please re-open this issue.

I am very appriciate to have discussion with you. Thank you.

[gwind]

It's a nice job, thanks for your share.