c言語 文字列 暗号化

Question

文字列の各アルファベットを３文字ずつ後ろのアルファベットにシフトし，文字列の並びを反転したいのですが

#include<stdio.h>
int main(void)
{
  char str[100];
  int i;
  printf("文字を入力してください:");
  gets(str);
  for(i=0;str[i]!='\n';i++)
  str[i]+=3;
  while(i>0)
  {
     i--;
     putchar(str[i]);
  }
  putchar('\n');
  return 0;
}
  

とプログラムし実行すると

文字を入力してください:COMPUTER
zsh: segmentation fault 

とエラーしてしまいます。
どうすれば処理できますでしょうか

Answer 1

gets関数は、バッファオーバーランするので（100文字以上入力すると変数の続きのメモリエリアを破壊する）使わない関数です。
代わりに入力バイト数上限を指定できるfgets関数を使います。fgetsは\nを残します。おそらくfgets用のサンプルを目にして参考にしたんでしょうね。

Answer 2

C言語関数リファレンス - gets(標準入力からの文字列入力)

結果文字列には'\n'は含まれません。
文字列終端の'\0'をチェックしましょう

Comments

[mdV7jWTlQaT74kE]

ありがとうございます。とてつもない凡ミスかましました。もう一度理解しなおそうと思います