C言語 入力した文字列を逆にしたいが表示されない

Question

前提・実現したいこと

入力された文字列を逆にしたいのですが最後にprintf関数で表示されません。
例えば abcdならdcbaとなるようにしたいです。

自分が想定しているこのプログラムの動き方

scanfで入力された文字列をchar型関数strに格納。
最初のfor文で文字数をカウントし、int型変数iに格納
int型変数flagにiを2で割った余りを格納する。あまり１の時は文字数っが偶数で０のときは奇数であると判断する。
文字数が奇数のとき、つまりflag＝０のとき「j番目の文字をi-j番目の文字と入れ替える」という動作を文字列の半分まで繰り返す。
文字列が偶数のとき、つまりflag=1のとき「j番目の文字をi-j番目のもじと入れ替える」という動作を文字列の半分ー１まで繰り返す。
その結果をprintf関数で出力する。

該当のソースコード

#include<stdio.h>
#define num 100
int main()
{
　char tmp,str[num];
　int i,j;
　int flag
　
printf("文字列を入力してください。:");
scanf("%s", str);

for (i = 0; str[i] != '\0' ; i++) {
　}
　flag = i % 2;
　
if (flag == 0) {
	for (j = 0; j < i / 2; j++) {
	        tmp = str[j];
		str[j] = str[i - j];
		str[i - j] = tmp;
　　　　　　　　　  }
                }
	

  else{
	for (j = 0; j < i / 2; j++) {
		tmp = str[j];
		str[j] = str[i - j];
		str[i - j] = tmp;
		break;
		}
	}
		printf("逆の%s", str);
	return 0;
}
		

	

Comments

[BeatStar]

まず、『自分が読み書き出来る人間用の言語で、コードの内容を説明』してみてください。

つまり、『ロジックを説明せよ』です。

[fana]

とりあえず不規則な空行多すぎて見づらい．
なんの意図でこんな（わざわざ"見たくない感"を誘発するような）形で提示しているの？

[HOTEISON_]

見づらいコードですみません。いろいろ入れた削除したりしているうちに改行が多くなってしまいました。見やすさを意識して記述します！！

Answer 1

考えすぎじゃないかなー... 文字列長の奇遇は気にせんでえぇ思う。

// strを反転する
void str_reverse(char str[]) {
  int lo = 0;
  int hi;
  for (hi = 0; str[hi] != '\0' ; hi++) {} // hi:文字列長
  while ( lo < hi ) {
    --hi;
    char tmp = str[lo];
    str[lo] = str[hi];
    str[hi] = tmp;
    ++lo;
  }
}

// おためし

#include<stdio.h>
#define num 100

int main() {
    char str[num];

    printf("文字列を入力してください。:");
    scanf("%s", str);
    str_reverse(str);
    printf("逆の%s", str);
    return 0;
}

[お小言] なんもかんもmainに書き下すのは(ビギナによくある)悪い癖。
やりたいことは"文字列の反転"、ならば"文字列を反転する"関数を書き、
それをmainから呼び出そう。

Comments

[HOTEISON_]

関数ってこうやって書くと今知りました。知識不足ですいません。丁寧な解答ありがとうございました。

Answer 2

関数の書き方を知らず、現在の知識だけで書くとすれば、

#include <stdio.h>

#define num 100

int main(void)
{
	char tmp, str[num];
	int i, j;

	printf("文字列を入力してください。:");
	scanf("%s", str);

	for (i = 0; str[i] != '\0'; i++) ;

	for (j = 0; j < --i; j++) {
		tmp = str[j];
		str[j] = str[i];
		str[i] = tmp;
	}
	printf("逆の文字列: %s\n", str);
	return 0;
}

Comments

[HOTEISON_]

たしかにこれなら自分の知識でもできると思います！！ただfor文の中で二つの変数を変化させるという発想がそもそも浮かんでこなかったためやはり皆さんに質問してよかったと思いました。
丁寧な回答ありがとうございました。

Answer 3

すでに解決済みになっていますが、気になるので回答します。

---

Ⅰ. 基礎を理解すべし

基礎が出来てから応用が出来ます。
でも質問を見る限り、『基礎が出来ていない』気がします。

Ⅱ. 用語はちゃんと使うべし

scanfで入力された文字列をchar型関数strに格納

char型『関数』ではなく『配列』では?

関数と配列は別物です。

一応、コードと文章から『配列』であることはわかりますが、
文章だけだと回答者はわからなくなります。

場合によっては『いや、これは変数だろ』とかってわかる事もありますが、
基本的にはわかりにくい。

そうなると質問者の意図と回答者の解釈がすれ違って、頓珍漢な回答になってしまいます。

人間同士ですらこれですから、プログラミングなんてきついです。

