質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

出力時の文字化けを改善したいです。

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 921

score 12

英文を入力し、英単語の出現頻度を求めるプログラムで、出力が文字化けしてしまいます。
*END*」を検出したらプログラムを終了し、例えば「This is a pen. This is a book.」という英文を入力したら、

This    2
is        2
a         2
pen.      1
book.     1

と表示されるようにしたいです。 
同じ単語であっても大文字と小文字の判定はしていないので違う単語と識別します。ピリオドがついている単語は違う単語と識別します。

元のプログラム↓

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

  int strcmp(const char *s1,const char *s2);
  struct node{
    char word[50];
    int kaisu;
    char tango;
    struct node *left;
    struct node *right;
};

  char wd[50];

  char *GetWord()
  {
    char a;
    int len;

  a=getchar();
  while(a==' ' || a=='\n') a=getchar();
  len=0;
  while(a!=' ' && a!='\n'){
    wd[len++]=a;
    a=getchar();
  }
  wd[len]='\0';
  return &wd[0];
  }

struct node *insert(struct node *x,int a)
{
  if(x==NULL){
    x=(struct node *)malloc(sizeof(struct node));
    x->tango=a;
    x->kaisu=1;
    x->left=NULL;
    x->right=NULL;
  }else if(a== x->tango)
     x->kaisu++;
   else if(a< x->tango)
    x->left=insert(x->left,a);
   else
    x->right=insert(x->right,a);
   return x;
  }

void treeprint(struct node *x)
{
  if(x!=NULL){
    treeprint(x->left);
    printf("%5c %5c\n",x->tango,x->kaisu);
    treeprint(x->right);
    }
}

  int main(void)
  {
    struct node *root;
    char *w;

    root=NULL;
    w=GetWord();
    while(strcmp(w,"***END***")!=0){
      root=insert(root,w);
      w=GetWord();
    }
    treeprint(root);
    return 0;
}

エラーコード

simple-getword.c: In function ‘main’:
simple-getword.c:66:24: warning: passing argument 2 of ‘insert’ makes integer from pointer without a cast [-Wint-conversion]
root=insert(root,w);
^
simple-getword.c:32:14: note: expected ‘int’ but argument is of type ‘char *’
struct node *insert(struct node *x,int a)

出力結果

This is a pen. This is a book.
*END*
�     
�     
�     
�     
�     
�     
�     
�     

修正したプログラム↓

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

  int strcmp(const char *s1,const char *s2);
  struct node{
    char word[50];
    int kaisu;
    struct node *left;
    struct node *right;
};

  char wd[50];

  char *GetWord()
  {
    char a;
    int len;

  a=getchar();
  while(a==' ' || a=='\n') a=getchar();
  len=0;
  while(a!=' ' && a!='\n'){
    wd[len++]=a;
    a=getchar();
  }
  wd[len]='\0';
  return &wd[0];
  }

struct node *insert(struct node *x,char a)
{
  if(x==NULL){
    x=(struct node *)malloc(sizeof(struct node));
    x->word=a;
    x->kaisu=1;
    x->left=NULL;
    x->right=NULL;
  }else if(strcmp(a,x->word)==0)
     x->kaisu++;
   else if(strcmp(a,x->word)<0)
    x->left=insert(x->left,a);
   else
    x->right=insert(x->right,a);
   return x;
  }

void treeprint(struct node *x)
{
  if(x!=NULL){
    treeprint(x->left);
    printf("%s %d\n",x->word,x->kaisu);
    treeprint(x->right);
    }
  }

  int main(void)
  {
    struct node *root;
    char *w;

    root=NULL;
    w=GetWord();
    while(strcmp(w,"***END***")!=0){
      root=insert(root,w);
      w=GetWord();
    }
    treeprint(root);
    return 0;
}


エラーコード

getword2.c: In function ‘insert’:
getword2.c:35:12: error: assignment to expression with array type
x->word=a;
^
getword2.c:39:19: warning: passing argument 1 of ‘strcmp’ makes pointer from integer without a cast [-Wint-conversion]
}else if(strcmp(a,x->word)==0)
^
In file included from getword2.c:3:0:
/usr/include/string.h:140:12: note: expected ‘const char *’ but argument is of type ‘char’
extern int strcmp (const char *s1, const char *s2)
^
getword2.c:41:19: warning: passing argument 1 of ‘strcmp’ makes pointer from integer without a cast [-Wint-conversion]
else if(strcmp(a,x->word)<0)
^
In file included from getword2.c:3:0:
/usr/include/string.h:140:12: note: expected ‘const char *’ but argument is of type ‘char’
extern int strcmp (const char *s1, const char *s2)
^
getword2.c: In function ‘main’:
getword2.c:65:24: warning: passing argument 2 of ‘insert’ makes integer from pointer without a cast [-Wint-conversion]
root=insert(root,w);
^
getword2.c:31:14: note: expected ‘char’ but argument is of type ‘char *’
struct node *insert(struct node *x,char a)

