А вы попробуйте структурировать код чуть более читабельным способом, тем более пока учитесь. Для примера:
(function(){
var someParentElement = $('.some-selector');
if(!someParentElement.length) return;
var $mapElement = $('#map'),
$mapOpenBtn = someParent.find('button.map-open'),
$mapCloseBtn = someParent.find('button.map-close'),
TXT_OPEN = 'открыть схему проезда',
TXT_CLOSE = 'закрыть схему проезда';
function closeMap(){
$mapElement.fadeOut(400);
$mapCloseBtn.text(TXT_OPEN);
mapCloseBtn.addClass('map-close').removeClass('map-open');
}
function openMap(){
$mapElement.fadeIn(400);
$mapCloseBtn.text(TXT_CLOSE);
$mapCloseBtn.addClass('map-open').removeClass('map-close');
}
mapOpenBtn.click(openMap);
mapCloseBtn.click(closeMap);
})();
И вы легко сможете понять где у вас проблемы.