char型の配列変数にchar型の変数を代入したい

Question

使用言語 C 環境 Visual Studio 2017
初めての質問です。
独学でプログラミングを始めたのですがわからないところがあり困っています。
char型の配列変数の使い方がいまいちわかりません。

キーボードからいくつかの単語を半角スペースで区切って入力し、最後に改行を行うと
それらの単語を単語ごとに入力された順に、単語と入力回数を出力する
というプログラムを書きたいです。
英単語です。

入力例:
cat dog dog bird cat dog
出力例:
cat 2
dog 3
bird 1

・どこが間違っているのか
・違う書き方（便利な手法があれば）
の二点を教えていだたきたいです。

以下は自分で書いてみたプログラムとエラー箇所です。

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

int main(void) {
char ward[1001]; //単語 字数は 1<=wards<=1000//
int i, j;
int ct = 1; //入力された単語の数
char list_W[1001][1001]; //単語リスト
int ct_Same_W[1001]; //同単語カウント
//半角スペースで区切られて単語(半角英字)が標準入力される(末尾は改行)
int lp;
while(lp == 1) {
scanf("%s", ward); //単語の入力
for (j = 0;j < ct;j++) {
if (list_W[j] == ward) { //同単語の場合、ct_Same_W[j]でカウント
ct_Same_W[j] += 1;
}
else if (j == ct - 1) { //同単語がない場合はlist_W[j]へ代入
list_W[j] = ward;
ct += 1;
}
}
for (i = 0;i <= strlen(ward);i++) {
if (ward[i] == '\n') { //改行があった場合、
lp == 0; //ループを離脱
}
}
}
for (i = 0;i <= j;i++) {
//単語リストとそれぞれの出現回数を表示
printf("%s %d\n", list_W[i], ct_Same_W[i]);
}
}

また、将来、実務で役立つようなプログラムの書き方でアドバイス等頂けると助かります。
例：変数名や変数の宣言の仕方など

Answer 1

そもそも文字列操作をCでやるな、という話がありますが、それでもCでやってみましょう。

個人的にscanfが好きじゃないので、fgetsとstrtokを組み合わせてやってみましょう。
strtokは非常に癖があり、スレッド安全ではなく、入力文字列を書き換えるので使いにくいですが、今回は利用できそうです。

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

#ifndef countof
#   if defined(_MSC_VER) && !defined(__c2__)
        //MSVC except Clang CodeGen
#       define countof _countof
#   else
#       define countof( arr ) (sizeof(arr) / sizeof(*arr))
#   endif
#endif

///! 単語情報を格納する構造体
typedef struct tag_word_info {
    ///! 見つかった数
    size_t count;
    ///! 単語
    char word[1001];
} word_info;

int main(void) {
    char line[1001];
    size_t ct = 0;   //入力された単語の数
    word_info word_list[1001];//単語リスト
    //半角スペースで区切られて単語(半角英字)が標準入力される(末尾は改行)
    if(NULL == fgets(line, countof(line), stdin)) return 1;
    for(const char* word = strtok(line, " \n"); NULL != word; word = strtok(NULL, " \n")) {
        size_t i;
        for (i = 0; i < ct && ct < countof(word_list); ++i) {
            if (0 == strcmp(word_list[i].word, word)) {
                //同単語の場合、ct_Same_W[i]でカウント
                ++word_list[i].count;
                break;
            }
        }
        if(i == ct) {
            //同単語がない場合はlist_W[i]へコピー
            //strcpy(word_list[i].word, word);
            memcpy(word_list[i].word, word, strlen(word) + 1);
            word_list[i].count = 1;
            ++ct;
        }
    }
    for (size_t i = 0; i < ct; ++i) {
        //単語リストとそれぞれの出現回数を表示
        printf("%s %zd\n", word_list[i].word, word_list[i].count);
    }
}

https://wandbox.org/permlink/AdnCtvSQJx5d32MM

それではいくつか。

• 変数の命名、もうすこしどうにかなりませんかね・・・
• 配列の添字として使う変数は原則size_t型にしましょう
• 文字列中から文字検索は自分でループを書かずに、strchr関数を利用しましょう(今回はstrtokでやってしまいましたが)
• 構造体をつかうと可読性が上がることがあります。

追記

もうすこし関数わけするべきだったという反省を活かし、C言語でオブジェクト指向してみました。C++書きたい・・・

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

#ifndef countof
#   if defined(_MSC_VER) && !defined(__c2__)
        //MSVC except Clang CodeGen
