C言語のプログラミングでつまずいてます

質問

ポインタをりようする。string.hとctype.hに含まれる関数は使用しないこと。

以下の実行例のように、文字列を受け取り、アルファベットの大文字を小文字にして表示する関数を作成したいのですが

実行例：
Original string is <AbCdE1f2>
Modified string is <abcde1f2>

#include <stdio.h>

#define NUMOFCHAR 10

void print_char(char *ptr);

int main(void)
{
char string[NUMOFCHAR+1];

printf("Input a sentence (# of letters <= %d): ", NUMOFCHAR);
fflush(stdout);
scanf("%s", string);
string[NUMOFCHAR] = '\0';

printf("Original string is <%s>\n", string);
printf("Modified string is <");
print_char(string);
printf(">\n");

return 0;
}

void print_char(char *ptr)
{
int i=0;

for(i=0 ; i<NUMOFCHAR+1 ; i++){

if( *ptr>='A' && *ptr<='Z'){

*ptr += 'A'-'a' ;
}
ptr++ ;
}
printf("%s",ptr);
}
よ書いたのですがビルドえらーがおこります。
どこがだめなのかわかりません。。。

C言語に詳しい人でわかるかた、どこがいけないのか、なぜいけないのか、どうすればいいのか詳しくおしえてください。
お願いします

回答No.1

wormhole · ベストアンサー

試しにコンパイルしてみたところ問題なくコンパイルできましたけど。
ビルドエラーが起こるということですが
どういうエラーメッセージが出力されていますか？

＞C言語に詳しい人でわかるかた、どこがいけないのか、なぜいけないのか、どうすればいいのか詳しくおしえてください。

どこがいけないかで、ひとまずいえることは
・出力されているエラーメッセージを読んでいないこと。

お礼

すみません。
なんかウィンドウズでやるとエラーないのにエラー表示されることがあるんです・・・
もう一回コピーしてはりつけたらエラーはありませんでした。
でも思惑通りに動作してませんでした。
大文字小文字のところはもう理解したので大丈夫ですが、ほかがまだなので詳しくかいていただけるとありがたいです。

回答No.8

TT414

>scanf("%s", string);
>string[NUMOFCHAR] = '\0';//完全に無駄です、この文は削除しましょう。
>printf("Original string is <%s>\n", string);

