[已解决]代码问题出在哪了 为什么有些数据的结果不对啊

初莫M

宝藏地图的每一个坐标点均有两行字符串，你需要将两个字符串均以下列方式转换成一个数字，最终数字就是字符串中所有字母的积，其中'A'是1，'Z'是26。例如，USACO就是21*19*1*3*15＝17955
。如果两个字符串转换为数字后mod 47的值相等，此处就可以行走，即输出“GO”，否则就必须呆在原地等待救援，即输出“STAY”。

输入描述
第1行：第一个长度为1~6的大写字母串。
第2行：第二个长度为1~6的大写字母串。

输出描述
输出“GO”或“STAY”。

样例
输入 复制
COMETQ
HVNGAT
输出 复制
GO
来源
算法宝典1 阶段1

#include<stdio.h>
#include<string.h>
#include<math.h>
int main()
{
    char s1[7],s2[7];
    scanf("%s%s",&s1,&s2);
    int i,a1,a2;
    for(i=0;i<strlen(s1);i++)
    {
        a1*=(s1[i]-'A'+1);
    }
    for(i=0;i<strlen(s2);i++)
    {
        a2*=(s2[i]-'A'+1);
    }
    if(fmod(a1,47)==fmod(a2,47)) printf("GO");
    else printf("STAY");
}

最佳答案

月排行榜 / 总排行榜

陶远航

2023-10-17 17:03:43

你的代码中存在一些问题，我将帮你一一指出并提供修正的建议。

1. 变量初始化：你在使用变量a1和a2之前没有进行初始化，这可能导致未定义的行为。在C语言中，局部变量不会自动初始化，它们的初始值是不确定的。你应该在使用这些变量之前将它们初始化为1。

2. 使用fmod函数：你使用fmod函数来计算两个数的模，但是在这个问题中，你应该使用%操作符来计算整数的模。

修正后的代码如下：

#include<stdio.h>
#include<string.h>

int main() {
    char s1[7],s2[7];
    scanf("%s%s",s1,s2);
    int i;
    long long a1=1,a2=1; // 初始化为1
    for(i=0;i<strlen(s1);i++)
    {
        a1*=(s1[ i]-'A'+1);
        a1%=47; // 在每次乘法后取模，防止溢出
    }
    for(i=0;i<strlen(s2);i++)
    {
        a2*=(s2[ i]-'A'+1);
        a2%=47; // 在每次乘法后取模，防止溢出
    }
    if(a1==a2) printf("GO\
"); // 使用%操作符计算模
    else printf("STAY\
");
    return 0;
}

这段代码首先读取两个字符串，然后计算每个字符串的字符对应的数字的乘积，并在每次乘法后取模，防止溢出。最后，如果两个字符串的乘积模47相等，就输出\"GO\"，否则输出\"STAY\"。
请注意，本回答来自GPT-4
如果问题已经解决，请设置最佳答案

跳转到最佳答案楼层

陶远航

你的代码中存在一些问题，我将帮你一一指出并提供修正的建议。

1. 变量初始化：你在使用变量a1和a2之前没有进行初始化，这可能导致未定义的行为。在C语言中，局部变量不会自动初始化，它们的初始值是不确定的。你应该在使用这些变量之前将它们初始化为1。

2. 使用fmod函数：你使用fmod函数来计算两个数的模，但是在这个问题中，你应该使用%操作符来计算整数的模。

修正后的代码如下：

#include<stdio.h>
#include<string.h>

int main() {
    char s1[7],s2[7];
    scanf("%s%s",s1,s2);
    int i;
    long long a1=1,a2=1; // 初始化为1
    for(i=0;i<strlen(s1);i++)
    {
        a1*=(s1[ i]-'A'+1);
        a1%=47; // 在每次乘法后取模，防止溢出
    }
    for(i=0;i<strlen(s2);i++)
    {
        a2*=(s2[ i]-'A'+1);
        a2%=47; // 在每次乘法后取模，防止溢出
    }
    if(a1==a2) printf("GO\
"); // 使用%操作符计算模
    else printf("STAY\
");
    return 0;
}

这段代码首先读取两个字符串，然后计算每个字符串的字符对应的数字的乘积，并在每次乘法后取模，防止溢出。最后，如果两个字符串的乘积模47相等，就输出\"GO\"，否则输出\"STAY\"。
请注意，本回答来自GPT-4
如果问题已经解决，请设置最佳答案

isdkz

问题出在代码中的两个变量a1和a2没有初始化。在C语言中，未初始化的变量的值是不确定的，可能是任意的。因此，在代码中计算两个字符串的积时，变量a1和a2的初始值是未定义的，导致结果不正确。

为了解决这个问题，我们需要在代码中对变量a1和a2进行初始化。可以将它们的初始值设置为1，因为任何数乘以1都等于它本身。

修改后的代码如下：

#include<stdio.h>
#include<string.h>
#include<math.h>
int main()
{
    char s1[7],s2[7];
    scanf("%s%s",&s1,&s2);
    int i,a1=1,a2=1; // 初始化a1和a2为1
    for(i=0;i<strlen(s1);i++)
    {
        a1*=(s1[ i]-'A'+1);
    }
    for(i=0;i<strlen(s2);i++)
    {
        a2*=(s2[ i]-'A'+1);
    }
    if(fmod(a1,47)==fmod(a2,47)) printf("GO");
    else printf("STAY");
}

现在，变量a1和a2在计算字符串的积之前都被初始化为1，这样就可以正确计算结果了。

Mike_python小

在给定的代码中，发现了两个问题：

1. 变量a1和a2没有初始化：在代码的开始处，应该为变量a1和a2赋初始值为1，以免在后续计算时出现未定义行为。

修正后的代码片段：

int i, a1 = 1, a2 = 1;

2. 数组大小不足：给定的字符串长度最大为6，但数组s1和s2的大小只有6。为了容纳字符串末尾的空字符\0，数组的大小应为7。

修正后的代码片段：

char s1[7], s2[7];scanf("%s%s", s1, s2);

经过上述修正后的完整代码如下：

#include <stdio.h>#include <string.h>#include <math.h>int main() {    char s1[7], s2[7];    scanf("%s%s", s1, s2);    int i, a1 = 1, a2 = 1;        for(i = 0; i < strlen(s1); i++) {        a1 *= (s1[i] - 'A' + 1);    }        for(i = 0; i < strlen(s2); i++) {        a2 *= (s2[i] - 'A' + 1);    }        if(fmod(a1, 47) == fmod(a2, 47))        printf("GO");    else        printf("STAY");            return 0;}

修复了上述问题后，代码应该能够正确运行并给出预期的输出。希望对您有帮助！如果还有其他问题，请随时提问。

球一个最佳答案谢谢啦！这对我非常重要！