{"id":1361,"date":"2022-08-30T15:15:53","date_gmt":"2022-08-30T15:15:53","guid":{"rendered":"https:\/\/unknownerror.org\/index.php\/2013\/11\/13\/javascript-making-variable-an-element-collection-of-common-programming-errors\/"},"modified":"2022-08-30T15:15:53","modified_gmt":"2022-08-30T15:15:53","slug":"javascript-making-variable-an-element-collection-of-common-programming-errors","status":"publish","type":"post","link":"https:\/\/unknownerror.org\/index.php\/2022\/08\/30\/javascript-making-variable-an-element-collection-of-common-programming-errors\/","title":{"rendered":"javascript making variable an element-Collection of common programming errors"},"content":{"rendered":"

I would like to pass the name of the element i am playing with and then change its class. I call these functions from another function.<\/p>\n

   function addErrClass(field){\n        field.removeClass(\"valid\");\n        field + Info.removeClass(\"valid\");\n        field.addClass(\"error\")\n        field + Info.addClass(\"error\");\n    }\n    function addValClass(field){\n        field.removeClass(\"error\");\n        field + Info.removeClass(\"error\");\n        field.addClass(\"valid\")\n        field + Info.addClass(\"valid\");\n    }\n<\/code><\/pre>\n
I call these functions by<\/p>\n
if(data == \"yes\")\n          {\n            addErrClass('name');\n            nameInfo.text(\"Please choose another usename, \" + checkuser + \" is in use\");                \n          } else {\n             addValClass('name');\n             nameInfo.text(\"Awesome, \" + checkuser + \" is available\");  \n          }\n<\/code><\/pre>\n
each of the elements have been given a variable name:<\/p>\n
var name = $(\"#nuser\");     var nameInfo\n   = $(\"#nuserInfo\");   var email = $(\"#email\");    var emailInfo =\n   $(\"#emailInfo\");\n<\/code><\/pre>\n
error received:<\/p>\n
Uncaught TypeError: Object name has no method ‘removeClass’<\/p>\n
essentially im trying to achieve the following with each function<\/p>\n
name.addClass or email.removeClass<\/p>\n
I will be using this on lots of fields, but thought id try and keep the code to a minimum for my problem.<\/p>\n
EDITED WITH ANSWER. THANKS<\/p>\n
if(data === \"yes\")\n{\n    addErrClass(name);\n    addErrClass(nameInfo);\n    nameInfo.text(\"Please choose another usename, \" + checkuser + \" is in use\");                \n } else {\n     addValClass(name);\n     addValClass(nameInfo);\n     nameInfo.text(\"Awesome, \" + checkuser + \" is available\");  \n }\n<\/code><\/pre>\n
    \n
  1. \n
    You are passing ‘name’ which is a string into the function. Since the String type does not have a method called ‘removeClass’ you are receiving this error. This method only exists on jQuery objects.<\/p>\n
    Since you said you had already instantiated a variable called name that is a jQuery object, try this:<\/p>\n
    if(data === \"yes\")\n{\n    addErrClass(name);\n    nameInfo.text(\"Please choose another usename, \" + checkuser + \" is in use\");                \n } else {\n     addValClass(name);\n     nameInfo.text(\"Awesome, \" + checkuser + \" is available\");  \n }\n<\/code><\/pre>\n<\/li>\n
  2. \n
    You seem to be passing the string ‘name’ to your function instead of the variable name which contains your jquery object. There’s no removeClass method on the string object.<\/p>\n<\/li>\n<\/ol>\n
    Originally posted 2013-11-13 09:49:47. <\/small><\/p>","protected":false},"excerpt":{"rendered":"
    I would like to pass the name of the element i am playing with and then change its class. I call these functions from another function. function addErrClass(field){ field.removeClass(“valid”); field + Info.removeClass(“valid”); field.addClass(“error”) field + Info.addClass(“error”); } function addValClass(field){ field.removeClass(“error”); field + Info.removeClass(“error”); field.addClass(“valid”) field + Info.addClass(“valid”); } I call these functions by if(data == […]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-1361","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/1361","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/comments?post=1361"}],"version-history":[{"count":0,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/posts\/1361\/revisions"}],"wp:attachment":[{"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/media?parent=1361"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/categories?post=1361"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/unknownerror.org\/index.php\/wp-json\/wp\/v2\/tags?post=1361"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}