A1~C3または1A~3Cまでの値を判定したい

ゲームの座標判定でA1~~C3または1A~~3Cまでを座標判定したいのですが、下記の画像のようにうまく判定できていません。
また、このif文だと座標が増えた場合に書く量が膨大になってしまいます。
質問したいことは、
・なぜ下記のコードで判定できていないのか
・座標が増えた場合の今よりも簡単なif判定のアドバイス
をお願いします。

C
1#include <stdio.h>
2#include <string.h>
3
4#define MAXDATA 255
5#define DATA 4  //自分の環境ではAなどが2バイト、1などが1バイトのため、最後に¥nを含めた
6　　　　　　　　　 // 4バイトにしています。（念のため、DATA 10などにもしてみましたがうまくいきませんでした）
7
8int main(void) {
9  char xy[DATA] = { };
10  char flag = 0;
11  char xy_last = 0;
12
13  while (flag == 0) {
14    printf("A1または1Aと入力してください\n");
15
16    fgets(xy, MAXDATA, stdin);
17
18    // 配列の最後を¥0にする(表示させる時の改行を防ぐ)
19    xy_last = strlen(xy) - 1;
20    xy[xy_last] = '\0';
21
22    if (strlen(xy) <= DATA) {
23      // (1~3, A~C)または、(A~C, 1~3)までで入力判定させたい
24      if (((xy == "A1" || xy == "A2" == xy == "A3") || (xy == "1A" || xy == "2A" == xy == "3A") || (xy == "B1" || xy == "B2" == xy == "B3") || (xy == "1B" || xy == "2B" == xy == "3B") || (xy == "C1" || xy == "C2" == xy == "C3") || (xy == "1C" || xy == "2C" == xy == "3C"))) {
25        printf("正しい入力です。\n");
26        flag++;
27      }
28    }
29  }
30
31  return 0;
32}
33
34```

kazuma-s

2020/08/04 02:26

> //自分の環境ではAなどが2バイト、1などが1バイトのため、文字コードが Shift-JIS で、「Ａ］は全角文字で入力しているんですか？また、「1」は、全角の「１」ではなく、半角の「1」ですか？

gyoruo

2020/08/04 02:30

すみません、UTF-8なのでAも1バイトでした。1も半角なので1バイトです。

行動規範の内容に同意します

回答5件

C
1#include <stdio.h>  // puts, fgets, sscanf
2
3#define MAXDATA 255
4
5int main(void)
6{
7	char xy[MAXDATA], c;
8	do {
9		puts("A1または1Aと入力してください");
10		fgets(xy, MAXDATA, stdin);
11	} while ((sscanf(xy, "%*1[A-C]%*1[1-3]%c", &c) != 1 || c != '\n')
12	      && (sscanf(xy, "%*1[1-3]%*1[A-C]%c", &c) != 1 || c != '\n'));
13    xy[2] = '\0';
14	puts("正しい入力です。");
15}

見たことのない関数がありますか？

投稿2020/08/04 02:59

編集2020/08/04 03:02

kazuma-s

総合スコア8224

gyoruo

2020/08/04 04:27

わざわざ書き直してくれたんですね！ありがとうございます！ sscanfは見たことないですが概要などを調べて理解してみます！

行動規範の内容に同意します

xy == "A1" は、xy配列の先頭要素へのポインターと、(どこかに配置されている)"A1"という配列の先頭要素へのポインターが一致しているか、という判定です。
ご希望の処理になっているでしょうか？

投稿2020/08/04 02:28

YT0014

総合スコア1750

gyoruo

2020/08/04 02:31

ご回答ありがとうございます！意味は理解できましたが、そのような処理では実現できないので他の方法を考えてみます。

行動規範の内容に同意します

・なぜ下記のコードで判定できていないのか

y_waiwaiさんの回答の通りです。

・座標が増えた場合の今よりも簡単なif判定のアドバイス

座標が16進文字列なら数値に変換すれば比較演算子で範囲チェックできますが、
飛び飛びの座標の場合ハッシュを作成してハッシュにあるか否かを判定した方が
可読性が高いと思います。
ハッシュ実装は以下をご参考に
お気楽Ｃ言語プログラミング超入門

投稿2020/08/04 01:43

DreamTheater

総合スコア1095

gyoruo

2020/08/04 01:46

そんな方法もあったんですね！見てみます！

行動規範の内容に同意します

C
1#include <stdio.h>
2#include <string.h>
3#include <stdbool.h>
4#include <ctype.h>
5
6#define DATA 4
7
8int main(void) {
9  char xy[DATA];
10  bool invalid = true;
11
12  while ( invalid ) {
13    printf("A1または1Aと入力してください\n");
14
15    fgets(xy, DATA, stdin);
16
17    // 配列の最後を¥0にする(表示させる時の改行を防ぐ)
18    xy[strlen(xy) - 1] = '\0';
19    if ( strlen(xy) == 2 ) {
20      char a = xy[0];
21      char n = xy[1];
22      if ( !isalpha(a) ) { char c = a; a = n; n = c; }
23      // a が A,B,C のいずれか かつ n が 1,2,3 のいずれか ならば正しい入力
24      if ( strchr("ABC",a) != NULL && strchr("123",n) != NULL ) {
25        printf("正しい入力です。\n");
26        invalid = false;
27      }
28    }
29  }
30
31  return 0;
32}