Code ausführen sonst weiter

[pat01]

Hi
Ich möchte, dass im untenstehenden Code

if (document.form.button2.value=2) ;{
document.location.href="/pfad/seite.html?navid=4";
}

ausgeführt wird falls es einen button2 mit Wert 2 gibt. Falls nicht, soll der Rest vom Script abgearbeitet werden. Ich erhalte aber immer einen Syntaxfehler, oder wenn ich 'else' entferne funzt das Teil nicht. Hat jemand eine Idee was ich hier falsch mache?


function changeLocationNavigation(navId, url, target){
if (document.form.button2.value=2) ;{
document.location.href="/pfad/seite.html?navid=4";
}
else
document.cookie = "wNavId=" + navId + "; path=/";
var hasParameters = /\?/;
if(url != ""){
if(url == "#"){
url = document.location.href;
url = url.replace(/[\&|\?]navid=\d+/i, "");
}
{
}
if (hasParameters.test(url) == false){
url = url + '?navId=' + navId;
}
else{
url = url + '&navId=' + navId;
}
if (target != "blank"){
top.location.href = url;
}
else{
window.open(url);
}
}
}

 

[Albu]

überprüf mal Deine {} Klammern!!!

 

[pat01]

Ich habe wohl eine Mattscheibe - ich bekomms einfach nicht hin.

 

[Bjoern]

*räusper*

if (document.form.button2.value=2 ) ;{
document.location.href="/pfad/seite.html?navid=4";
}

Der Vergleichsoperator ist ==

 

[Albu]

wo ich den Code vom Bjoern seh....
ein ; hinter einer if ist _nie_ eine gute Idee!!!!

im Allgemeinen: sortiere Deinen Code mal ordentlich oder besser formatiere ihn gescheit.... das kann ja kein Schwein lesen... und dabei ist das "nur" ein kurzes Skript

 

[Bjoern]

Da sind sowieso überall Syntaxfehler.

else
document.cookie = "wNavId=" + navId + "; path=/";
var hasParameters = /\?/;
if(url != ""){
if(url == "#"){
url = document.location.href;
url = url.replace(/[\&|\?]navid=\d+/i, "");
}
{
}

Allein das ist gut.
Keine Klammern um den Else-Block und ganz unten wird ein Anweisungsblock völlig ohne Sinn und Verstand geöffnet und geschlossen.

 

[pat01]

Hmmm... hat sich eh erledigt, der 'button2' wird immer mitgeschickt, also nützt das nix.
Ich habe dann mit

function changeLocationNavigation(navId, url, target){
document.cookie = "wNavId=" + navId + "; path=/";
var hasParameters = /\?/;
if(url != ""){
if(url == "#"){
url = document.location.href;
url = url.replace(/[\&|\?]navid=\d+/i, "");
}
{
}
if (navId == 2); document.location.href="/pfad/seite.html?navid=4";
if (hasParameters.test(url) == false){
url = url + '?navId=' + navId;
}
else{
url = url + '&navId=' + navId;
}
if (target != "blank"){
top.location.href = url;
}
else{
window.open(url);
}
}
}

versucht die '2' abzufragen, bzw. durch eine '4' zu ersetzen. Aber das funzt auch nicht. Ist das überhaupt möglich?

In der Seite steht:
<a href="javascript:changeLocationNavigation('2','/pfad/seite.html','');">

aber eben diese 2 sollte eine vier sein. Das Script generiert übrigens mehrere solche Links - ich kann also nicht einfach

url = url + '?navId=' + navId;

durch

url = url + '?navId=' + 2;

ersetzen...