#       define countof _countof
#   else
#       define countof( arr ) (sizeof(arr) / sizeof(*arr))
#   endif
#endif

///! 単語情報を格納する構造体
typedef struct tag_word_info {
    ///! 見つかった数
    size_t count;
    ///! 単語
    char word[1001];
} word_info;

typedef struct tag_word_list word_list;
size_t word_list_find_word(const word_list* words, const char* key_str);
void word_list_print(const word_list* words);
void word_list_push_back(word_list* words, const char* word);
struct tag_word_list {
    size_t (*find_word)(const word_list* words, const char* key_str);
    void (*print)(const word_list* words);
    void (*push_back)(word_list* words, const char* word);
    ///! 現在の大きさ(1001以下)
    size_t size;
    ///! リスト
    word_info list[1001];
};

/**
 * @brief find word from list
 * @params words[in] read-only pointer to word list
 * @params key_str[in] read-only pointer to key string to find.
 * @return Found index. If not found, this function will return `words->size`.
 */
size_t word_list_find_word(const word_list* words, const char* key_str){
    size_t i;
    for (i = 0; i < words->size; ++i) {
        if (0 == strcmp(words->list[i].word, key_str)) return i;
    }
    return i;
}
/**
 * @brief print-out word list
 * @params words[in] read-only pointer to word list
 */
void word_list_print(const word_list* words){
    for (size_t i = 0; i < words->size; ++i) {
        //単語リストとそれぞれの出現回数を表示
        printf("%s %zd\n", words->list[i].word, words->list[i].count);
    }
}
/**
 * @brief append new word to word list
 * @params words[in,out] pointer to word list
 * @params word[in] Read-only pointer to string to append.
 */
void word_list_push_back(word_list* words, const char* word){
    if(countof(words->list) <= words->size) return;
    memcpy(words->list[words->size].word, word, strlen(word) + 1);
    words->list[words->size].count = 1;
    ++words->size;
}
/**
 * @brief create word list
 */
word_list* new_word_list(){
    word_list* re = (word_list*)malloc(sizeof(word_list));
    if(NULL == re) return NULL;
    //set function pointer
    re->find_word = word_list_find_word;
    re->print = word_list_print;
    re->push_back = word_list_push_back;
    //init size is zero
    re->size = 0;
    return re;
}
/**
 * @brief delete word list
 * @params words[in,out] pointer to word list
 */
void delete_word_list(word_list* words){
    free(words);
}
int main(void) {
    char line[1001];
    word_list* words = new_word_list();//単語リスト
    if(NULL == words) return 1;
    //半角スペースで区切られて単語(半角英字)が標準入力される(末尾は改行)
    if(NULL == fgets(line, countof(line), stdin)){
        delete_word_list(words);
        return 1;
    }
    for(const char* word = strtok(line, " \n"); NULL != word; word = strtok(NULL, " \n")) {
        const size_t found_index = words->find_word(words, word);
        if(words->size == found_index) {
            //同単語がない場合は追加
            words->push_back(words, word);
        }
        else {
            //同単語の場合、カウントアップ
            ++words->list[found_index].count;
        }
    }
    words->print(words);
    delete_word_list(words);
}

https://wandbox.org/permlink/qo52P1eMw7KXXnAQ

Comments

[start08]

実際にこのプログラムに取り組んでいると文字列操作をC言語であまりやらない方がいいという事に気づきました。
書いてくださったコードもひとつひとつ意味を調べながら確認したいと思います。
質問してよかったです！
ありがとうございました！

Answer 2

幾つか前提変更しましたのでご参考のPGです。
scanf("%s")では改行文字はスペース文字と同じで入力の区切りとみなされて、それ自身を取り込むことはできないのでEOFで終了としました。

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

int main(void) {
    char ward[1001];    //単語  字数は 1<=wards<=1000//
    int i, j;
    int ct = 1;                 //入力された単語の数
    char list_W[1001][1001]={'\0'};     //単語リスト クリア
    int ct_Same_W[1001];        //同単語カウント
    //半角スペースで区切られて単語(半角英字)が標準入力される(末尾は改行)
    // int lp=1;
    while(scanf("%s", ward)!=EOF) {  // 単語入力
        for (j = 0; j<ct; j++) {
            if (strcmp(ward,list_W[j])==0) {    //同単語の場合、ct_Same_W[j]でカウント
                ct_Same_W[j] += 1;
                break;
            }
            else if (j == ct - 1) {       //同単語がない場合はlist_W[j]へ代入
                strcpy(list_W[j], ward);
                ct_Same_W[j] = 1;
                ct += 1;
                break;
            }
        }
        // for (i = 0;i <=  strlen(ward);i++) { // \nは入力されないのでEOF（ctl+d）としました。
        //     if (ward[i] == '\n') {        //改行があった場合、
        //         lp == 0;            //ループを離脱
        //     }
        // }
    }
    for (i = 0; i<ct-1 ;i++) {
    //単語リストとそれぞれの出現回数を表示
        printf("%s %d\n", list_W[i], ct_Same_W[i]);
    }
}

