How would I refactor this javascript code to avoid repetition

Explanation:

    //if the following elements don't change in the lifetime of the page
    //better if they get referenced in a scope outside the handler, that way
    //they are not fetched from the DOM every time the handler is fired
var eventFields = $('.event_fields')
  , postType = $('.new_post #post_type')
  , postOptions = $(".post_options")

    //jQuery allows multiple targets by separating them with commas. 
    //This array join method is an easy way to combine them into
    //comma-separated selectors. makes them easy to read as well     
  , selectors = [
      '#make_post_event',
      '#make_post_image',
      '#make_post_video'
    ].join(',')
  ;

//we use the joined selectors here
//if you are on jQuery 1.7, live has been deprecated. use .on()
//if possible, use delegated .on() handlers as well to bind only a single
//handler rather than have each target have it's own handler
$(selectors).on('click', function () {

      //we cache a few stuff that's used multiple times, especially $(this)
  var id = this.id
    , $this = $(this)
      //I noticed your button's only differ at the last segment
      //it's the only difference in the code of each as well
      //so we merge the code, and extract the last segment to know which
    , button = id.substring(id.lastIndexOf('_')+1)
    , active = $this.hasClass('active')
    ;

  //end use that segment accordingly
  postType.val(button);

  //i found a minor difference though, when the button is event, use toggle
  if(button === 'event'){eventFields.toggle();}
  else {eventFields.hide();}

  //other than that, everything else is just about... the same
  postOptions.removeClass('active');
  if (!active) $this.addClass('active');

});