Comments on: Sacrificial Lambda

By: Jonas

Jonas — Mon, 26 Jan 2009 15:12:58 +0000

Sientz : You could set an id to a div wrapping the "containers" and simplify the hide_function and even integrate the hide and show_function by sending the object the you want to display in the funcion_calling.


hide_show_divs = function(d) {
  //HIDE ANY VISIBLE DIVS BEFORE CONTINUING
  $("#all_contents div").each(function(){
    if ($(this).is(':visible')) {
      $(this).hide('fast');
    }
  }
  $(d).show('fast');
  return false;  
}

I'm sure this can be optimized

By: Sientz

Sientz — Thu, 21 Feb 2008 06:22:49 +0000


prepare_links = function() {
prepare_links = false; ///!forgotten

Do I need to use that =false somewhere in my script? I have created a div_switch function that hides all my functions at the bottom, where would I insert the div_switch = false; if needed??


$(document).ready(function(){
  
  //THIS HIDES ALL LISTED DIVS ON LOADING OF PAGE
  $('div.blackbook').hide();
  $('div.canvas').hide();
  $('div.pencil').hide();
  $('div.murals').hide();
  $('div.shows').hide();
  $('div.tattoos').hide();
  $('div.miscellaneous').hide();
  
  $('a#blackbook').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.blackbook').show('fast');
    return false;
  });

  $('a#canvas').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.canvas').show('fast');
    return false;
  });    

  $('a#pencil').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.pencil').show('fast');
    return false;
  });    

  $('a#murals').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.murals').show('fast');
    return false;
  });    

  $('a#shows').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.shows').show('fast');
    return false;
  });    

  $('a#tattoos').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.tattoos').show('fast');
    return false;
  });    

  $('a#miscellaneous').click(function() {
    //HIDE DIVS
    hide_divs();
    //SHOW LISTED DIV
    $('.miscellaneous').show('fast');
    return false;
  });    

});

//HIDES ANY VISIBLE DIVS

hide_divs = function() {
 
    //HIDE ANY VISIBLE DIVS BEFORE CONTINUING
    if ($('div.main').is(':visible')) {
    $('div.main').hide('fast');
    }
    if ($('div.blackbook').is(':visible')) {
    $('div.blackbook').hide('fast');
    }    
    if ($('div.canvas').is(':visible')) {
    $('div.canvas').hide('fast');
    }
    if ($('div.pencil').is(':visible')) {
    $('div.pencil').hide('fast');
    }
    if ($('div.murals').is(':visible')) {
    $('div.murals').hide('fast');
    }
    if ($('div.shows').is(':visible')) {
    $('div.shows').hide('fast');
    }
    if ($('div.tattoos').is(':visible')) {
    $('div.tattoos').hide('fast');
    }
    if ($('div.miscellaneous').is(':visible')) {
    $('div.miscellaneous').hide('fast');
    }
}

If there is an easier way to do this, please feel free to educate me, I am a rookie at this scripting stuff.

By: Marat

Marat — Fri, 13 Apr 2007 13:57:58 +0000

You have forgoten next moment. To set the whole construction callback to null. Your example overflows browser buffer.
prepare_links = function() { prepare_links = false; ///!forgoten
So, what do you thing about it?

By: Jörn

Jörn — Mon, 04 Sep 2006 07:53:51 +0000

Oops, it can be written even DRYer:

do_load = function() {
$(’#calendar’).load(this.href, prepare_links);
return false;
}
prepare_links = function() {
$(’.previous-month, .next-month’).click(do_load);
}
$(prepare_links);

Heh.

By: Jörn

Jörn — Sun, 03 Sep 2006 20:37:35 +0000

You could take this one step further and make it as DRY (Do not Repeat Yourself) as possible:

PLAIN TEXT

CODE:

do_load = function() {
   $('#calendar').load(this.href, prepare_links);
   return false;
}
prepare_links = function() {
  $('.previous-month, .next-month').click(do_load);
}
$(document).ready(function() {
   prepare_links();
});

I hope that code is readable...

By: Patrick Hall

Patrick Hall — Sun, 03 Sep 2006 20:03:55 +0000

Thanks, I finally stumbled across this technique when I got sick of wrapping all the functions I wanted to run in $('document').ready() with function(){} wrappers. This way is much cleaner, it's nice to see that I'm on the right track with that.

The recursion in prepare_links is also something new to me; I didn't realize that you could self-refer to a function definition in this way. Neat. Confusing, but neat.

PS this blog is a great idea!

By: karl

karl — Sun, 03 Sep 2006 18:07:02 +0000

Hey Dan,
So, a "lambda function" is the same thing as an anonymous function, right?