warningとnoteの改善策もよければ教えていただきたいです

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • maisumakun

    2019/07/18 10:09

    コードが途中で途切れているようです(警告の出ている「root=insert(root,w);」の箇所も見当たらなないです)。

    キャンセル

  • mudannkesseki

    2019/07/18 10:13

    ご指摘ありがとうございます。修正いたしました。

    キャンセル

  • SHOMI

    2019/07/18 20:02

    warningやnoteの理由はすべてメッセージに出力されていますが、あなたなりに調べたり考えたりしてみて何がわからなかったのでしょうか。

    キャンセル

  • mudannkesseki

    2019/07/19 11:28

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

    キャンセル

回答 4

0

エラーメッセージに提示されているコードは見当たりませんが。

そのコードでそのエラーが出たんでしょうか


     root=insert(root,w);

insert関数の第二引数は、定義ではint となってますが、char* を渡しています
これをどうにかしましょう

#ところが、  x->tango=a; tango はchar ですね

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/18 15:37

    ありがとうございます。回答を元に編集いたしました。

    キャンセル

  • 2019/07/18 18:12

    修正後でも話になってません。

    C言語の基礎からやり直そうレベルです

    おそらく何かの課題か宿題かでしょうけど、先生やら教授に聞いてみればどうでしょう

    キャンセル

  • 2019/07/19 11:27

    解決しました。ありがとうございました。
    プログラミングやりたくてやってるわけじゃないので。

    キャンセル

  • 2019/07/19 13:06

    まあ、ガッコの授業ならやらない訳にはいかないと思いますが、自分でやらないと身につきませんぜw
    #で、同様なもう一つの質問がありますが、同級生でしょうか

    キャンセル

0

原因としては、警告にある通り、引数の型が合っていないためです。
具体的にはコードが書いてないのでアドバイス不能です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/18 10:18

    あと、これは「文字化け」じゃなくて、文字じゃないものを出力しているのが現状。

    キャンセル

  • 2019/07/18 15:37

    ありがとうございます。回答を元に編集いたしました。

    キャンセル

0

struct node *insert(struct node *x, int a)


第二引数はchar*では?

x->tango = a;


charに文字列は格納できません。
文字列の格納先はx->wordでは?

}else if(a== x->tango)
else if (a == x->tango)
else if (a< x->tango)


C言語は==で文字列比較はできません。strcmp()を使用してください。
比較したいのもx->wordでは?

printf("%5c %5c\n",x->tango,x->kaisu);


でやりたいのは以下では?

printf("%s %d\n",x->word,x->kaisu);

(追記)

struct node *insert(struct node *x,char a)


char*になっていませんが…

x->word=a;


文字列のコピーはstrcpy()です。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/18 15:37

    ご丁寧にありがとうございます。回答を元にプログラムを編集いたしました。

    キャンセル

check解決した方法

-1

コメントにしている行が不必要、aに*をつけるなどして解決しました。

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

  //int strcmp(const char *s1,const char *s2);
  struct node{
    char word[50];
    int kaisu;
    struct node *left;
    struct node *right;
};

  char wd[50];

  char *GetWord()
  {
    char a;
    int len;

  a=getchar();
  while(a==' ' || a=='\n') a=getchar();
  len=0;
  while(a!=' ' && a!='\n'){
    wd[len++]=a;
    a=getchar();
  }
  wd[len]='\0';
  return &wd[0];
  }

struct node *insert(struct node *x,char *a)
{
  if(x==NULL){
    x=(struct node *)malloc(sizeof(struct node));
    strcpy(x->word,a);
    //x->word=a;
    x->kaisu=1;
    x->left=NULL;
    x->right=NULL;
  }else if(strcmp(a,x->word)==0)
     x->kaisu++;
   else if(strcmp(a,x->word)<0)
    x->left=insert(x->left,a);
   else
    x->right=insert(x->right,a);
   return x;
  }

void treeprint(struct node *x)
{
  if(x!=NULL){
    treeprint(x->left);
    printf("%s %d\n",x->word,x->kaisu);
    treeprint(x->right);
    }
  }

  int main(void)
  {
    struct node *root;
    char *w;

    root=NULL;
    w=GetWord();
    while(strcmp(w,"***END***")!=0){
      root=insert(root,w);
      w=GetWord();
    }
    treeprint(root);
    return 0;
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る