構造体のメンバ変数と文字を比較したい

Question

前提・実現したいこと

構造体のメンバ変数?に格納した文字を比較したい

発生している問題・エラーメッセージ

構造体に格納した文字と比較したい文字が比較できない

該当のソースコード

#define		_CRT_SECURE_NO_WARNINGS
#include	<ctype.h>
#include	<stdio.h>
#include	<stdlib.h>
#include	<string.h>
#include	"conioex.h"

#define		MAX_MONSTERNAME	(255)
#define		MAX_MONSTER	(999)
#define		NOW_MONSTER (176)

// 構造体
typedef struct
{
	～省略～

	char	Type[256];

	～省略～

}MONSTER;

MONSTER	Monster[MAX_MONSTER];

MONSTER* pMonster = &Monster[0];

//プロトタイプ宣言
void InputStatus(MONSTER*);
void OutputStatus(MONSTER*, const char* a);

int saidaisu;

int main()
{
	～省略～
	//ステータス入力
	for (int i = 0; i < saidaisu; i++)
	{
		printf("%d体目を入力してください\n", i + 1);
		InputStatus(&pMonster[i]);
	}
	system("cls");
	//ステータス表示
	OutputStatus(&pMonster[saidaisu - 1], FILENAME);
	～省略～
	return 0;
}

void InputStatus(MONSTER* p)
{
	printf("タイプ：");
	scanf("%s", (&p->Type));
}

void OutputStatus(MONSTER* p , const char* a)
{
	for (int i = 0; i < saidaisu; saidaisu--)
	{
		if (pMonster[saidaisu - 1].Type == "赤")
		{
			fprintf(pFile, "%s", pMonster[saidaisu - 1].Type);
		}
		else if (pMonster[saidaisu - 1].Type == "青")
		{
			fprintf(pFile, "%s", pMonster[saidaisu - 1].Type);
		}
		else if (pMonster[saidaisu - 1].Type == "緑")
		{
			fprintf(pFile, "%s", pMonster[saidaisu - 1].Type);
		}
		else if (pMonster[saidaisu - 1].Type == "紫")
		{
			fprintf(pFile, "%s", pMonster[saidaisu - 1].Type);
		}
		else if (pMonster[saidaisu - 1].Type == "黄")
		{
			fprintf(pFile, "%s", pMonster[saidaisu - 1].Type);
		}
		else
		{
			fprintf(pFile, "なにこれ");
		}
}

試したこと

pMonster[saidaisu - 1].Typeを（）で囲んでみる
赤、青、緑、紫、黄、のそれぞれを""ではなく''で囲んでみる

()で囲んでみるのは意味ありませんでした
''で囲むと==に赤線が引かれて、オペランド型に互換性がありません("char*"と"int")と出ます

補足情報（FW/ツールのバージョンなど）

VisualStudio2022
Windowsデスクトップウィザード

Comments

[jimbe]

OutputStatus のパラメータが無意味ですし、 for ループもダメでしょうし、pMonster も Monster で良いのでは。

[cafea]

お気になさらず

Answer 1

※ (直接の)回答ではありません

そもそもMONSTERのタイプが文字列なのは妥当なんだろうか。
ロジカルな意味とその表現(表記)を混ぜるとややこしくならないか?

僕なら

enum color { red, green, blue ... };

typedef struct {
  enum color Type; // red/green/blue... のいずれか
} MONSTER;

とするやろなー...

※ int Type; // 1ならred / 2ならgreen... でもかまわんけど

[追記] C++なら：
せめて std::string Type; とします。 std::string なら == で比較できます。

Comments

[cafea]

enumでcolorを実装したとき、どのように比較を書いたらいいですか？

[episteme]

たとえば
enum color strtoType(const char* str) {
strが"赤" なら red を返す
strが"青" なら以下同文
どれでもないなら -1 を返す(enum color に unknown=-1 を追加するのがbetter)
}
を用意して、入力された文字列を食わす。
# Type を文字列にするのはその逆: const char* Typetostr(enum color) を定義する。

[cafea]

まだ初心者なもので
strが"赤" なら red を返す
strが"青" なら以下同文
どれでもないなら -1 を返す(enum color に unknown=-1 を追加するのがbetter)
をどう書けばいいのかわからないのですが、どう書くのが良いでしょうか？
なんとなく if と代入を使うのかなぁとは想像できるのですけど。

[episteme]

if ( strcmp(str, "赤") == 0 ) return red; // strが"赤" なら red を返す

# ﾍﾞｽﾄｱﾝｻｰつけたんだから解決したんですよね?

[cafea]

解決はしたのですが、epistemeさんのやり方がプログラム的には良い気がしてお聞きしたかったのです。
ありがとうございます。

[episteme]

僕の回答をﾍﾞｽﾄｱﾝｻｰにしてくれたのは嬉しいけど、質問:「構造体のメンバ変数と文字を比較したい」と整合しないぞ?

Answer 2

文字列の比較にはstrcmp(より正確には_mbscmp)を使ってください。

if ( strcmp(pMonster[saidaisu - 1].Type, "赤") == 0){
    // 略
}

蛇足ながら、for文にかなり違和感を感じます(saidaisu--??)

Comments

[cafea]

ありがとうございました。

Answer 3

if (pMonster[saidaisu - 1].Type == "赤")

これでは（C言語の）文字列の比較はできません
strcmp関数で調べてみよう

Comments

[cafea]

ありがとうございます。