Lynn_oyl 发表于 2023-10-3 10:21:36

C++题目求助改错

题目详情:
石头剪刀布
说明
石头剪刀布是常见的猜拳游戏。石头胜剪刀,剪刀胜布,布胜石头。如果两个人出拳一样,则不分胜负。
一天,小A和小B正好在玩石头剪刀布。已知他们的出拳都是有周期性规律的,比如:“石头-布-石头-剪刀-石头-布-石头-剪刀……”,就是以“石头-布-石头-剪刀”为周期不断循环的。请问,小A和小B比了N轮之后,谁赢的轮数多?

输入格式
输入包含三行。
第一行包含三个整数:N,NA,NB,分别表示比了N轮,小A出拳的周期长度,小B出拳的周期长度。0 < N,NA,NB < 100。
第二行包含NA个整数,表示小A出拳的规律。
第三行包含NB个整数,表示小B出拳的规律。
其中,0表示“石头”,2表示“剪刀”,5表示“布”。相邻两个整数之间用单个空格隔开。
输出格式
输出一行,如果小A赢的轮数多,输出A;如果小B赢的轮数多,输出B;如果两人打平,输出draw。

样例
输入数据 1
10 3 4
0 2 5
0 5 0 2
输出数据 1
A

提示
对于测试数据,猜拳过程为:
A:0 2 5 0 2 5 0 2 5 0
B:0 5 0 2 0 5 0 2 0 5
A赢了4轮,B赢了2轮,双方打平4轮,所以A赢的轮数多。


我的代码:
#include <bits/stdc++.h>
using namespace std;
char who_win(int x,int y){
    if(x+y==5){
      if(x==5) return 'A';
      else return 'B';
    }else return x<y?'A':y<x?'B':'C';
}
int main(){
    int n,na,nb;
    cin >>n>>na>>nb;
    int a={},b={};
    for(int i=0;i<na;i++) cin >>a;
    for(int i=0;i<nb;i++) cin >>b;
    int as=0,bs=0;
    for(int i=1;i<=n;i++){
      if(who_win(a,b)=='A') as++;
      if(who_win(a,b)=='B') bs++;
    }
    string r=as>bs?"A":as<bs?"B":"draw";
    cout<<r;
    return 0;
}
请问我的代码错在哪?

isdkz 发表于 2023-10-3 10:21:50

这个程序的问题在于它在定义数组时使用了变量na和nb,这是不允许的。在C++中,定义数组时必须使用常量表达式。因此,你需要使用动态分配的数组来解决这个问题。同时,你也需要将who_win函数的返回值从char类型改为string类型,以便于最后输出结果。下面是修改后的代码:

#include <bits/stdc++.h>
using namespace std;

string who_win(int x,int y){
    if(x+y==5){
      if(x==5) return "A";
      else return "B";
    }else return x<y?"A":y<x?"B":"C";
}

int main(){
    int n,na,nb;
    cin >>n>>na>>nb;
    int* a = new int;
    int* b = new int;
    for(int i=0;i<na;i++) cin >>a[ i];
    for(int i=0;i<nb;i++) cin >>b[ i];
    int as=0,bs=0;
    for(int i=1;i<=n;i++){
      if(who_win(a,b)=="A") as++;
      if(who_win(a,b)=="B") bs++;
    }
    string r=as>bs?"A":as<bs?"B":"draw";
    cout<<r;
    delete[] a;
    delete[] b;
    return 0;
}

陶远航 发表于 2023-10-3 10:22:03

你的代码存在以下几处错误:

1. 数组声明时,使用了变量作为数组大小。C++中的数组大小必须是常量表达式,不能使用变量。你可以使用动态数组(vector)来替代。

2. who_win函数的返回值应该是字符类型,表示谁赢得了比赛,但你的代码返回的是'A'、'B'或'C'。实际上,应该返回字符串 "A"、"B" 或 "draw",以便与后续的比较操作兼容。

3. 在主循环中,计算 i%na-1 和 i%nb-1 的索引时存在问题。当 i 等于 0 时,这些表达式将导致索引为负数,从而访问到无效的内存位置。

下面是修改后的代码:

#include <iostream>
#include <vector>
using namespace std;

string who_win(int x, int y) {
    if (x == y) {
      return "draw";
    } else if ((x == 0 && y == 2) || (x == 2 && y == 5) || (x == 5 && y == 0)) {
      return "A";
    } else {
      return "B";
    }
}

int main() {
    int n, na, nb;
    cin >> n >> na >> nb;

    vector<int> a(na), b(nb);
    for (int i = 0; i < na; i++) {
      cin >> a[ i];
    }
    for (int i = 0; i < nb; i++) {
      cin >> b[ i];
    }

    int as = 0, bs = 0;
    for (int i = 0; i < n; i++) {
      string result = who_win(a, b);
      if (result == "A") {
            as++;
      } else if (result == "B") {
            bs++;
      }
    }

    string r = (as > bs) ? "A" : ((as < bs) ? "B" : "draw");
    cout << r;

    return 0;
}


此代码修正了上述提到的问题,并且使用了动态数组(vector)来存储输入数据。希望能帮助到你!
如果问题已经解决,请设置最佳答案

Mike_python小 发表于 2023-10-3 10:22:23

问题分析:

1. 数组的大小声明方式不正确。
2. 输入部分的循环条件错误,导致输入出错。
3. 输出判断条件错误。

修改后的代码如下:


#include <iostream>
using namespace std;

char who_win(int x, int y) {
    if (x + y == 5) {
      if (x == 5)
            return 'A';
      else
            return 'B';
    } else {
      return x < y ? 'A' : y < x ? 'B' : 'C';
    }
}

int main() {
    int n, na, nb;
    cin >> n >> na >> nb;
   
    int a = {}, b = {};

    for (int i = 0; i < na; i++)
      cin >> a;

    for (int i = 0; i < nb; i++)
      cin >> b;

    int as = 0, bs = 0;

    for (int i = 0; i < n; i++) {
      if (who_win(a, b) == 'A') {
            as++;
      } else if (who_win(a, b) == 'B') {
            bs++;
      }
    }

    string r = as > bs ? "A" : (as < bs ? "B" : "draw");
    cout << r;

    return 0;
}


请注意,修改后的代码修复了数组声明大小的问题,并且修复了输入循环条件和输出判断条件。

球一个最佳答案谢谢啦!这对我非常重要!{:10_254:}{:10_254:}{:10_254:}

Lynn_oyl 发表于 2023-10-3 10:33:00

Mike_python小 发表于 2023-10-3 10:22
问题分析:

1. 数组的大小声明方式不正确。

谢谢
页: [1]
查看完整版本: C++题目求助改错