不同子串的串联

时间:2015-05-15 21:00:36

标签: java string out-of-memory substring concatenation

question - 以字典顺序排列给定字符串的所有不同子字符串并将它们连接起来。打印连接字符串的第K个字符。确保给定的K值有效,即将存在第K个字符

输入格式 第一行将包含数字T,即测试用例的数量。 每个测试用例的第一行将包含一个包含字符(a-z)的字符串,第二行将包含一个数字K.

输出格式 打印第K个字符(字符串为1索引)

约束 1≤T≤5 1≤length≤105 K将是一个合适的整数。

示例输入#00

1
dbac
3

示例输出#00

c

解释#00

按字典顺序排列时的子串如下

a,ac,b,ba,bac,c,d,db,dba,dbac 在连接它们时,我们得到

aacbbabaccddbdbadbac 这个字符串中的第三个字符是c,因此是答案。

这是我的代码:

import java.io.*;
import java.util.*;
import java.text.*;
import java.math.*;
import java.util.regex.*;

public class Solution 
{

public static void gen(String str,int k)
{


      int i,c;ArrayList<String>al=new ArrayList<String>();
     for(c=0;c<str.length();c++)
    {
        for(i=1;i<=str.length()-c;i++)
        {
            String sub = str.substring(c,c+i);
            al.add(sub);
        }
    }

    HashSet hs = new HashSet();
    hs.addAll(al);
    al.clear();
    al.addAll(hs);

    String[] res = al.toArray(new String[al.size()]); 
    Arrays.sort(res);

    StringBuilder sb= new StringBuilder();

        for(String temp:res)
        {
           sb.append(temp);   
        }

    String s = sb.toString();
    System.out.println(s.charAt(k-1));
}


public static void main(String[] args) 
{
    Scanner sc = new Scanner (System.in);
    int t = Integer.parseInt(sc.nextLine());

        while((t--)>0)
        {
            String str = sc.nextLine();
            int k = Integer.parseInt(sc.nextLine());                 
            gen(str,k);

        }

   }
 }

此代码适用于小输入,例如上面的测试用例,但是对于大输入,它要么超时要么显示这样的东西我确实知道问题是关于内存,任何替代方法来做这个问题或者无论如何重用相同的内存??

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
at java.util.Arrays.copyOfRange(Arrays.java:2694)
at java.lang.String.<init>(String.java:203)
at java.lang.String.substring(String.java:1913)
at Solution.gen(Solution.java:19)
at Solution.main(Solution.java:54)

2 个答案:

答案 0 :(得分:1)

你的内存不足。您可以通过使用-Xms256m -Xmx1024启动JVM来增加JVM正在使用的内存,您可以尝试一些优化。

public static void gen(String str, int k) {

    int i, c;

    //Adding directly to the Set prevents a larger list because you remove the duplicates
    Set<String> set = new TreeSet<String>();

    for (c = 0; c < str.length(); c++) {
        for (i = 1; i <= str.length() - c; i++) {
            String sub = str.substring(c, c + i);
            set.add(sub);
        }
    }
    //TreeSet already orders by the String comparator


    StringBuilder sb = new StringBuilder();

    for (String temp : set) {
        sb.append(temp);
        if(sb.length()>k){
           break;
        }
    }

    String s = sb.toString();
    System.out.println(s.charAt(k - 1));
}

[编辑]增加了小的性能提升。尝试看它是否变得更快,我没有看看StringBuilder.length()的性能,看它是否会改善或减少。

答案 1 :(得分:1)

根据您给出的限制(最多105个字符),您不应该遇到内存不足的问题。也许你正在测试非常大的字符串。

所以如果你有,这里有一些你浪费记忆的地方:

  • 填写完集后,将其复制到列表中。这意味着子串集合的两个副本,而您将不再使用该集合。
  • 将列表复制到数组后,您现在有三个子串集合副本,但您不再使用该列表。
  • 现在您创建一个StringBuilder并将所有子字符串放入其中。但是知道整个连接字符串并不是很有趣。我们只需要一个字符,那么为什么要将串联放在内存中呢?此外,在上面所有浪费的副本中,至少你没有自己复制子字符串。但是现在您要将它们附加到StringBuilder,您正在创建它们的副本。这将是一个很长的字符串。
  • 然后使用StringBuildertoString()的内容复制到新字符串中。这会创建一个非常大的连接字符串的副本(我们已经说过我们实际上并不需要)。

您已经获得了使用TreeSet并直接填充它而不是创建列表,集合和排序列表的合理建议。下一步是从该集合中提取正确的字符,而不实际保持连接的字符串

因此,假设您的集合被称为set

Iterator<String> iter = set.iterator();

int lengthSoFar = 0;
String str = null;

while ( lengthSoFar < k && iter.hasNext() ) {

     str = iter.next();           // Got the next substring;
     lengthSoFar += str.length();
}

// At this point we have the substring where we expect the k'th
// character to be.

System.out.println( str.charAt( k - lengthSoFar + str.length() - 1 );

请注意,程序需要更长时间才能获得高k的值而不是低值,但通常它会比构建整个连接字符串更快,因为你会在你完成后立即停止找到正确的子串。