Finding pairs that add up to a given sum, using recursion












0












$begingroup$


I'm currently tasked with using recursion to find pairs that add up to a given sum. How would I make the function recursiveFixedSumPairs function more efficient and truncate this code? Also, what is the current runtime? I believe it is n since it will go through the array only once always starting at the first index.



public class HW4 {



    public static void main(String args) {



        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};

        int k;

        k = 43;

        System.out.println("k = " + k);

        findFixedSumPairs(a, k);

    }



    public static void findFixedSumPairs(int a, int k) {

       recursiveFixedSumPairs(a, -1, 0, k);

    }



    private static void recursiveFixedSumPairs(int array, int subPair1, int index, int k) {

//        // pairs whose sum equals k are printed in this method

//        if ((array[subPair1] + array[subPair2]) == k) {

//            System.out.println("[" + array[subPair1] + ", " + array[subPair2] + "]");

//        } else if ((array[subPair1] + array[subPair2]) > k) {

//            recursiveFixedSumPairs(array, subPair1, subPair2 - 1, k);

//        } else {

//            recursiveFixedSumPairs(array, subPair1 + 1, subPair2, k);

//        }



            if (index == array.length) {

                return;

            }



            else if (index == array.length-1) {



                if (subPair1!= -1 && subPair1 + array[index] == k) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                }



                else {

                    return;

                }

            }



            if (subPair1 != -1) {



                if (array[index] == k- subPair1) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                    return;

                } else {

                    recursiveFixedSumPairs(array, subPair1, index + 1, k);

                }



            } else {

                recursiveFixedSumPairs(array, array[index], index+1, k);

                recursiveFixedSumPairs(array,-1, index+1, k);

            }

        }

}





java recursion complexity k-sum







share|improve this question











$endgroup$



migrated from stackoverflow.com 2 hours ago


This question came from our site for professional and enthusiast programmers.














  • 2




    $begingroup$
    Does this code work? If so, Code Review might be a better place to post this.
    $endgroup$
    – Kevin Anderson
    3 hours ago
















0












$begingroup$


I'm currently tasked with using recursion to find pairs that add up to a given sum. How would I make the function recursiveFixedSumPairs function more efficient and truncate this code? Also, what is the current runtime? I believe it is n since it will go through the array only once always starting at the first index.



public class HW4 {



    public static void main(String args) {



        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};

        int k;

        k = 43;

        System.out.println("k = " + k);

        findFixedSumPairs(a, k);

    }



    public static void findFixedSumPairs(int a, int k) {

       recursiveFixedSumPairs(a, -1, 0, k);

    }



    private static void recursiveFixedSumPairs(int array, int subPair1, int index, int k) {

//        // pairs whose sum equals k are printed in this method

//        if ((array[subPair1] + array[subPair2]) == k) {

//            System.out.println("[" + array[subPair1] + ", " + array[subPair2] + "]");

//        } else if ((array[subPair1] + array[subPair2]) > k) {

//            recursiveFixedSumPairs(array, subPair1, subPair2 - 1, k);

//        } else {

//            recursiveFixedSumPairs(array, subPair1 + 1, subPair2, k);

//        }



            if (index == array.length) {

                return;

            }



            else if (index == array.length-1) {



                if (subPair1!= -1 && subPair1 + array[index] == k) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                }



                else {

                    return;

                }

            }



            if (subPair1 != -1) {



                if (array[index] == k- subPair1) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                    return;

                } else {

                    recursiveFixedSumPairs(array, subPair1, index + 1, k);

                }



            } else {

                recursiveFixedSumPairs(array, array[index], index+1, k);

                recursiveFixedSumPairs(array,-1, index+1, k);

            }

        }

}





java recursion complexity k-sum







share|improve this question











$endgroup$



migrated from stackoverflow.com 2 hours ago


This question came from our site for professional and enthusiast programmers.














  • 2




    $begingroup$
    Does this code work? If so, Code Review might be a better place to post this.
    $endgroup$
    – Kevin Anderson
    3 hours ago














