Javascript - si / y / si no funciona - javascript, if-statement

Primero que nada ... Soy bastante nuevo en javascript ...

Aquí está el código que estoy usando actualmente, sin embargo, no parece estar funcionando correctamente. Esto es lo que estoy tratando de lograr ...

Si algo tiene el valor de "composición tipográfica" de "Completoset "y su valor" principal "es cualquier cosa menos" A2 tándem ", entonces su variable de título debe ser" Line Set w / ALinks: ". Si tiene cualquier otro valor" typeset "o es" s "main "el valor es" A2 tándem ", entonces la variable de título debe ser" Conjunto de líneas: "

¿Qué está pasando actualmente? No parece estar diferenciando entre el valor "principal" que es "A2 tándem". Aún está aplicando el título "w / ALinks:" si el valor "principal" es "A2 tándem".

¿Ayuda?

    if (accCat == "Line Sets"){
document.getElementById("o1").value = document.getElementById("main").value;
document.getElementById("o2").value = document.getElementById("mainsize").value;
document.getElementById("o3").value = document.getElementById("typeset").value;
document.getElementById("o4").value = document.getElementById("typeline").value;

var canopyValue = document.getElementById("main").value;
var title;

if ((document.getElementById("typeset").value == "Complete set") && (canopyValue != "A2 tandem")){
title = "Line Set w/ ALinks: ";
}else{
title = "Line Set: ";
}

Respuestas

2 para la respuesta № 1

if (accCat == "Line Sets"){ //<-------- YOU DIDN"T CLOSE THIS
document.getElementById("o1").value = document.getElementById("main").value;
document.getElementById("o2").value = document.getElementById("mainsize").value;
document.getElementById("o3").value = document.getElementById("typeset").value;
document.getElementById("o4").value = document.getElementById("typeline").value;

var canopyValue = document.getElementById("main").value;
var title;

if ((document.getElementById("typeset").value == "Complete set") && (canopyValue != "A2 tandem")){
title = "Line Set w/ ALinks: ";
}else{
title = "Line Set: ";
}

Agrega un } hasta el final del código.

También un consejo rápido:

En lugar de escribir document.getElementById("someId").value cada vez, simplemente cree una función que obtenga el elemento.

var get = function(id){
return document.getElementById(id);
}

Entonces puedes acceder a él como get("o1").value = get("main").value

---

1 para la respuesta № 2

Le faltan corchetes en su primera if. Lo abrió, pero el corchete de cierre no aparece hasta después de su else.

---

1 para la respuesta № 3

No cerró el corchete de apertura del si inicial. De todos modos, como eres nuevo en Javascript, considera que Javascript tiene 2 operadores para IGUALES. Echa un vistazo a http://www.w3schools.com/js/js_comparisons.asp

Dado que x = 5

== valor igual

x == 8 da falso
x == 5 da verdadero

=== igual valor y tipo igual

x === "5" falso x === 5 verdadero

---

0 para la respuesta № 4

No cerraste el original if. Agrega un } hasta el final.

---