Comments

[start08]

scanfがとても不便だったり便利だったり、色々調べてるうちになんとなく理解できました。
書いてくださったプログラムも参考にして改めて自分で書いてみたいと思います。
質問してよかったです、ありがとうございました！

Answer 3

文字列のコピーには、代入ではなくstrcpyを使います。
list_W[j] = ward;を、strcpy(list_W[j], ward);に置き換えてください。

どうしても=で代入したい場合は、一文字ずつ処理するか、ポインタを用いる必要があります。

...あと、wardじゃなくてwordじゃないですかね。

---

私も書いてみました。せっかくなので載せます。
純粋なC言語はしばらく触ってませんので、よくない点も多々あると思いますが...

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

#define BUF_LEN 1000
#define WORD_LEN 20
#define WORD_TYPE 500

typedef struct NL_WORD {
    char word[WORD_LEN+1];
    int count = 0;
} NL_WORD;

char *read_token(char **, char *);
int find_name(NL_WORD *, int, char *);

int main(void) {
    char buffer[BUF_LEN+1];    
    char *buf_ptr = buffer;

    char input_word[WORD_LEN+1];
    
    NL_WORD word_info[WORD_TYPE] = {};
    int read_word_num = 0;

    int i;

    fgets(buffer, BUF_LEN, stdin);
    while(read_token(&buf_ptr, input_word) != NULL) {
        int index;
        if( (index = find_name(word_info, read_word_num, input_word)) != -1 ) {
            word_info[index].count++;
        } 
        else {
            strcpy(word_info[read_word_num].word, input_word);
            word_info[read_word_num].count = 1;
            read_word_num++;
        }
    }

    for(i = 0; i < read_word_num; i++) {
        printf("word: %s\n", word_info[i].word);
        printf("num:  %d\n", word_info[i].count);
    }
    
    system("PAUSE");
    return EXIT_SUCCESS;
}

char *read_token(char **buffer, char *dst) {
    int start, end;
    for(start = 0; isspace((*buffer)[start]); start++) {
        if( (*buffer)[start] == '\0' ||
            (*buffer)[start] == '\n' ) return NULL;
    }
    *buffer += start;

    for(end = 0; !isspace((*buffer)[end]); end++) {
        dst[end] = (*buffer)[end];
    }
    dst[end] = '\0';
    *buffer += end;

    return *buffer;
}

int find_name(NL_WORD *word_list, int word_list_len, char *input_word) {
    int i;
    for(i = 0; i < word_list_len; i++) {
        if( !strcmp( word_list[i].word, input_word ) ) return i;
    }
    return -1;
}

このコード、yumetodoさんのコードの劣化版みたいになってますね...
上げてから気付きました。

---

命名というか、変数名の書き方で、次のような分類があります。

名前(例示)	別名	説明
UpperCamelCase	PascalCase	単語の頭文字を大文字にして繋げる
lowerCamelCase	camelCase	単語の頭文字は大文字、でも最初だけ小文字
SNAKE_CASE	SNAIL_CASE	全部大文字、単語の繋ぎはアンダーバー
snake_case	snail_case	全部小文字、単語の繋ぎはアンダーバー

言語によって、どのケースを用いるかが大体決まっています。
名前を見ただけで、『あ、これは関数だ』『変数だ』『マクロだ』と判断できるので重要です。

また、明白でない場合、略語はあまり用いない方が良いかもしれません。
コードが横に長くなりますが、Cは自由構文なので、適宜改行で調整できます。

Comments

[start08]

早い解答ありがとうございます。
早速試してみます。

確かに ward ではなく word ですね！
徹夜と英語の勉強不足が響いて恥ずかしいミスをしていましたorz

[start08]

変数の宣言についての解説ありがとうございます。
書いてくださったコードも、にらめっこしつつ、ひとつひとつ調べてみたいと思います。
とても参考になりました、ありがとうございます。
質問してよかったです！

