Quelle est la meilleure façon de tester une chaîne vide dans Go?

Question

Quelle est la meilleure méthode (plus idomatique) pour tester des chaînes non vides (dans Go)?

if len(mystring) > 0 { } 

Ou:

if mystring != "" { } 

Ou autre chose?

Answer 1

Les deux styles sont utilisés dans les bibliothèques standard de Go.

if len(s) > 0 { ... } 

se trouvent dans le paquet strconv: http://golang.org/src/pkg/strconv/atoi.go

if s != "" { ... } 

se trouvent dans le paquet encoding/json: http://golang.org/src/pkg/encoding/json/encode.go

Les deux sont idiomatiques et et assez claire. C'est plus une question de goût personnel et de clarté.

Russ Cox écrit dans un golang-nuts thread:

Celui qui rend le code clair.
Si je suis sur le point de regarder l'élément x généralement j'écris
len (s)> x, même pour x == 0, mais si je me soucie de
"est-ce cette chaîne spécifique" J'ai tendance à écrire s == "".

Il est raisonnable de supposer qu'un compilateur mature compilera
len (s) == 0 et s == "" dans le même code efficace.
En ce moment, 6g etc compile s == "" dans un appel de fonction
alors que len (s) == 0 ne l'est pas, mais cela a été sur ma liste de tâches à corriger.

Effacer le code.

Answer 2

Cela semble être une microoptimisation prématurée. Le compilateur est libre de produire le même code pour les deux cas, ou au moins pour ces deux

if len(s) != 0 { ... } 

et

if s != "" { ... } 

parce que la sémantique est bien égal.

Answer 3

La vérification de la longueur est une bonne réponse, mais vous pouvez également prendre en compte une chaîne "vide" qui est également uniquement un espace. Non « techniquement » vide, mais si vous tenez à vérifier:

package main 

import (
    "fmt" 
    "strings" 
) 

func main() { 
    stringOne := "merpflakes" 
    stringTwo := " " 
    stringThree := "" 

    if len(strings.TrimSpace(stringOne)) == 0 { 
    fmt.Println("String is empty!") 
    } 

    if len(strings.TrimSpace(stringTwo)) == 0 { 
    fmt.Println("String two is empty!") 
    } 

    if len(stringTwo) == 0 { 
    fmt.Println("String two is still empty!") 
    } 

    if len(strings.TrimSpace(stringThree)) == 0 { 
    fmt.Println("String three is empty!") 
    } 
} 

Answer 4

En supposant que les espaces vides et tous les avant et arrière espaces blancs doivent être enlevés:

import "strings" 
if len(strings.TrimSpace(s)) == 0 { ... } 

Answer 5

Il serait plus propre et moins ERROR- enclins à utiliser une fonction comme celle-ci:

func empty(s string) bool { 
    return len(strings.TrimSpace(s)) == 0 
}