Question

我正在尝试生成两个输入字符串的子序列，但我的代码需要很长时间来生成输出。只需要建议来优化给定的代码。下面是代码

  import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;



class Subsequences {

       public static void combinations(String suffix,String prefix,List seq){
        if(prefix.length()<0)return;
//        System.out.println(suffix);
        seq.add(suffix);
        for(int i=0;i<prefix.length();i++)
         combinations(suffix+prefix.charAt(i),prefix.substring(i+1,prefix.length()),seq);
    }

    public static void main(String args[] ) throws Exception {

        List seq1=new ArrayList();
        List seq2=new ArrayList();
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        String line = br.readLine();
        int N = Integer.parseInt(line);
        for (int i = 0; i < N; i++) {
        combinations("",br.readLine(),seq1);
        combinations("",br.readLine(),seq2);
        seq1.retainAll(seq2);
        if(seq1.size()>1){
            System.out.println("Yes");
        }else{
            System.out.println("No");
        }
        seq1.clear();
        seq2.clear();
        }


    }
}

这里我使用递归生成字符串的所有子串并将所有这些存储到arraylist中。然后使用retainAll我比较之间的任何常见元素。我目前需要1.9秒才能运行。需要在1秒内完成。

Answer 1

我认为你的意思是运行不能编译程序需要很长时间。我还假设您需要CPU配置代码才能真正了解问题所在。我建议你在做任何改动之前这样做。例如，从视觉vm开始，甚至使用飞行记录器。

大部分时间都花在combinations方法上，但您还没有显示出来。

然而，对于这类问题，一个问题是假设创建对象的时间是90-99％。我建议你也做一个内存配置文件，并尝试减少你创建的对象的数量。

只有在尽可能调整内存和CPU使用率之后，您才可以考虑使用多个线程。

Answer 2

我不完全确定你的combinations方法做了什么，但它肯定没有找到子串。使用名为prefix的变量来保存后缀，使用变量suffix来保存前缀是一种很难使代码难以破译的好方法。您还引用了一个单独的combinations方法，其中包含5个参数，这些参数不会出现在您的班级中。

您的性能问题可能是由于使用了List，您应该使用Set。将ArrayList替换为HashSet，看看是否能解决问题。

需要有关代码优化的建议

2 个答案: