C言語で文字列を表示する方法について

Question

前提・実現したいこと

C言語の問題集を学習中なのですが、文字列を表示する関数を作成したのですが、
最後の1文字が化けてしまいます。初歩的な質問ですいませんが、何が原因なのか教えて頂けると助かります。

該当のソースコード

#include <stdio.h>

void put_string(const char *s){
while(*s++)
putchar(*s-1);
}

int main(void){
char *s = "ABCDEFG";
put_string(s);

return 0;   

}

Answer 1

putchar(s-1);→putchar((s-1));
「追記」

usr ~/Project/test % ./a.out
BDFHJL�usr ~/Project/test % cat tst.c
#include <stdio.h>

void put_string(const char *s)
{
    while (*s++)
        putchar(*s - 1);
}

int main(void)
{
    char *s = "ACEGIKM";
    put_string(s);

    return 0;
}
usr ~/Project/test %

Comments

[cateye]

*s-1 は(*s)-1と解釈されます。なので、*(s-1)でポインタをの一個手前に移動してからその中身の出力します。

[err0r]

分かりやすくご解説いただいてありがとうございました。(*s)-1の状態でもABCDEFまで表示出来ていたのはJISコードで10進数の文字から１を引かれていたからで、最後の文字が化けていたのは、ヌル文字（値0）から1を引いた物を表示しようとしたから。。　という理解で正しいでしょうか？
'A'　=　41,　'B'　=　42,　'C'　=　43　...

[cateye]

前の状態で文字列を変えてやってみました。→「追記」
で、“ヌル文字（値0）から1を引いた”
１，最後の文字'G'はループに入る。s++でｓは'\0'を指してる。
２．'\0'-1は0xffになる。
・・・ということで、0xFF(-1)を出力しようとしたといううことです。

[y_waiwai]

while(*s)
putchar(*s++);
}
あるいは
while(*s)
putchar(*s);
s++;
}

[err0r]

教えて頂きありがとうございました。
-1は0xFFになるんですね！知りませんでした。

y_waiwai様もコードの修正案ありがとうございます。無駄な演算子がないのでこちらの方が効率が良いですね。こういうコードがすぐ作れるよう努力したいです！