C言語ポインタ書籍の勉強でエラーを解決したい。

Question

前提・実現したいこと

C言語ポインタ書籍の勉強中にコードを写したところ、エラーが発生しましたが解決の方法がよく分かりません。
型にバイトサイズが越えてるよ。っていうエラーメッセージは読めるのですがアドバイスをお願いします。C言語は初めてでvscodeでコードランナーを使ってコンパイルしています。

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

hello.c:48:34: error: character too large for enclosing character literal type
        if( (p = strrchr(command,'¥n')) != NULL){
                                 ^
hello.c:48:34: warning: multi-character character constant [-Wmultichar]
hello.c:49:18: error: character too large for enclosing character literal type
            *p = '¥0';
                 ^
hello.c:49:18: warning: multi-character character constant [-Wmultichar]
2 warnings and 2 errors generated.

該当のソースコード

ソースコード


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

int list(void){
    printf("func list¥n");
    return 1;
};

int show(void){
    printf("func show¥n");
    return 1;
};

int quit(void){
    exit(0);
};

struct command{
    char *com_str;
    int (*com_func)(void);
};

struct command coms[] = {
    {"ls",list},{"dir",list},
    {"cat",show},{"type",show},
    {"quit",quit},{"exit",quit},
    {NULL,NULL}
};

int do_command(char *command){
    struct command *p;
    for(p = coms; p->com_str != NULL; p++){
        if( strcmp(p->com_str,command)==0){
            return p->com_func();
        }
    }
    printf("command not found¥n");
    return 0;
}

int main(void){
    char command[80];
    char *p;
    while(1){
        printf(">");
        fgets(command,80,stdin);
        if( (p = strrchr(command,'¥n')) != NULL){
            *p = '¥0';
        }do_command(command);
    }return 0;
}

試したこと

一部をコメントアウトして試してみましたが、更にエラーが増えました。
hello.c:15:5: note: include the header <stdlib.h> or explicitly provide a declaration for 'exit'
と出たので#includeを追記しました。

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

vscode 拡張機能　c/c++ code runner Macです。 環境抜かりすみません。

Comments

[jimbe]

「発生している問題・エラーメッセージ」にご提示されているエラーメッセージをコードのマークダウンで囲って頂けますか。
エラー個所を示す「^」の位置がどこなのかが分かりません。

Answer 1

以下のコードの右上にあるコピーアイコンをクリックして、自分の環境にペーストしてコンパイルしてみてください。

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

int list(void){
    printf("func list¥n");
    return 1;
};

int show(void){
    printf("func show¥n");
    return 1;
};

int quit(void){
    exit(0);
};

struct command{
    char *com_str;
    int (*com_func)(void);
};

struct command coms[] = {
    {"ls",list},{"dir",list},
    {"cat",show},{"type",show},
    {"quit",quit},{"exit",quit},
    {NULL,NULL}
};

int do_command(char *command){
    struct command *p;
    for(p = coms; p->com_str != NULL; p++){
        if( strcmp(p->com_str,command)==0){
            return p->com_func();
        }
    }
    printf("command not found¥n");
    return 0;
}

int main(void){
    char command[80];
    char *p;
    while(1){
        printf(">");
        fgets(command,80,stdin);
        if( (p = strrchr(command,'\n')) != NULL){
            *p = '\0';
        }do_command(command);
    }return 0;
}

Comments

[yochan0913]

コードまでコピペしやすく解説していただきありがとうございました！無事動きました！

Answer 2

'\'＋1文字は"エスケープシーケンス"です。コンパイラによってコンパイル環境にあった文字もしくは文字列に変換されます。

Windows では \n は 0x0d,0x0a という 2 バイトに変換され、つまり文字では無く文字列になりますので「'」で囲むのは適さない…ということになるのではないでしょうか。
文字列の中から改行を探すのでしたら strstr で ”\n” をパラメータとしてみては如何でしょう。

'\0' のほうは *p が char 型なのに対し '\0' が int に解釈されて large とか…見た目のまま 2 文字なのでエラーとか…あんまり無さそうですが(デフォルトの)コンパイルオプションでそのような設定になっているのかもしれません。
いっそ *p = (char)0; にしてみるのは如何でしょうか。

Comments

[majiponi]

あれ、windowsのテキストモードだと、ファイルから読み取ったときにcrlfをlfに直して、書き込むときにlfをcrlfに変換したような気がするのですが…? (つまり、ソースコード上ではlfだけでOK)

[jimbe]

コンパイラが言っているは定数の方と思っています。

[yochan0913]

*p = '\0';の¥0を\0に変更したら動きました！理由や他の手法まで詳しく解説していただきありがとうございます。助かりました。

[otn]

> Windows では \n は 0x0d,0x0a という 2 バイトに変換され、つまり文字では無く文字列になります
間違っています。

> '\0' のほうは *p が char 型なのに対し '\0' が int に解釈されて large とか…
これも関係ないです。char型にint値を代入するのはごく普通です。

> ¥0を\0に変更したら動きました！
その通りで、原因は、バックスラッシュと円記号の間違いです。

[jimbe]

>> Windows では \n は 0x0d,0x0a という 2 バイトに変換され、つまり文字では無く文字列になります
>間違っています。

自身の中で原因が特定できてなかったので表現の仕方も含めて無責任な書き方になってしまいました。

> '\0' のほうは *p が char 型なのに対し '\0' が int に解釈されて large とか…
これも関係ないです。char型にint値を代入するのはごく普通です。

私も「ごく普通」にそのように書いていますので、何故そうなるのか分かっていませんでした。
何かそのようにするオプションでもあるのかと思ったのはその為です。

>> ¥0を\0に変更したら動きました！
>その通りで、原因は、バックスラッシュと円記号の間違いです。

エスケープシーケンスにするつもりが単なる 2 文字となってしまっていて "「'」で囲むには large だ" と言われていたということでしょうか。
tetatail 上では見た目全然分からないですね ^^;

[jimbe]

ベストアンサーは bboydaisuke さんにされたほうが良いように思います。

[yochan0913]

更に詳しくありがとうございました！マークダウン修正指摘もありがとうございます。

Answer 3

'¥n', '¥0' を '\n', '\0' に変えればよいと思います。Mac か Windows か書いてませんが、Mac だったら ¥ と \ は違う文字になります。

参考: Mac バックスラッシュ 入力方法

Comments

[Zuishin]

ウィンドウズでも違う文字です。バックスラッシュのグリフが円マークの形をしているフォントが数個採用されているだけです。

[yochan0913]

macとWindowsでは¥と\は違うのですね。知らなかった。ありがとうございます。

Answer 4

円記号を、全角文字で打っていませんか? 半角バックスラッシュが円記号で表示されているのであれば問題ないですが…。