Сергей

// $(function(){}) и $(document).ready(function(){}) одно и то же
// Нет смысла вкладывать их друг в друга
$(function () {
    $(document).ready(function () {

        // Если informer может принимать только значения 0 и 1,
        // гораздо логичнее сделать эту переменную булевой
        var informer = 0;
        
        // Крутые парни ставят точки с запятой в конце выражений
        // Конечно, если не хотят потом ловить баги automatic semicolon insertion по ночам
        var sfId_Start
        var sfId_Target

        console.log(informer)

        // Зачем у td два обработчика клика?
        $('td').click(function () {
            init()
                
            ++informer;
            if (informer > 1) {
                informer = 0
            }


            //  if (informer === 1 ) {   }
            console.log(informer)

        });

        function init() {

            // Зачем делать анонимную функцию, в которой просто вызывается другая функция?
            $('td').click(function () {
                getId(this);
            });

            // Зачем эта функция сюда вложена?
            function getId(obj) {

                var idsf
                                
                // Этот код не имеет смысла
                // Почему при любом условии в переменную idsf пишется одно и то же значение?
                if (informer === 0) {
                    idsf = $(obj).attr('id')
                }
                
                // Зачем проверять, что 1 не равно 0?
                if (informer === 1 && informer !== 0) {
                    idsf = $(obj).attr('id')
                }

                // Зачем писать одно и то же в обе переменные?
                sfId_Start = idsf
                sfId_Target = idsf
            }
        }
    })
});