String equality issue in C#

Question

I have the following if-else statement in a C# method (which returns a JSON object). The else statement executes even though the condition for the if statement (language == "en") is true. Why is this? The method is called by the following line(s) of code in another file:

Ux.locale.Manager.setConfig({
        ajaxConfig : {
            method : 'GET'
        },
        language   : 'en',
        tpl        : 'getLocale.castle?language={locale}',
        type       : 'ajax'
    });

The language value is combined with the tpl value to produce the URL that calls the method (in my case: getLocale.castle?language=en).

    [return: JSONReturnBinder]
     public Locale GetLocale(string language)
     {
        if (language == "en")
        {
           Locale englishLang = new Locale();
           englishLang.region.center.title = "Center Region";
           englishLang.region.east.title = "East Region - Form";
           englishLang.buttons.save = "Save";
           englishLang.fields.labels.firstName = "First Name";
           englishLang.fields.labels.lastName = "Last Name";
           englishLang.fields.labels.chooseLocale = "Choose Your Locale";       
           return englishLang;
       } else {
            Locale frenchLang = new Locale();
            frenchLang.region.center.title = "Region Centre";
            frenchLang.region.east.title = "Region Est - Formulaire";
            frenchLang.buttons.save = "Enregistrer";
            frenchLang.fields.labels.firstName = "Prenom";
            frenchLang.fields.labels.lastName = "Nom";
            frenchLang.fields.labels.chooseLocale = "Choisissez vos paramètres régionaux";
            return frenchLang; 
       }
}

I have tried using if (language.Equals("en")) but even then the else statement executes and not the if statement.

Are you sure that the value of language inside c# code is "en"? — Alessandro D'Andria
– Alessandro D'Andria, Commented Mar 17, 2014 at 15:50
And language is guaranteed to be "en"? I mean, you double-checked this? — JensG
– JensG, Commented Mar 17, 2014 at 15:50
Are you 100% sure this is the correct way of passing that argument to the method? Can you set a breakpoint inside the method to verify that it isn't null? — p.s.w.g
– p.s.w.g, Commented Mar 17, 2014 at 15:50
I do recommend to add verbose logging to the GetLocale method — alex.b
– alex.b, Commented Mar 17, 2014 at 15:51

James · Accepted Answer · 2014-03-17 15:51:01Z

1

The else statement executes even though the condition for the if statement (language == "en") is true. Why is this?

Most likely cause? language does not equal "en". Make sure you don't have any extra whitespace in your variable (common pitfall and can sometimes be unapparent to the human eye when inspecting the debugger) and check the casing is correct.

For example, try

language.Trim().ToLower() == "en"

and chances are your problem will dissappear.

answered Mar 17, 2014 at 15:51

James

82.4k19 gold badges172 silver badges245 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

anitacynax · Accepted Answer · 2014-03-18 08:53:11Z

0

The problem was that the variable language was used twice (once in language property in the line language : 'en' and a second time in the line tpl: 'getLocale.castle?language={locale}'. This caused confusion when the language parameter was passed to the getLocale method (the correct language parameter was not being passed). When I changed the name of the language parameter, the problem was solved and the if-statement was executed.

answered Mar 18, 2014 at 8:53

anitacynax

2511 gold badge7 silver badges14 bronze badges

Collectives™ on Stack Overflow

String equality issue in C#

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related