Показать сообщение отдельно
  #1 (permalink)  
Старый 07.01.2015, 09:16
Интересующийся
Отправить личное сообщение для erilar Посмотреть профиль Найти все сообщения от erilar
 
Регистрация: 07.01.2015
Сообщений: 11

Оптимизация кода
есть рабочий код:
$(document).ready(function(){
    $('body').on('click','.song', function(){
        var clkid = this.id; 

            $.ajax({ 
                type: "POST",
                url: "/cont/song.php",
                data: "id="+clkid,
                success: function(html){ 
                $("section").html(html);
                history.pushState(null, null, "?option=main&cont=song&id="+clkid);
                },beforeSend: function(){                       
                $("#loader").css("display", "block");
            }
        }).done(function(data){                
            $("#loader").css("display", "none");
            window.scrollTo(0,580);  
            });
        return false; 
    });
});


Хочу его улучшить, т.к. подобных функций в файле несколько. Сделал так, но не работает. Помогите найти ошибку:
var post_url;
var data_id;
var hist_url;
var clkid; 

$(document).ready(function(){    
    $('body').on('click','song', function(){
        post_url = '/cont/song.php';
        data_id  = 'id=';
        hist_url = '?option=main&cont=song&id=';
        clkid    = this.id; 
        post_act(post_url,data_id,hist_url,clkid);
return false;
    });  
});


function post_act(){      
        
    $.ajax({ 
        type: "POST",
        url: post_url,
        data: data_id+clkid,
        success: function(html){ 
        $("section").html(html);
        history.pushState(null, null, hist_url+clkid);
        },beforeSend: function(){                       
        $("#loader").css("display", "block");
        }
    }).done(function(data){                
        $("#loader").css("display", "none");
        window.scrollTo(0,580);  
        });
      
}
Ответить с цитированием