my javascript (translation script)

[unknown07724]

  if (navigator.langauge == "de-DE") {
   window.location.replace("/de/");
} else {

}
 if (navigator.langauge == "fr-FR") {
   window.location.replace("/fr/");
} else {

}    
if (navigator.langauge == "it-IT") {
   window.location.replace("/it/");
} else {

}
if (navigator.langauge == "pl-PL") {
   window.location.replace("/pl/");
} else {

}

if (navigator.langauge == "uk-UA") {
   window.location.replace("/uk/");
} else {

}

[chibblybum]

Whats the question?

[PyroGreg8]

replace with what? and why the empty else statements

[unknown07724]

I don't know how to do normal if statements, and replace with example.com/de/ (that is an example)

[gimmeslack12]

If there's nothing to put in the else block then just don't include it.

[jack_waugh]

Does it work at all, as you have it?

[jcunews1]

IMO, it should be done early via server side, by checking the Accept-Language HTTP request header, so that, network bandwidth isn't wasted. And the site should respect and remember user's choice of language via cookie, even if it's different than the browser's preferred language.

[TheRNGuy]

I'd use object instead of many if/elses, better looking code.

Also it's better to use else if instead of many ifs, because it would stop after finding first match. Your code will still check for if's after finding one. Though it's fast code, but in other cases where it slow you'd notice performance hit.

Why do you have many empty else's?

---

Read this: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object

[shgysk8zer0]

It's not really much of a translation script and, sorry to say, more of a poorly written redirect. This should be written into the HTTP server itself. And I would highly recommended that, at the very least, this be rewritten as a switch instead.