0












0








0





$begingroup$


I'm currently tasked with using recursion to find pairs that add up to a given sum. How would I make the function recursiveFixedSumPairs function more efficient and truncate this code? Also, what is the current runtime? I believe it is n since it will go through the array only once always starting at the first index.



public class HW4 {



    public static void main(String args) {



        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};

        int k;

        k = 43;

        System.out.println("k = " + k);

        findFixedSumPairs(a, k);

    }



    public static void findFixedSumPairs(int a, int k) {

       recursiveFixedSumPairs(a, -1, 0, k);

    }



    private static void recursiveFixedSumPairs(int array, int subPair1, int index, int k) {

//        // pairs whose sum equals k are printed in this method

//        if ((array[subPair1] + array[subPair2]) == k) {

//            System.out.println("[" + array[subPair1] + ", " + array[subPair2] + "]");

//        } else if ((array[subPair1] + array[subPair2]) > k) {

//            recursiveFixedSumPairs(array, subPair1, subPair2 - 1, k);

//        } else {

//            recursiveFixedSumPairs(array, subPair1 + 1, subPair2, k);

//        }



            if (index == array.length) {

                return;

            }



            else if (index == array.length-1) {



                if (subPair1!= -1 && subPair1 + array[index] == k) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                }



                else {

                    return;

                }

            }



            if (subPair1 != -1) {



                if (array[index] == k- subPair1) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                    return;

                } else {

                    recursiveFixedSumPairs(array, subPair1, index + 1, k);

                }



            } else {

                recursiveFixedSumPairs(array, array[index], index+1, k);

                recursiveFixedSumPairs(array,-1, index+1, k);

            }

        }

}





java recursion complexity k-sum







share|improve this question











$endgroup$




I'm currently tasked with using recursion to find pairs that add up to a given sum. How would I make the function recursiveFixedSumPairs function more efficient and truncate this code? Also, what is the current runtime? I believe it is n since it will go through the array only once always starting at the first index.



public class HW4 {



    public static void main(String args) {



        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};

        int k;

        k = 43;

        System.out.println("k = " + k);

        findFixedSumPairs(a, k);

    }



    public static void findFixedSumPairs(int a, int k) {

       recursiveFixedSumPairs(a, -1, 0, k);

    }



    private static void recursiveFixedSumPairs(int array, int subPair1, int index, int k) {

//        // pairs whose sum equals k are printed in this method

//        if ((array[subPair1] + array[subPair2]) == k) {

//            System.out.println("[" + array[subPair1] + ", " + array[subPair2] + "]");

//        } else if ((array[subPair1] + array[subPair2]) > k) {

//            recursiveFixedSumPairs(array, subPair1, subPair2 - 1, k);

//        } else {

//            recursiveFixedSumPairs(array, subPair1 + 1, subPair2, k);

//        }



            if (index == array.length) {

                return;

            }



            else if (index == array.length-1) {



                if (subPair1!= -1 && subPair1 + array[index] == k) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                }



                else {

                    return;

                }

            }



            if (subPair1 != -1) {



                if (array[index] == k- subPair1) {

                    System.out.println(""+subPair1 +" "+ array[index]);

                    return;

                } else {

                    recursiveFixedSumPairs(array, subPair1, index + 1, k);

                }



            } else {

                recursiveFixedSumPairs(array, array[index], index+1, k);

                recursiveFixedSumPairs(array,-1, index+1, k);

            }

        }

}





java recursion complexity k-sum




java recursion complexity k-sum






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited 4 mins ago









200_success

129k15153416




129k15153416










asked 3 hours ago







FanonX











migrated from stackoverflow.com 2 hours ago


This question came from our site for professional and enthusiast programmers.









migrated from stackoverflow.com 2 hours ago


This question came from our site for professional and enthusiast programmers.










  • 2




    $begingroup$
    Does this code work? If so, Code Review might be a better place to post this.
    $endgroup$
    – Kevin Anderson
    3 hours ago














  • 2




    $begingroup$
    Does this code work? If so, Code Review might be a better place to post this.
    $endgroup$
    – Kevin Anderson
    3 hours ago








