自作関数を用いた文字の反転

質問

タイトルのまんまですが入力された文字を反転して表示する方法がわかりません。

#include <stdio.h>

char reversecopy(char moji[])
{
int i;
char reversemoji[51];

for(i = 50; i > 0; i--) {
reversemoji[i] = moji[i];
}

return reversemoji[50];
}

void main(void)
{
char n[51];

printf("何か文字を入力してください： ");
scanf("%s" ,n);

reversecopy(n);

n[50] = '\0';

printf("%s\n" ,n);

return;
}

ここまでは作ったのですが、表示されるのは反転されていない文字です。どこかおかしい点があったら（絶対あるはずですが）ご指摘いただければ助かります。
よろしくお願いしますm(_ _)m

回答No.2

Werner · ベストアンサー

> reversecopy(n);
ここで文字列を反転しているつもりなのだと思いますが、
関数reversecopy内では
ローカル変数reversemojiに文字列をコピーしているだけで、
引数mojiの文字列は変更していません。
よって関数reversecopyを呼び出しても
引数で与えた文字列nは変化しません。

引数の文字列mojiに変更を加える処理をするか、
関数reversecopy呼び出し側でコピー先reversemojiを準備し
reversemojiも引数とするなどする必要があります。

> for(i = 50; i > 0; i--) {
> 　　reversemoji[i] = moji[i];
> }
また、この記述で反転した文字列をreversemojiに
格納しているつもりなのかもしれませんが、
これは文字列を「後ろからコピー」しているだけで反転はしません。
（コピーと考えても、moji[0]がコピーされていないという問題があります。）

文字列を反転するには関数strlenなどを使用して文字列の長さを調べ、
文字列中の最後の文字が反転文字列中では最初の文字になるようにコピーします。

#1の
> for(i = 50; i > 0; i--) {
> 　　reversemoji[i] = moji[50-i];
> }
では文字列の長さが51の時以外は正しく動作しないので
50の代わりに文字列の長さ-1を使います。
（iが0の時ループを抜けてしまうのでforループの継続条件も変更する必要がありますが）

お礼

ご回答ありがとうございましたm(_ _)m
そういえばstrlenはなんか授業で言っていた気がします・・・
忘れてました。これを使えば良かったんですね・・・
ご指摘ありがとうございます。

回答No.7

yaemon_2006

"反転して表示する"だけなら、

void Reverse(char *str)
{
int i;

for(i = strlen(str) - 1; i >= 0; i --) printf("%c", str[i]);
return;
}

でもいいかな。

お礼

ご回答ありがとうございますm(_ _)m
こんなにさっぱりしているものなんですね・・・
勉強になります！

回答No.6

mac_res

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

#define N_MAX_WORD 51

void
reversecopy(char *buf)
{
int i, j, n;
char c;

n = strlen(buf);
n--;
for (i = 0, j = n; i < n / 2; i++, j--) {
c = buf[i];
buf[i] = buf[j];
buf[j] = c;
}
}

int
main(void)
{
char buf[N_MAX_WORD];
int n;

printf("何か文字列を入力してください： ");
fgets(buf, N_MAX_WORD, stdin);

n = strlen(buf);
n--;
buf[n] = 0;
reversecopy(buf);

printf("%s\n", buf);

return 0;
}

お礼

ご回答ありがとうございますm(_ _)m
ご丁寧にプログラムを全部書いていただいてありがとうございます。
まだ知らない関数もあるけれど勉強になります。

回答No.5

ko_kinta

reversecopy()にscanfした文字列を渡していますが、反転文字を格納しているのはreversecopyのローカル変数reversemojiです。
printfに渡しているのはscanfした文字配列ですから反転されていないのは当たり前ですね。

修正方法としては、まずmain()で反転後の文字を格納する配列を用意してください。
標準入力から文字列を取得するのに、fscanf()はオーバーフローの危険があるのでお勧めしません。
fgets()の方が安全でしょう。

入力された文字列の長さをstrlen()で判断します。

reversecopyには反転前、後のふたつのポインタと文字列長を渡してください。
printfするのはもちろん反転後の配列。

reversecopy()では文字列の最終にはnull stopがあることを忘れないでくださいね。

お礼

ご回答ありがとうございますm(_ _)m
fscanfとfgetsっていうのがあるんですか・・・
まだ知らないですが調べてみます。
ご丁寧にありがとうございました。

回答No.4

yaemon_2006

>for(i = 50; i > 0; i--) {
>reversemoji[i] = moji[i];

　"文字列の終端"と"配列の終端"は、違います。

お礼

ご回答ありがとうございますm(_ _)m
うーん、まだ良くわかりませんが調べてみます。
これも勉強ですから。

回答No.3

dra2jp

配列要素を宣言しなくてもきちんと格納されることを利用した書き方や
関数の値受け渡しはきちんとできているのに
コピーしかせずに終了しているソースを見たら

学校の課題だって事まるわかりですよ。
学校で、このプログラムの続きを書いて来いといわれてわからないから
そっくりそのまま聞いたんでしょう。

ここでわからない部分を聞くのはいいですが、全部のソースを書いて答えだけ教えてくれというのはやめましょう。
そうじゃなくて、どうしてもわからない一部だけを聞くようにしてください。
プログラムを勉強しているという事は、仮にも好きでその専門のコースに進んでいるんでしょう。
自分の力になりませんよ。
と、もしかしたらいやみに取られるような言葉はこの辺にして。

このプログラムどおりじゃなくていいのでしたら
文字列処理の関数は沢山ありますからそれを利用すると便利です。
includeの意味はわかりますか？
文字列処理でしたらstring.hをインクルードすれば
様々な関数が使用できるようになります。
例えば２つの文字列をくっつけるような作業もたった１つの行ですみます。

いやみでいっているんじゃないです、私も過去にそういう傾向があって自分のためにならなかったので
同じ事をしてほしくないだけです。
頑張ってください。

お礼

ご回答ありがとうございますm(_ _)m
一応書き込んだものは自分で書いたものなのですが、配列要素を宣言しなくてもきちんと格納されることを利用した書き方というのは
reversecopy(n);
のことですか？
このやり方は、前に質問をして答えていただいた方から学んだものです。
あと、自分も質問の仕方が曖昧で誤解を招いてしまったことを謝ります。「どこかおかしい点があったら（絶対あるはずですが）ご指摘いただければ助かります。」というのは、
「おかしいところがあったらヒントを出して下さると助かります」
と言うつもりで書いたのですが、少し曖昧でしたね。すいませんでした。

というかきっと悪いのは学校の課題の出し方です。
入力された文字列を反転させて表示する関数を作れ、とだけしか言われなかったので（いつもこんな感じですが）授業で習った関数だけしか使ってはいけないのかどうかもわかりません。

あぁ、なんだか愚痴みたいになってしまいましたが、ありがとうございましたm(_ _)m
これからもがんばります。

長文ですいませんでした。

回答No.1

silverbear

このPCにコンパイラが入っていないのと、ろくに見てないのではっきりとはいえませんが。

for(i = 50; i > 0; i--) {
reversemoji[i] = moji[i];
}

を

for(i = 50; i > 0; i--) {
reversemoji[i] = moji[50-i];
}

のようにしたらいいんじゃないかと思います。

お礼

えーと、あとで自分もこう考えたのですが↑の方のご回答にはそれではできないということです。
なにはともあれご回答ありがとうございますm(_ _)m