JavaScript - document.getElementById vs jQuery

Hi,
I am currently using document.getElementById to pass values to my AddItem function but wondering if I should use jQuery instead.

I would also like to pass data-price to my function AddItem?

Your help would be appreciated.

Thanks

		
		function AddItem(id, qty, spec1, spec2, specsfound){
			
		alert(" id "+id+" qty "+qty+" spec1 "+spec1+" spec2 "+spec2+" specsfound "+specsfound)
			
		if (!(qty > 0)) { 
		alert("Sorry this item is currently out of stock.");
		exit; 
		}	
	
		var updateURL = "/index.php?id="+id+"&action=add_item&qty="+qty+"&spec1="+spec1+"&spec2="+spec2;

		$.ajax({
			url: updateURL, success: function(data){
		
		etc
		..
		.

		}
		



<div class="curvedbox styled-select" >Size</br>
          <select class="shop_product" name="spec1_grid3" id="spec1_grid3"  style.zindex="-6">
			<option value="115" data-price="27.39">115 - $ 27.39</option>
			<option value="120" data-price="27.39">120 - $ 27.39</option>
			<option value="125" data-price="27.39">125 - $ 27.39</option>
			<option value="130" data-price="27.39">130 - $ 27.39</option>
			<option value="135" data-price="29.50">135 - $ 29.50</option>
			<option value="140" data-price="29.50">140 - $ 29.50</option>
			<option value="145" data-price="29.50">145 - $ 29.50</option>
			<option value="150" data-price="29.50">150 - $ 29.50</option>          
			</select>
          </div>
		  
         
<input name="spec2_grid3" type="hidden" id="spec2_grid3" value="">
		    


<div class="addtocart_div">
<input class="button green" name="additem" id="addtocart_grid1096" value="Add to Cart" type="button" onClick="javascript:AddItem('1096',document.getElementById('qty_grid3').value,document.getElementById('spec1_grid3').value,document.getElementById('spec2_grid3').value,'Y')">
</div>


		  
<div class="curvedbox styled-select" >Size</br>
          <select class="shop_product" name="spec1_grid4" id="spec1_grid4"  style.zindex="-6">
			<option value="115" data-price="27.39">115 - $ 27.39</option>
			<option value="120" data-price="27.39">120 - $ 27.39</option>
			<option value="125" data-price="27.39">125 - $ 27.39</option>
			<option value="130" data-price="27.39">130 - $ 27.39</option>
			<option value="135" data-price="29.47">135 - $ 29.47</option>
			<option value="140" data-price="29.47">140 - $ 29.47</option>
			<option value="145" data-price="29.47">145 - $ 29.47</option>
			<option value="150" data-price="29.47">150 - $ 29.47</option>          
			</select>
          </div>
		  
<input name="spec2_grid4" type="hidden" id="spec2_grid4" value="">


<div class="addtocart_div">
<input class="button green" name="additem" id="addtocart_grid1098" value="Add to Cart" type="button" onClick="javascript:AddItem('1098',document.getElementById('qty_grid4').value,document.getElementById('spec1_grid4').value,document.getElementById('spec2_grid4').value,'Y')">
</div>

Open in new window

sabecsAsked:
Who is Participating?
 
leakim971Connect With a Mentor PluritechnicianCommented:
replace :
<input class="button green" name="additem" id="addtocart_grid1096" value="Add to Cart" type="button" onClick="javascript:AddItem('1096',document.getElementById('qty_grid3').value,document.getElementById('spec1_grid3').value,document.getElementById('spec2_grid3').value,'Y')">

Open in new window

by :
<input class="button green" name="additem" id="addtocart_grid1096" value="Add to Cart" type="button" />

Open in new window


and replace :
function AddItem(id, qty, spec1, spec2, specsfound){
	 
		alert(" id "+id+" qty "+qty+" spec1 "+spec1+" spec2 "+spec2+" specsfound "+specsfound)

Open in new window

by :
$(document).on("click", "button[name=additem]", function() {
      var id = $(this).attr("id").replace(/\D/g, "");
     AddItem(id, $("#qty_grid3").val(),  $("#spec1_grid3").val(), $("#spec2_grid3").val(), 'Y'); // yes we jQuery instead document.getElementById
});

function AddItem(id, qty, spec1, spec2, specsfound){
			
		alert(" id "+id+" qty "+qty+" spec1 "+spec1+" spec2 "+spec2+" specsfound "+specsfound)

Open in new window

0
 
GaryCommented:
Unless you have a need for the all the other stuff jQuery can do then stick with document.getElementById
No point adding a compressed 35kb js library just for the sake of that
0
 
aboo_sCommented:
JQuery is a big library that you will have to know how to use ,but it has a lot of features and functions based on javascript that are pre made. So if you are planning on a big project with a lot of code and functions I would suggest you download and use it. But if you are only interested in passing this value you speak of ,well then stick with simple javascript ,this will do. This is also faster for the client end side(not that much though).
0
Never miss a deadline with monday.com

The revolutionary project management tool is here!   Plan visually with a single glance and make sure your projects get done.

 
Alexandre SimõesManager / Technology SpecialistCommented:
Just to add to the already said above:
jQuery is just an helper library for javascript. This means that it's just javascript anyway.
The great advantage of using jquery is that it unifies the browsers differences performing an action in the correct way based on the browser it's running on.
For instance, the simple $(document).ready(...) has to be implemented in native javascript in different ways across browsers. With jquery you call it the exact same way.
This is true for a lot of other jQuery methods. If you feel curious you can dig into the code. You'll learn a lot of things, I can guarantee you.

So addressing directly your issue and continue with the same idea as above, jQuery selector $('#myobject') directly points to document.getElementById which all browsers support. It's in fact the fastest and native way to query the DOM.
This means that in terms of performance you won't notice any difference between both and as I see you need $.ajax, you probably want to stick with some standard code rules using jquery where no performance is impacted :)
0
 
sabecsAuthor Commented:
Thanks for your comments, much appreciated.
I am already using jQuery on my website so its already available, leakim971 comments look good but just wondering how to also get value of selected  data-price also?
0
 
GaryCommented:
Do you mean data-price?

    price = $("#selector").data("price")

else to get the value

    price = $("#selector").val()
0
 
sabecsAuthor Commented:
Thanks Gary,
I am trying to get the price of the selected option.

For example so how would I get the data-price of spec1_grid3 below if 130 was selected?


<select class="shop_product" name="spec1_grid3" id="spec1_grid3"  style.zindex="-6">
			<option value="115" data-price="27.39">115 - $ 27.39</option>
			<option value="120" data-price="27.39">120 - $ 27.39</option>
			<option value="125" data-price="27.39">125 - $ 27.39</option>
			<option value="130" data-price="27.39">130 - $ 27.39</option>
			<option value="135" data-price="29.50">135 - $ 29.50</option>
			<option value="140" data-price="29.50">140 - $ 29.50</option>
			<option value="145" data-price="29.50">145 - $ 29.50</option>
			<option value="150" data-price="29.50">150 - $ 29.50</option>          
			</select>

Open in new window

0
 
GaryCommented:
As Above So Below

$("#spec1_grid3").click(function(){ // or whatever your click function is
    price = $(this).data("price")
})

Open in new window

0
 
Chris StanyonConnect With a Mentor Commented:
You can't grab the data attributes from $(this) because that's the <SELECT> and it has no data attributes. What you need to do is 'find' the selected option within the SELECT and read the data attribute from that:

price = $("#spec1_grid3").find(":selected").data('price');

Open in new window

0
 
sabecsAuthor Commented:
Thanks for all your comments and feedback, I am almost there.

Just one last thing I can't work out how to call my function using name instead of class?
I tried  
$("#additem").click(function(){
but no luck


//<input class="button green" name="additem" id="addtocart_grid_4_pid_1098" value="Add to Cart" type="button" >
		
		$(function(){
		  $(".button").click(function(){
		   
		  
		  var addtoitem =  this.id;
		  var result = addtoitem.split('pid');
		  
		  //get Product ID
		  var pid = result[1].replace(/\D/g,'');
		  alert("pid "+pid);

		  //get grid item count
		  var cnt = result[0].replace(/\D/g,'');
		  alert("cnt "+cnt);
		  
		  
		  //get selected Price
		  var price = $("#spec1_grid"+cnt).find(":selected").data('price');
		  alert("Price "+price);
		  
		  //get size selected
		  var size = $("#spec1_grid"+cnt).find(":selected").val();
		  alert("size "+size);

		  //get quantity selected
		  var qty = $("#qty_grid"+cnt).find(":selected").val();
		  alert("qty "+qty);
		  
		  });

		});

Open in new window

0
 
GaryConnect With a Mentor Commented:
$("[name=additem]").click(function(){ 

Open in new window

0
All Courses

From novice to tech pro — start learning today.