Simple jquery question, mostly a syntax thing I think

[mattgaunt]

Hey everyone

This is pretty simple thing and I know Im being a little bit uptight about how to go about this as I know one method that works, but I was hoping to do it my way but im new to jquery so for all I know it may not be possible.

Anyway this is the code I know that works

<script type="text/javascript">
    $(document).ready(
        function()
	{
	    $("#Prof").hide();
	}
    );
</script>

however I was kind of hoping to achieve this by doing the following

<script type="text/javascript">
    $(document).ready(
        initPage()
    );
	
    function initPage()
    {
        $("#Prof").hide();
    }
</script>

Is this possible or am I being stupid wanting to achieve this, this way?

Cheers
Matt

 

[mattgaunt]

Fixed it lol

Spent quite awhile on this and then before you know it I mucked around and found the fix lol

<script type="text/javascript">
	$(document).ready(
		function()
		{
			initPage();
		}
		);
	
	function initPage()
	{
		$("#debbyFullProf").hide();
	}
	</script>

 

[Inquisitor]

There's no need to wrap initPage in an anonymous function:

$(document).ready(initPage);

function initPage()
{
	$("#debbyFullProf").hide();
}

It's better practice to use the first way of doing it that you posted, though. Declaring a named function is redundant and pollutes the global namespace unnecessarily.

 

[suarve]

Hey everyone

This is pretty simple thing and I know Im being a little bit uptight about how to go about this as I know one method that works, but I was hoping to do it my way but im new to jquery so for all I know it may not be possible.

Anyway this is the code I know that works

<script type="text/javascript">
    $(document).ready(
        function()
    {
        $("#Prof").hide();
    }
    );
</script>

however I was kind of hoping to achieve this by doing the following

<script type="text/javascript">
    $(document).ready(
        initPage()
    );
    
    function initPage()
    {
        $("#Prof").hide();
    }
</script>

Is this possible or am I being stupid wanting to achieve this, this way?

Cheers
Matt

Surely your first way is best?

 

[mattgaunt]

Well thats what i was thinking but it didn't seem to work, soo just had to go with what worked

 

[Inquisitor]

Huh? The first code snippet in your original post works and is the best one to use! Declaring a separate initPage function is unnecessary and could be considered bad practice.

 

[Pho]

In the first one you use "#Prof" but in the second "#debbyFullProf" maybe that was why? .