超出时限

时间:2015-12-13 08:34:31

标签: java dynamic-programming

问题(SPOJ.com - 问题FARIDA)。我正在使用(https://codinghangover.wordpress.com/2014/01/15/spojfarida-princess-farida/)上给出的相同方法。

以下是我的解决方案==>

import java.util.HashMap;
import java.util.Map;
import java.util.Scanner;

class FARIDA {
    public static long maxC(Map<String, Long> map, long c[], int s, int e)
    {
        if(s>e)
            return 0;
        if(map.containsKey(s+"|"+e))
            return map.get(s+"|"+e);
        if(s==e)
            map.put(s+"|"+e, c[s]);
        else 
            map.put(s+"|"+e, Math.max(c[s]+ maxC(map,c,s+2,e),maxC(map,c,s+1,e)));

        return map.get(s+"|"+e);

    }
    public static void main(String[] args) {
        Scanner in = new Scanner(System.in);
        int t = in.nextInt();
        for(int j=1;j<=t;j++)
        {
            int n=in.nextInt();
            long c[]= new long[n];
            for(int i=0; i<n; i++)
             c[i]=in.nextLong();
            Map<String, Long> map = new  HashMap<String, Long>();
            System.out.println("Case "+j+": "+maxC(map,c,0,n-1));
        }
        in.close();
    }
}

为什么我在java中获得TLE?需要什么样的优化? HashMap有什么问题吗?

2 个答案:

答案 0 :(得分:1)

我认为获得TLE的唯一可能原因是您使用的是使用字符串作为键的HashMap。因此,当您尝试访问HashMap时,您浪费时间,并且HashMap将您输入的字符串与HashMap中已有的所有键匹配。无需使用HashMap。您可以使用数组实现所有这一切,并将数组的索引作为键。我已将map从HashMap更改为long数组。

像这样的东西::

public static long maxC(long map[], long coins[], int n) // n is the total number of monsters
{
    if(n == 0) // If there are no monsters in the path
        return 0;
    if(n == 1) // Just in case there is only one monster in the way
        return coins[0];
    map[0] = coins[0];
    map[1] = Math.max(map[0], coins[1]);
    for(int i = 2; i < n; i++) {
        map[i] = Math.max(map[i-2] + coins[i], map[i-1]);
    }
    return map[n - 1];
}

for循环中,我首先考虑是否只有2个怪物,如果有3个怪物,请使用此解决方案,等等。

这显着降低了程序的复杂性,因为现在您不必匹配字符串。此外,这里我使用了自下而上的方法,你绝对可以修改上面的方法并使用上下方法。虽然我更喜欢自下而上的方法,因为我们不在这里进行任何递归调用,我相信这节省了一些时间,因为我们没有从堆栈中推送和弹出函数状态。

编辑::

上下方法::

public static long maxC(long map[], long coins[], int n)
{
    if(n-1 < 0)
        return 0;
    if(map[n - 1] != 0) {
        return map[n - 1];
    }
    map[n - 1] = Math.max(maxC(map, coins, n-2) + coins[n - 1], maxC(map, coins, n-1));
    return map[n - 1];
}

在这里,如果没有怪物,我会返回0,并返回map[n-1]我已经有一个我之前计算过的解决方案。

您对函数的初始调用看起来像这样(来自main)::

maxC(map, c, n);

在任何情况下我们都不需要较低的索引,所以我将其删除了。

你可以尝试上述任何一种方法,我相信你会获得AC。 :d

答案 1 :(得分:0)

我认为有一个更好,更简单的方法。 让我们说我们有一组正整数。我们所要做的就是最大化问题所述的总和。要做到这一点

  1. 查找数组中的最大值(只要它不是-1)
  2. 将其添加到sum,并更改其值和相邻的两个(i + 1和 i-1)到-1(因此不再考虑它们)。
  3. 将-1的计数存储在数组中并继续循环直到 countOfNegOnes==(n-1)countOfNegOnes==(n)
  4. 输出sum