C言語について

biwyfiさん

まずは、数値→文字変換を一個にまとめる。
#define itoc(c) ((c)+'0')
で取り合えず代用。
ちなみに、ASCIIコードでない限り、数字の文字が連続している保証は無い。
例えば、「壱弐参...」は不連続だから、個別変換は全くの間違いではない。
（でも、その場合は変換表を使うのが普通）

で、無駄に大量の作業変数を使用しているのは、先の指摘通りなのだが、それ自体は、当人が分かり易ければ問題は無い。
が、同一変化は一つにまとめないといけない。
lとmが共に10倍され、nが1/10になるんだから、本質的に全部一つになる筈。
あと、作業用変数も、実作業直前で宣言するのが妥当。これを行うと、
int n = x/ (10*m);
int l = m*10;
int j = x - n*l;
int k = j / m;
m*=10;
とループ変数がm一個にまとまり、括弧を使って、作業変数を除去すると
int k =(x-(x/(m*10))*10*m)/m;
m*=10;
になる。

で、“m*10”が大量に出るのは、「何か」がおかしいから。
それから、ループ内で変更する変数は、for文で明記するべき。
あと、現状では、数値0が表示出来ない。

正しくまとめると、こうなる。
int base = 10;
for (i=0, m=base; ; i++, m*=base) {
int k = (x - (x/m)*m)/(m/base);
a[i] = itoc(k);
if (x<m) break;
}
数値の0を表示させるので、初期条件を省略した。
そのままでは、当然無限ループになるので、breakで脱出する。
あと、数値が剥き出しなのは気持ち悪いので、変数を一個追加した。
（これで、itocを工夫すれば、16進変換も対応してしまう）

do-while文を好む人も居るが、個人的には、ループ変数が明記されるfor文が好み。
breakでの脱出を嫌う人も居るが、初期条件で省略してあるのだから、何処かでbreakして居るのが自明なので問題は無い。
尤も、この記述では、mがINT_MAXを超えると異常動作する。

元の記述を重視して敢えて奇妙な記述を使ったが、余りを‘x%base’で求めて、‘x/=base’とするのが、本来の姿。
でも、ここは、練習を兼ねて、記述しない事にする。

それから、引数xを保存しながら、作業用途で代用破壊するのは好ましくない。
作業変数に格納保存するなら、作業変数の方を変更するのが筋。

それ以前に、符号は文字は、出力の先頭に付くのだから、最初に処理してしまえば、何の保存も不要。
if (x<0) *y++ = '-';
をxの絶対値化前に追加すれば済む話。

最後のwhile文も、記述が致命的に間違って居るのは、先の回答者の指摘通り。
で、iをループにするのだから、ここも、for文が自然。
for (; i>=0; i--) *y++ = a[i];
*y='\0';
で、完成かな？
「あれっ」と思うかも知れないが、実は先の修正でbreakで脱出する仕様に変更したので、iの初期値が、実は1減っているのだ。

pyon_naniwa2001さん

念のため、
数字0~9の文字コードがこの順で連続しているのは、Cの規格で保証されている。

とりあえず、

>if (k == 0) {
>a[i] = '0';
>}
>...
>else if (k == 9) {
>a[i] = '9';
>}
a[i] = '0' + k;

umiushi711さん

void itoa(char *y, int x)
{
＿＿sprintf(y, "%d", x);
}