Question

我正在尝试解决problem PLD on SPOJ，但我正在第9个测试用例中获得WA。

我的方法：
我正在实现Manacher的算法，我相信如果有错误，那么在这段代码中它可能是错误的。

if((k%2==0)&&(p[i]>=k)&&(temp[i]=='#'))
                                       count++;
if((k%2==1)&&(p[i]>=k)&&(temp[i]!='#'))
                                       count++;

但根据我的方法，如果字符是#，那么以它为中心的回文字符串的最大长度可以是偶数，所以如果p[i] >= k，那么如果我们找到一个，我就会增加数量回文串长均匀。

类似地，对于以第i个位置为中心的字符[考虑除＃以外的输入字符]，但是对于奇数长度的字符串。请帮帮....

#include<stdio.h>
#include<string.h>
char a[30002],temp[60010];
int p[60010];
int min(int a,int b)
{
    if(a<b)
           return a;
    return b;
}
int main()
{
    //freopen("input.txt","r+",stdin);
    //freopen("a.txt","w+",stdout);

    int k,len,z;
    scanf("%d",&k);
    getchar();
    gets(a);
    len=strlen(a);

    //Coverting String
    temp[0]='$';
    temp[1]='#';
    z=2;
    for(int i=1;i<=len;i++)
    {
            temp[z++]=a[i-1];
            temp[z++]='#';
    }
    len=z;
    int r=0,c=0,check=0,idash,t,count=0;
    for(int i=1;i<len;i++)
    {
            check=0;
            idash=c-(i-c);
            p[i]=r>i?min(r-i,p[idash]):0;

            t=p[i];
            while(temp[i+p[i]+1]==temp[i-1-p[i]])
                                                p[i]++;

            if(r<i+p[i])
            {
                        c=i;
                        r=i+p[i];
            }
            if((k%2==0)&&(p[i]>=k)&&(temp[i]=='#'))
                                                   count++;
            if((k%2==1)&&(p[i]>=k)&&(temp[i]!='#'))
                                                  count++;
    }

    printf("%d",count);
    //getchar();
    //getchar();    
    return 0;
}

Answer 1

您可能希望利用逻辑表达式的C ++ 短路评估例如，重新排列订单，以便首先检查“＃”：

if ((temp[i] == '#') && (k % 2 == 0) && (p[i] >= k))

在上面的重新排列中，如果字符不是'＃'，则不评估其他表达式。

您可能希望将(p[i] >= k)提取到外部if语句，因为它们都是通用的：

if (p[i] >= k)
{
  if ((temp[i] == '#') && (k % 2 == 0)) ++count;
  if ((temp[i] != '#') && (k % 2 == 1)) ++count;
}

上述修改将导致只对表达式(p[i] >= k)进行一次评估。

同时检查for循环以查看是否存在不会更改或重复的语句或表达式。如果循环内的语句或表达式没有改变，则称为不变，并且可以在循环之前或之后移动。

可以评估重复的语句或表达式（例如数组索引计算）并将其存储在临时变量中。虽然好的编译器可能会这样做（取决于优化级别），但在性能要求中，您可能希望帮助编译器。

另一个建议是用指向该位置的指针或对该位置的引用替换p[i]。同样，这是为了在未优化设置优化时帮助编译器：

  int& p_slot_i = p[i];  // This syntax needs checking
// or
  int * p_slot_i = &p[i];
//...
  t = *p_slot_i;
  while(temp[i + *p_slot_i + 1] == temp[i - 1 - *p_slot_i)
  {
    *p_slot_i++;
  }

最后，消除空格，空白行和花括号不会影响程序性能。一行或多行间隔的程序将具有精确的汇编转换和精确性能。所以请添加空格，空白行和花括号以提高可读性。

编辑1：min（）的表现
您可能希望将min()函数声明为inline，以便向编译器建议您希望将函数粘贴到调用函数的位置，而不是调用函数。函数调用会减慢程序执行速度。

SPOJ PLD [错误答案]

1 个答案: