Question

I want to put my two functions into one.
They look like this now:

function MeYouFriend(){
    $.ajax({ 
       type: "POST",
       url: "misc/AddFriend.php",
    data: {
    mode: 'ajax',
    friend: 'ignore',
    uID : $('#uID').val(),
    fID : $('#fID').val()
    },
       success: function(msg){
$('#friendsPop').jGrowl('close');
        }
     });
}
function MeYouFriendY(){
    $.ajax({ 
       type: "POST",
       url: "misc/AddFriend.php",
    data: {
    mode: 'ajax',
        friend: 'confirm',
    uID : $('#uID').val(),
    fID : $('#fID').val()
    },
       success: function(msg){
$('#friendsPop').jGrowl('close');
        }
     });
}

The difference is only friend: ‘confirm’ and friend: ‘ignore’ , how can i put them together to one and make it simple?

Answer 1

Change your call to

MeYouFriend(true) //To Confirm
MeYouFriend(false) //To Ignore

Than change the function to accept the parameter.

function MeYouFriend(confirm){
    var c = confirm ? 'confirm' : 'ignore';

    $.ajax({ 
       type: "POST",
       url: "misc/AddFriend.php",
       data: {
          mode: 'ajax',
          friend: c,
          uID : $('#uID').val(),
          fID : $('#fID').val()
       },
       success: function(msg){
           $('#friendsPop').jGrowl('close');
        }
     });
}

Alternate Slightly More Compact Version

function MeYouFriend(confirm){
    $.ajax({ 
       type: "POST",
       url: "misc/AddFriend.php",
       data: {
          mode: 'ajax',
          friend: confirm ? 'confirm' : 'ignore',
          uID : $('#uID').val(),
          fID : $('#fID').val()
       },
       success: function(msg){
           $('#friendsPop').jGrowl('close');
        }
     });
}