C
1　  #include <stdio.h>
2    #include <string.h>
3  
4  
5    int main(){
6  
7            int  i;
8            char x[100];
9            char s[100];
10            char a[256] = "'";
11            char b[6] = "'";
12    
13
14            fgets(s,sizeof(s),stdin);
15            sscanf(s,"%s",x);
16
17            s[1] = "'";
18 
19            strcat(a,s);
20            strcat(a,b);
21 
22            printf("%s",a);
23 
24 
25            return 0;
26

打ち込んだ数字をシングルクオーテーションで囲んでやるというプログラムなのですが

なぜか実行結果が下のようになってしまいます

beike@beike-VirtualBox:~/cj$ ./a.out
1
'1g'

原因がわかる方回答よろしくお願いします

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

回答1件

ベストアンサー

    s[1] = "'";

これが原因。
文字を入れるつもりだろうけど、文字列を代入しようとしている

s[1] = ''';
で、これでもだめで、
s[1] = ''';
s[2] = '\0';
とするべし

fgets(s,sizeof(s),stdin);

        sscanf(s,"%s",x);

んで、これは意味不明。

投稿2018/08/14 05:47

編集2018/08/14 06:10

y_waiwai

総合スコア87747

退会済みユーザー

2018/08/14 05:54

なんとかできたのですが、\'をはじめてお目にしたのですが\'とは何ですか？

y_waiwai

2018/08/14 05:56

' の文字自身。

退会済みユーザー

2018/08/14 05:57

int x; fgets(s,sizeof(s),stdin); sscanf(s,"%d",&x) の方がいいんですかね

退会済みユーザー

2018/08/14 05:58

￥をつける必要があるのですね

y_waiwai

2018/08/14 06:06 編集

x は使ってないようだから、そもそもsscanf の行は不要なんではあとでxつかうというなら sscanf(s,"%s",x); でいいけど

退会済みユーザー

2018/08/14 06:36 編集

流石に、xはint型だから%sだめなのでは、、、。僕は参考書をみて学んだのですが scanfを使わないでこれ使えって、、、。 fgets sscanf についてご教授いただきたいです

episteme

2018/08/14 06:41

char x[100]; じゃなかったのか?

退会済みユーザー

2018/08/14 06:46

すみません僕のコメントでint x; と定義しています

episteme

2018/08/14 06:48

あ、質問コードじゃないのね。それは失礼。

y_waiwai

2018/08/14 06:53

scanf系の関数は危険なので使わないほうがいいです。意図しない入力でフリーズしたりバッファオーバフロー起こしたり。また、引数の型をチェックしないという欠陥もあり

退会済みユーザー

2018/08/14 06:55

scanf系とはsscanfも含まれますか？

y_waiwai

2018/08/14 07:01

含まれます。まあ、まだsscanfは事前チェックで前出の地雷は避けることはできるんですが、それでも使うべきものではないと、私は思ってます。

退会済みユーザー

2018/08/14 07:04

では、何を使えばよろしいのですか？

y_waiwai

2018/08/14 07:10

数値変換させたいと言うなら、他に変換してくれる標準関数はあります strtol とか strtod とか。なんなら自分で変換するような関数組んでもいいですし。

rubato6809

2018/08/14 11:21

例えば入力した＜文字列＞を＜数値＞に変換したいなら sscanf() でも atoi() でも strtol() でも使って変換する必要がありますけど、、、そもそも fgets() で入力した＜文字列＞を、"%s" で＜文字列＞として読み込みなおす必要があるのか？？？私はy_waiwaiさんほどscanf()系を毛嫌いしてないんですが（笑）、 sscanf() する必要は無いと思うんですよ。fgets() で読み込んだ文字列は、そのまま次の処理に使えば良い。まあ、改行文字 '\n' を消したほうが都合良いだろうな、というのはありますけど、それはsscanf() でなく、もっと簡単にできる事なのだし。

rubato6809

2018/08/14 11:44 編集

> s[1] = "'"; > 文字を入れるつもりだろうけど、文字列を代入しようとしている文字列を代入する、という言い方は初心者には誤解される危険性大なので、違う言い方をさせていただきます。「文字列の＜先頭アドレス＞をs[1] に代入する」と言ったほうが良い。文字列を代入する、と、アドレスを代入する、ではイメージが違うと思うんですよ。そこはきっちりさせたほうが良いと思う。