>int i=0;
>for(i=0 ; i<NUMOFCHAR+1 ; i++){//バグおよび無駄です、「for(i=0 ; *ptr ; i++){」にしましょう。
>if( *ptr>='A' && *ptr<='Z'){

void print_char(char *ptr){
int i;char *s;
for(s=ptr;*ptr;ptr++)
for(i=0;i<26;i++)
if(*ptr=="ABCDEFGHIJKLMNOPQRSTUVWXYZ"[i]){
*ptr=abcdefghijklmnopqrstuvwxyz[i];
break;
}
printf("%s",s);
}

回答No.7

Tacosan

if( *ptr>='A' && *ptr<='Z')
で「大文字かどうか」を正しく判定できるとは限らないんだけど, そのことは無視していいの?

お礼

無視したらだめです…
どうしたらいいでしょうか?

回答No.6

和泉 博（@hiroshi09s）

＞どこがいけないのか、なぜいけないのか、どうすればいいのか詳しくおしえてください。

void print_char(char *ptr)　で受けて、ptr++ ; のようにポインタを変えておきながら、printf("%s",ptr);　として変わってしまっている ptr を出力しているところが大きな誤りです。

int i; を尊重して　void print_char(char *ptr)　はポインタで受けるのではなく、文字配列　void print_char(char ptr[])　として受けましょう。

void print_char(char ptr[]){
int i;

for(i=0 ; ptr[i] != '\0' ; i++){ //ヌルまで繰り返す
if( ptr[i]>='A' && ptr[i]<='Z'){
ptr[i] += 'a' - 'A' ;
}
}

printf("%s",ptr);
}

お礼

ありがとうございます!

書き忘れててすみません
書き換える内容は下記の部分だけでお願いできますか?
参考にしてまた明日やってみますのでお暇があればまたお願いします
ありがとうございました
{
int i=0;
for(i=0 ; i<NUMOFCHAR+1 ; i++){
if( *ptr>='A' && *ptr<='Z'){
*ptr += 'A'-'a' ;
}
ptr++ ;
}
printf("%s",ptr);
}

回答No.5

wormhole

＞全然くわしくないので質問もうまくできません。。。

くわしいくわしくないは関係ないと思いますけど。
期待する結果と実際の結果がどう違うのかも説明しきれませんか？

＞引数の文字列を変換して返す
＞と
＞引数の文字列を変換したものを出力
＞の違いもよくわかりません。
＞結局同じ結果になるようなきがします・・・＞＜

print_char()の処理からすれば同じにはなりませんよ。
前者だと少なくともprintf()で出力することはありませんし
後者だと*ptr += 'A'-'a';のように引数の文字列を書き換える必要はありません。

前者と後者の違いが具体的にわからないのであれば

main()内のprint_char(string);のあとにprintf("%s", string);をいれてみてください。
出力するだけのprint_char()が、stringの内容を書き換えている(変換したものを返している)のが確認できると思います。

お礼

前者だと少なくともprintf()で出力することはありませんし→returnを使うということですか?(どのように使うかわからなくてprintfでも同じかと思ったのでprintfつかいました)

後者だと*ptr += 'A'-'a';のように引数の文字列を書き換える必要はありません。→ではどうするのですか??全然わかってなくてすみません…

main()内のprint_char(string);のあとにprintf("%s", string);をいれてみてください。
出力するだけのprint_char()が、stringの内容を書き換えている(変換したものを返している)のが確認できると思います。
→やってみます

また明日やってみますのでよければ明日もお願いします

何度も回答ありがとうございます

回答No.4

wormhole

＞でも思惑通りに動作してませんでした。
＞大文字小文字のところはもう理解したので大丈夫ですが、ほかがまだなので詳しくかいていただけるとありがたいです。

あなたが思われている「ほか」とは何でしょうか？
私が思うとこの「ほか」をあげると動作結果とは直接関係のない
とこまで及んでしまいますけど。
どういう事か、あえて1点あげるなら
・print_char()は「引数の文字列を変換して返す」
「引数の文字列を変換したものを出力」どちらが目的ですか？
関数名からするとprint_char()は後者の出力するのが目的だと思いますが、
実際のコードは引数の文字列そのものを加工してしまっているので、
前者と後者2つの目的を持つことになってしまってます。

お礼

すみません
全然くわしくないので質問もうまくできません。。。

引数の文字列を変換して返す
と
引数の文字列を変換したものを出力
の違いもよくわかりません。
結局同じ結果になるようなきがします・・・＞＜
具体的にどうちがってくるのでしょうか・・・；

違うのなら目的は後者だとおもいます。

プログラムをどうかきかえればうまくいくのでしょうか？？＞＜

回答No.3

not_ace

ビルド自体は通ると思うのですが・・・どうなんでしょう？

プログラムが上に張られたものそのものならビルドは通りますが、
意図通りの動作はしないと思われます。

以下要修正点

大文字と小文字の大きさは逆です。
誤：*ptr += 'A'-'a' ;

正：*ptr += 'a'-'A' ;

printf("%s",ptr);
for文のなかでptrを直接書き換えているので文字列の末尾を
printfが参照しようとします。

for文で回す前にptrの値を別の変数に保存するなりしましょう。

お礼

ありがとうございます＾＾
前半は理解しました。
for(i=0 ; i<NUMOFCHAR+1 ; i++){

if( *ptr>='A' && *ptr<='Z'){

*ptr += 'A'-'a' ;
}
ptr++ ;
}
printf("%s",ptr);
}
こうはんですが、ptr++で文字列の最後までいくからそれをプリントしてしまうということですよね。
ｐｔｒ＋＋がないとptrの最初の文字をさすことになりますよね？。
ということは、ptrを置き換えなくても最後のprintf("%s",ptr);を１つ上の中かっこ｛｝のなかにいれればいけますか？
あと初歩的で申し訳ないのですが、このptrというのはデータ・アドレスでいうとアドレスをさしていますよね？
なぜ最後のprintf("%s",ptr);はprintf("%s",*ptr);にならないのでしょうか？*ptrはデータなので配列のなかみを表示するとかんがえているのですが・・・

回答No.2

kmee

まずは、その「ビルドエラー」のメッセージを読むこと。
プロフェッショナルでも、そこから始めます。
質問するにしても、エラーメッセージがあるのと無いのとでは、回答する手間が段違いです。

もっとも、このプログラムを実際やってみたところ「ビルドエラー」にはなりません。
実行自体は正常に終わって、結果が期待通りでない、ということになりそうです。
実行時のエラーになる可能性はあります。

> *ptr += 'A'-'a' ;
x+=y;は x = x+y ;です。なので
*ptr = *ptr + ('A'-'a');
です。*ptr='A'だった場合、この計算がいくつになると思いますか?上の式から括弧をはずしてみましょう。

下から2行目
> printf("%s",ptr);
このときにポインタptrがどこを指しているか、よく考えてください

> printf("Modified string is <");
> print_char(string);
> printf(">\n");

を

printf("Modified string is <");
print_char(string);
printf(">\n");
printf("Modified string(2) is <%s>". string);

とすると、ヒントになるかもしれません

お礼

ありがとうございます。
前半は理解しました。
でも、後半がいまいちよくわかりません。
教科書は何度もよんでるのですが、ポインタについてあまりつかみきれてないので・・・
ptrはアドレスを、*ptrはデータをさしているなど、どれがデータどれがアドレスかはわかるのですが・・・