文字列中の空白部の読み込みの方法を教えてくださいm(_ _)m

質問

いつもお世話になっています！！キーボードから以下のような文字列を読み込み、大文字のみをカウントするプログラムを作りたいのですが、空白部になると次の文字を判定せずにプログラムが停止してしまいます。。。どなたかアドバイスをお願いしますm(_ _)m
○文字列
My name is Marly.

○プログラムソース
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

int main(void){

char *s;
int i,j=0,m;

s=(char*)malloc(sizeof(char));

printf("please iput sentens\n");
scanf("%s",s);

m=strlen(s);

for(i=0;i<m;i++){
if( 0x41 <= s[i] && s[i] <= 0x5A) j++;
}

printf("%d",j);

free(s);
return(0);
}

回答No.1

dtyu · ベストアンサー

scanf()した後のsやmの値を表示してみればいいんじゃないですか。

お礼

ありがとうございます。
自分の考えていたエラーがそもそもおかしかったことに気づけました。

回答No.5

mac_res

scanf(3)を使った別解です。バッファオーバフローの危険性がある点、文字列の長さがBUFSIZの制限を受ける点でNo.3に劣ります。
-- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< --
#include<stdio.h>
#include<ctype.h>

int
main(void){
char buf[BUFSIZ];
char *p;
int i;

printf("please iput sentens\n");

scanf("%[^\n]", buf);
i = 0;
for (p = buf; *p != 0; p++) {
if (isupper(*p)) i++;
}
printf("Number of upper char = %d.\n", i);
return(0);
}

補足

No3に引き続きお願いします。
"%[^\n]"とは「文字列として改行まで読み込み」という意味でしょうか？

回答No.4

mac_res

fgets(3)を使った別解です。文字列がBUFSIZの長さに制限されている点でNo.3に劣ります。
-- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< --
#include<stdio.h>
#include<ctype.h>

int
main(void){
char buf[BUFSIZ];
char *p;
int i;

printf("please iput sentens\n");

fgets(buf,BUFSIZ,stdin);
i = 0;
for (p = buf; *p != 0; p++) {
if (isupper(*p)) i++;
}
printf("Number of upper char = %d.\n", i);
return(0);
}

補足

３つも解答を用意していただいてありがとうございます。
失礼ながら私は最近からC言語の勉強を始めたので分からないキーワードが多々あります。。。もしよければ各解答に質問をするので答えて頂けないでしょうか？
BUFSIZって何のことでしょうか？別のサイトをみたら「バッファのデフォルト時のサイズ」と書かれていたのですが、そもそもバッファが何かよく分かっていません。。

回答No.3

mac_res

scanf(3)では空白の前までしか読みませんし、バッファオーバーフローの危険性の問題があります。
数値による大文字の判定はシステム依存性があります。
-- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< -- 8< --
#include<stdio.h>
#include<ctype.h>

int
main(void){
int c;
int i;

printf("please iput sentens\n");

i = 0;
while((c = getchar()) != EOF) {
if (c == '\n') break;
if (isupper(c)) i++;
}
printf("Number of upper char is %d.\n", i);
return(0);
}

補足

EOFとは何ですか？別のサイトで見たら「ファイルの終端を示す」と書かれていたのですが、getchar関数は標準入力なのでファイルとは関係ないような。。。

回答No.2

smat7

空白の判定というよりsの領域確保に問題があります
s=(char*)malloc(sizeof(char));
これではsに割り当てられるのは１バイトしかありません
scanfを呼んだ段階でメモリ破壊がおきています
必要とする最大文字数分を,例えば
#define MAX_STR 256
なら
s=(char*)malloc(sizeof(char)*(MAX_STR+1));
などとして確保してください

お礼

ありがとうございます！
なるほど動的メモリというだけでただmallocを使えばいいというわけではないんですね。。。勉強になりました！！