2




2




$begingroup$
Does this code work? If so, Code Review might be a better place to post this.
$endgroup$
– Kevin Anderson
3 hours ago




$begingroup$
Does this code work? If so, Code Review might be a better place to post this.
$endgroup$
– Kevin Anderson
3 hours ago










1 Answer
1






active

oldest

votes


















0












$begingroup$

You made it too hard:



public class HW4 {



    public static void main(String args) {



        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};

        int k;

        k = 43;

        System.out.println("k = " + k);

        findSumPairs(a, k, 0);

    }





    private static void findSumPairs(int array, int target, int index) {

        for (int i=index+1; i<array.length; i++)

        {

            if (array[index] + array[i] == target)

            {

                System.out.println(array[index] + " " + array[i]);

            }

        }

        if (index < array.length -1)

        {

            findSumPairs(array, target, ++index);

        }

    }

}





share|improve this answer









$endgroup$













    Your Answer





    StackExchange.ifUsing("editor", function () {
    return StackExchange.using("mathjaxEditing", function () {
    StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
    StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
    });
    });
    }, "mathjax-editing");

    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "196"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: false,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: null,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f213929%2ffinding-pairs-that-add-up-to-a-given-sum-using-recursion%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown
























    1 Answer
    1






    active

    oldest

    votes








    1 Answer
    1






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    0












    $begingroup$

    You made it too hard:



    public class HW4 {
    

    
    public static void main(String args) {
    

    
        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};
    
        int k;
    
        k = 43;
    
        System.out.println("k = " + k);
    
        findSumPairs(a, k, 0);
    
    }
    

    

    
    private static void findSumPairs(int array, int target, int index) {
    
        for (int i=index+1; i<array.length; i++)
    
        {
    
            if (array[index] + array[i] == target)
    
            {
    
                System.out.println(array[index] + " " + array[i]);
    
            }
    
        }
    
        if (index < array.length -1)
    
        {
    
            findSumPairs(array, target, ++index);
    
        }
    
    }
    
}





    share|improve this answer









    $endgroup$


















      0












      $begingroup$

      You made it too hard:



      public class HW4 {
      

      
    public static void main(String args) {
      

      
        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};
      
        int k;
      
        k = 43;
      
        System.out.println("k = " + k);
      
        findSumPairs(a, k, 0);
      
    }
      

      

      
    private static void findSumPairs(int array, int target, int index) {
      
        for (int i=index+1; i<array.length; i++)
      
        {
      
            if (array[index] + array[i] == target)
      
            {
      
                System.out.println(array[index] + " " + array[i]);
      
            }
      
        }
      
        if (index < array.length -1)
      
        {
      
            findSumPairs(array, target, ++index);
      
        }
      
    }
      
}





      share|improve this answer









      $endgroup$
















        0












        0








        0





        $begingroup$

        You made it too hard:



        public class HW4 {
        

        
    public static void main(String args) {
        

        
        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};
        
        int k;
        
        k = 43;
        
        System.out.println("k = " + k);
        
        findSumPairs(a, k, 0);
        
    }
        

        

        
    private static void findSumPairs(int array, int target, int index) {
        
        for (int i=index+1; i<array.length; i++)
        
        {
        
            if (array[index] + array[i] == target)
        
            {
        
                System.out.println(array[index] + " " + array[i]);
        
            }
        
        }
        
        if (index < array.length -1)
        
        {
        
            findSumPairs(array, target, ++index);
        
        }
        
    }
        
}





        share|improve this answer









        $endgroup$



        You made it too hard:



        public class HW4 {
        

        
    public static void main(String args) {
        

        
        int a = {1, 5, 8, 11, 14, 15, 20, 23, 25, 28, 30, 34};
        
        int k;
        
        k = 43;
        
        System.out.println("k = " + k);
        
        findSumPairs(a, k, 0);
        
    }
        

        

        
    private static void findSumPairs(int array, int target, int index) {
        
        for (int i=index+1; i<array.length; i++)
        
        {
        
            if (array[index] + array[i] == target)
        
            {
        
                System.out.println(array[index] + " " + array[i]);
        
            }
        
        }
        
        if (index < array.length -1)
        
        {
        
            findSumPairs(array, target, ++index);
        
        }
        
    }
        
}






        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered 2 hours ago









        see sharpersee sharper

        101




        101






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Code Review Stack Exchange!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid …



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            Use MathJax to format equations. MathJax reference.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f213929%2ffinding-pairs-that-add-up-to-a-given-sum-using-recursion%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            -

            Popular posts from this blog

            Сан-Квентин

            Image
            Сан-Квентин Местоположение Сан-Квентин, Калифорния Координаты 37°56′13″ с. ш. 122°29′20″ з. д. H G Я O L Текущий статус Действует Режим безопасности минимальный и максимальный Количество мест 3 302 Открытие Июнь 1852 Находится в ведомстве California Department of Corrections and Rehabilitation Начальник Mike Martel, начальник  Сан-Квентин на Викискладе Сан-Квентин (англ.  San Quentin ) — тюрьма, располагающаяся на 432 акрах (около 1,7 км²), на мысе Сан-Квентин, в округе Марин, штат Калифорния, США. Тюрьма штата Калифорния Сан-Квентин была открыта в июле 1852 года и является старейшей в штате. Она была построена заключёнными, которые проживали в течение строительства на тюремном судне Вабан. В Сан-Квентине отбывали наказание и мужчины, и женщины до 1934 года, когда построили женскую тюрьму в Техачапи и туда перевели всех заключённых женщин. В Сан-Квентине приводятся в исполнение смертные казни, для этого имеется газовая камера, но...
            Read more

            8-я гвардейская общевойсковая армия

            Image
            8-я гвардейская общевойсковая ордена Ленина армия Награды: Войска: РККА → Советская армия → СВ России Род войск: Формирование: 1943 Расформирование (преобразование): 1992 Предшественник: 62-я армия (1942—43) → 8-я гвардейская общевойсковая армия (1943—92) → 8-й гвардейский армейский корпус (1992—98) Боевой путь Изюм-Барвенковская операция Донбасская операция Запорожская операция Битва за Днепр Днепропетровская операция Березнеговато-Снигирёвская операция Одесская операция Люблин-Брестская операция Варшавско-Познанская операция Восточно-Померанская операция Берлинская наступательная операция У этого термина существуют и другие значения, см. 8-я армия. Фотография Гвардейского Красного Знамени (лицевая сторона) 8-й гвардейской армии , образца 1943 года. Фотография Гвардейского Красного Знамени (оборотная сторона) 8-й гвардейской армии , образца 1943 года. 8-я гвардейская общевойсковая ордена Ленина армия  — гвардейское фор...
            Read more

            Алькесар

            Image
            Муниципалитет Алькесар Alquézar Герб 42°10′16″ с. ш. 0°01′27″ в. д. H G Я O L Страна Испания   Испания Автономное сообщество Арагон Провинция Уэска Комарка Сомонтано-де-Барбастро Алькальд Хосе Марияно Альтемир Ласкорс (ИСРП) История и география Площадь 32,36 км² Высота НУМ 660 м Часовой пояс UTC+1, летом UTC+2 Население Население 313 человек ( 2010 ) Плотность 9,92 чел./км² Этнохороним alquezarano/-a Цифровые идентификаторы Телефонный код +34 974 Почтовые индексы 22112 Автомобильный код HU alquezar.org   (исп.) Показать/скрыть карты Алькесар Алькесар Алькесар  Аудио, фото и видео на Викискладе Алькесар (исп.  Alquézar ) — муниципалитет в Испании, входит в провинцию Уэска, в составе автономного сообщества Арагон. Муниципалитет находится в составе района (комарки) Сомонтано-де-Барбастро. Занимает площадь 32,36 км². Население — 321 челове...
            Read more