[yumetodo]

>このコード、yumetodoさんのコードの劣化版みたいになってますね...

find_nameはたしかに関数として処理をくくりだすべきだったな・・・、反省。

>read_token

strtokよりこれのほうが本当はいいよなぁ、本当にstrtokの設計考えたやつは頭おかC・・・

Answer 4

素早い回答、およびプログラムまで書いてくださってありがとうございます。
回答者様方の書いたものに比べたら大したものではありませんが、
ヒントを頼りに自分なりの答えを書いてみました。

不完全ながらもほぼ半日かけてしまいましたがひとまずはスッキリしました。
とても楽しかったです！
学習不足を痛感したのと同時に、違う言語も学習し、使い分ける事が重要にも感じました。

どの回答者様も興味深い回答をしてくださったのですが、変数の宣言の仕方についても教授いただいた方の回答をベストアンサーにしました。
まだ全て読んでいないので、休憩しながらでもにらめっこしたいと思います。

余談ですが、現在大学４年の就活中で、就活を始めた時期にプログラマーへと志望を変え、文系ながらもコツコツ取り組んでおります。
いまだに内定を取れていませんがどうしてもプログラマーになりたく、頑張っていたのですが、
難しいプログラミングの概念に挫折しそうになっていたところ、たまたまこのサイトに辿り着きました。
今日は本当に質問してよかったと感じています。
また、わからないことがあったら質問させていただきたいと思います。
本当にありがとうございました。

Comments

[yumetodo]

まあなんか回答の殆どが有効活用されなかったなという思い。
つーかなんでもかんでも自分でループ書くの本当に良くない、保守性もテスト可能性もかけらもない
。fgetsの戻り値見ないとかありえないし、構造体使わないのも本当にありえないし、
そもそもここは回答欄で質問者が書き込む場所ではないし、なんだかなという。

[start08]

まだまだ曖昧な知識で、今の自分が取り組める課題ではなかったと実感しております。
なんでもかんでもループを書かず、strtokなどを使うべきでした。
特に、戻り値、ポインタ、アドレスなどと言った欠かせない基礎概念をしっかり理解することをさけていたところが致命的でした。
最優先で学習します。
まずはすべての回答のわからないところひとつひとつを調べてみます。
保守性、テストの可能性についても意識していきたいと思います。
プログラマとしての常識がまったく身についてないことを痛感しましたし、このサイトの使い方自体間違えていたので、恥ずかしい次第です。
またプログラミングの基礎を１から学びなおします。

[start08]

確認のような質問になってしまいますが、お時間があればお答えいただけると助かります。

まず、「構造体」というのは、変数の宣言の仕方を工夫して、関連のある変数をひとくくりにしてわかりやすくするという認識でだいたいあってますでしょうか？
(イメージとしては、配列変数を一列に並んだロッカーに例えて、ロッカールームを用意する、、というような感じ？、3次配列変数と違って色々な型の配列変数をひとくくりにできる)

書き方としては、int main(void)の上に宣言して、後にその内容を記述する定義(単純ですいません)、といった、
structを用いて、宣言→プログラム→定義
であってますでしょうか

初歩的な質問で煩わせてしまい申し訳ありません。

[yumetodo]

>まず、「構造体」というのは、変数の宣言の仕方を工夫して、関連のある変数をひとくくりにしてわかりやすくするという認識でだいたいあってますでしょうか？

なんか違う。構造体とは任意の型のデータの集合です。整数型(int, unsigned long, etc...)や浮動小数点型(flaot, double)やその配列型やポインタ型、構造体型などを内包できます。

struct {
//member variable declarations
}

までが型名です。ただこんな長い型名をいちいち書いていられないので

struct /*tag name*/ {
//member variable declarations
}

tag nameをつけます。するとたとえば

struct tag_Hoge {
int a;
float b;
long c[2];
double* d;
};

という構造体があったとき

struct tag_Hoge

を型名として利用できます。しかしstructと書くのすら億劫です。

typedef struct tag_Hoge {
int a;
float b;
long c[2];
double* d;
} Hoge;

や

typedef struct tag_Hoge Hoge;
struct tag_Hoge {
int a;
float b;
long c[2];
double* d;
};

のようにふつうします。

[start08]

詳しい解説ありがとうございました！
実際に構造体を用いたプログラムをいくつか書いて、使えるようにしたいと思います。
オブジェクト指向にも興味がわいたのでC++にも触れてみたいと思います。