プログラミングは「機械とのコミュニケーション」です。

なので用語はちゃんとしましょう。

Ⅲ. 可読性を上げるべし

質問にあるコードではインデント(半角スペースやTabキーでやるアレ)が滅茶苦茶です。

読めなくはないですが、読む気が失せます。

日本語で書けば、

オイラは数学が苦手なんだな。
だけど　　　　　お金は　　　　好 　　　　きなんだな。
だから


東  京 さ出て、


お                 金  を
も
う
けるんだな。

のような感じに近いです。(内容は適当)

読む気、失せますよね?

初心者の内は仕方ないですが、会社での開発はチーム開発のようです。

そうなると複数人で開発することに。

でもそういう読みづらいものを書くとどうなるでしょうか。

仮に自分ひとりで開発するとしても、プログラムは書いて終わり……じゃなく、
保守とかもします。

一か月前の自分が書いたコードなんて忘れているのが当たり前のレベルです。
なので、読みやすいコードを書くように心がけましょう。

Ⅳ. コードを読むときは『この行が何をしているか』を考えながら読むべし

他人が書いたコード、自分が書いたコード関係なく、コードを読むときは、
『一行レベルで、何をしているのかを考えながら読む』です。

たとえば、ソースコードとしてのファイルをバックアップ(あるいは単なるコピー)を取っておいて、
一行レベルでコメントにする。とかです。

実際にやってみましょうか。

// 変数iの初期値: 0, 配列の中身が終端文字でない間, +1しながらループ
for (i = 0; str[i] != '\0' ; i++) {
　}
  // フラグ? は、iを2で割った余りとする
　flag = i % 2;
　
// フラグ? が 0 (= 偶数) なら
if (flag == 0) {
    // 変数j の初期値: 0, j　が iの2分の1　未満の間, +1 しながらループ
    for (j = 0; j < i / 2; j++) {
            // 変数 tmp に str[j] の値をセット
            tmp = str[j];
        // str[j] は str[i - j] の値とする
        str[j] = str[i - j];
        // str[i-j] は tmp の値とする
        str[i - j] = tmp;
        // → つまり、str[j] と str[i - j]を入れ替える
　　　　　　　　　  }
                }

といった感じです。

これを日本語でも中国語でも英語でも韓国語でもスペイン語でもなんでもいいので、
自分が読み書きできる言語で書き下す。

1. ユーザ入力によって文字列を受け取り、配列 str にセット
2. 配列の長さを forを使って調べる(= 変数 i にセット)
3. (2)で得た長さを2で割ったときの余りを取得( flagにセットし、偶数・奇数を判断するため )
4. 長さが偶数なら
    4.1. 変数j の初期値: 0, j　が iの2分の1　未満の間, +1 しながらループ
    4.2. str[j] と str[i - j] を入れ替える
5. それ以外(= 長さが奇数)なら
    ...
6. 結果(strの値)を出力する

と言う風にやる。(疑似コード?)

これを現実世界で、手作業でやってみる。
数学の問題とかのような感じで。

脳内でもいいです。とにかくやってみる。

そうすると、(5.1) に相当する部分が(4.1)と同じになっています。

でも質問者さんのお考えでは、『文字列の半分ー１まで繰り返す』ですよね?

計算が合いません。

このように、自分が考えたロジックとの差異を見付けるためにもこういう考え方を学ぶべきです。

Ⅴ. 『現実世界でならどうするか』を考えるべし

プログラミングは元々、『現実世界でのシミュレーション』です。

『プログラムは魔法でもなんでもなく、人間が現実世界でやっていることを逐一指示されながら処理しているだけの代物である』です。

よって、『現実世界でならどうするか』を考えるのです。

そうすると、

[言い換えQ]

ある人(Aさん)が単語を言うので、それの逆から読んだ単語を書いてくれ。

例:
point -> tniop

この場合、どうしますか?

私なら、

1. ホワイトボードを用意する
2. ホワイトボードに Aさんが言った単語をを書く
3. (2)の単語を後ろから読み込んで
    3.1. 一文字ずつ余白に書いていく
4. 余白に書いた単語が結果的に答えになる

ですかね。

別に『偶数なら~』とかやらなくてもこれぐらいならできますよね。

単純にこれをやればいいだけです。

(上記はホワイトボードですが、ノートでも黒板でもなんでもいいです)

シンプルにできるならシンプルな方が望ましいです。
(アルゴリズム的に無理だったり、計算量を意識して……とかだと別ですが)

シンプル イズ ベスト!

注意!: 別に質問にあるような考え方でもいいです。(ロジックと仕様さえ合っていれば)
質問者さんなりに考えてのロジックなので問題ありません。
別にダメということではなく、あくまで私の考えです。