language.MatchStrings()返回无效内容问题及Go代码求助

Let's dig into why your language.MatchStrings() call is returning wonky results—there are a few clear issues in your code that are likely causing this, plus some easy fixes:

1. Critical Variable Typo Breaks Cookie Handling #

First up, you've got a variable name mismatch that's making your cookie logic fail entirely:

lang, _ := r.Cookie("lang")
// ...
if cookie != nil { // ❌ 'cookie' isn't defined anywhere!
    cookieValue = lang.String()
}

This means cookieValue always stays an empty string, so MatchStrings() is only using the Accept-Language header (or falling back to your default if that's unmatchable). Even if the cookie exists, your code never reads its value.

2. You're Using the Wrong Cookie Property #

On top of the typo, you're calling lang.String() to get the cookie value—but r.Cookie() returns an http.Cookie struct. The String() method for this struct outputs the full cookie line (like lang=fr; Path=/; HttpOnly), not just the language value you need. That garbage input is definitely throwing off the language matcher.

3. Ignoring Errors Hides Debug Info #

You're discarding errors from both r.Cookie() and language.MatchStrings() with _. These errors could tell you exactly what's broken—like if the cookie doesn't exist, or the language tags in the header/cookie are malformed. Always log these during debugging!

Fixed & Improved Code #

Here's the cleaned-up version with all the fixes, plus better error handling:

var languageMatcher = language.NewMatcher([]language.Tag{
    language.English, // Fallback to English if no matches
    language.French,
})

// Get the lang cookie with proper error checking
langCookie, err := r.Cookie("lang")
var cookieValue string
if err == nil { // No error means the cookie exists
    cookieValue = langCookie.Value // Use the actual cookie value, not the full cookie string
}

accept := r.Header.Get("Accept-Language")
tag, matchErr := language.MatchStrings(languageMatcher, cookieValue, accept)
if matchErr != nil {
    log.Errorf(ctx, "Failed to match language: %v", matchErr)
    // Fall back to our default language if matching fails
    tag = language.English
}

log.Debugf(ctx, "Localized %s to %s. cookie: %s | header: %s", path, tag.String(), cookieValue, accept)
// Proceed with http.Redirect...

What Changed: #

• Fixed Variable Typo: Renamed the cookie variable to langCookie (clearer naming) and checked the correct variable for existence.
• Used Correct Cookie Value: Switched from lang.String() to langCookie.Value to get just the language code from the cookie.
• Added Error Handling: Now we log matching errors and fall back to the default language if something goes wrong.
• Cleaner Naming: Variables are more descriptive, making it easier to follow the logic later.

This should resolve the invalid results from language.MatchStrings()—the cookie value issue was probably the biggest culprit.

内容的提问来源于stack exchange，提问作者Maxime VAST