-2
if(typeof    (document.getElementById("courseId").value!=="undefined") || document.getElementById("courseId").value!==null)
    {
        Courseid = document.getElementById("courseId").value;
    } 
MrCode
  • 61,589
  • 10
  • 82
  • 110
Matrix
  • 83
  • 1
  • 1
  • 7

3 Answers3

9

rewrite it that way:

if(document.getElementById("courseId") && document.getElementById("courseId").value)
{
    CourseId = document.getElementById("courseId").value;
} 
Marco Forberg
  • 2,617
  • 5
  • 21
  • 33
2

If you explicitly want to check for undefined and null you can do

if(document.getElementById('courseId') === null || 
   document.getElementById('courseId') === undefined) {
    //logic
}
glosrob
  • 6,547
  • 2
  • 40
  • 68
0

An input's value will never be null or undefined - it will be the empty string, "".

Eric
  • 91,378
  • 50
  • 226
  • 356
  • `document.getElementById("myInput").value.typeof` returns `undefined` for me. Even when I assign a value to it by typing in the input box like typing `20` it still returns `undefined`. – Shayan Mar 03 '19 at 15:22