[parthmody90]

hi my send() function is simply not getting called...please help

this is my javascript section

 <script type='text/javascript'>
        function store(x,y,z) { 			
var date = x+" "+y+" "+z;
document.getElementById("date").value=date;
			

        }
		function send(){
			
		var events=document.getElementById("event").value;
	location.href="q.php?day=" + x + "&month=" + y + "&year=" + z;
		
		}

    </script>

this is my html part(button)

 <input type="button" value="Click me!" onclick="send()" />

[Martyr2]

You realize that you are using the variables "x","y" and "z" in the function store(). They are local to that function only. They are not seen in send(). Thus that line is going to error out saying that x,y,z are not defined. Unless of course you have x,y z also defined globally. But I doubt that is the case here.

This post has been edited by Martyr2: 29 January 2012 - 02:44 PM

[parthmody90]

Martyr2, on 29 January 2012 - 02:43 PM, said:

You realize that you are using the variables "x","y" and "z" in the function store(). They are local to that function only. They are not seen in send(). Thus that line is going to error out saying that x,y,z are not defined. Unless of course you have x,y z also defined globally. But I doubt that is the case here.

hey i have modified the script so that the variables become global...still it isnt working

<script type='text/javascript'>
	var myObj = {};
        myObj.store = function store(x,y,z) { 			
var date = x+" "+y+" "+z;

document.getElementById("date").value=date;
myObj.p = x;
myObj.q = y;
myObj.r = z;			

        }
		
		
		
		myObj.send = function send(){
			
		var events=document.getElementById("event").value;
location.href="q.php?day=" + p + "&month=" + q + "&year=" + r;		
		}

    </script>

[thrca]

You are not ever calling the store() function, thus, even though you have defined your x,y,z in the global scope, they are still empty.

[Jstall]

Hi,

Your syntax is a bit off. For what you want to do you would use the syntax

 object.methodName = function(){//function definition here}

You also want to use the this keyword when you are inside method definitions and are assigning a value to an object's property. Something like this:

<script type='text/javascript'>
	var myObj = {};
	myObj.store = function(x,y,z) { 			
		var date = x+" "+y+" "+z;
		document.getElementById("date").value=date;
		//you are assigning properties to your object, use this keyword
		this.p = x;
		this.q = y;
		this.r = z;			
	}
	
	myObj.send = function(){
		var events=document.getElementById("event").value;
		location.href="q.php?day=" + this.p + "&month=" + this.q + "&year=" + this.r;		
	}
	//calling the methods
	myObj.store(3,4,5);
	myObj.send();
</script>

You also declare the variable "events" inside your send method but you don't do anything with it.

[parthmody90]

its ok guys i figured it out myself...thanks for the help anyways...appreciate it!

[Jstall]

Hi,

When you find a solution to a problem you are posting about could you post your solution? This could help people searching for a solution to a similar problem

[parthmody90]

Jstall, on 30 January 2012 - 12:32 PM, said:

Hi,

When you find a solution to a problem you are posting about could you post your solution? This could help people searching for a solution to a similar problem

sure definitely!heres the solution guys:

 <script type='text/javascript'>
	
   var xx, yy, zz; //These are your global variables
   function store(x,y,z) {   
      xx = x;
      yy = y;
      zz = z;          
      var date = x+" "+y+" "+z;
      document.getElementById("date").value=date;
   }

   function send(){
      var events=document.getElementById("event").value;
      location.href="q.php?day=" + xx + "&month=" + yy + "&year=" + zz;      
   }
</script>