学校でC言語の基礎学んだから見て

#include <stdio.h>
#include <stdlib.h>

int main() {

char* input = malloc(sizeof(char));
if (input == NULL){
perror("ERROR: input memory allocation failed.");
exit(EXIT_FAILURE);
}

int ch = 0;

int i = 0;

printf("Please give an integer: ");

while ((ch = getchar()) != EOF && ch != '\n'){
input = realloc(input, (size_t)(i+1) * sizeof(char));
if (input == NULL){
perror("ERROR: input memory allocation failed.");
exit(EXIT_FAILURE);
}
if (ch < 48 || ch > 57){
free(input);
fprintf(stderr, "ERROR: Please give a valid integer.\n");
exit(EXIT_FAILURE);
}
input[i] = ch;
i++;
}

input[i] = '\0';

printf("Given integer is: %s\n", input);

free(input);

return 0;
}
　

配列の長さが1足りてないからNULL文字入れるとこでバッファオーバーフローしてる
それ以外はよく書けてると思う
ちゃんと細かいエラーハンドリングとfree忘れないの偉い

ちゃんとputs使ってね

パッと見putsに置き換えられそうな所無くない？
どこのことを言ってるんだろう

よくこんなん手打ちしていられるなぁと感心するばかり
絶対どこかでタイプミスする自信ある

>>5
タイプミスはエラー出るからわかるんだ
とにかく書いて後から手直しするだけだからそんなに神経使うものではないんだ

>>4
"Please give an integer:\n"でいいだろ
というか自動メモリを確保してる場合じゃないしそれ以前の問題だけど
楽しければ言うほど悪くないか

どんどん書いて

Cなんもわからんからこれが何をしてるのか教えて

>>7
そこ改行入れたくないんじゃない？
入力待ちでカーソルが次の行に来ちゃうのはCLIアプリとして変だと思う

メモリ確保については確かに改善できるね
入力された整数をintやlongに突っ込む前提で桁数決め打っておいて上限超えたらエラー吐くようにしてもいいし、桁数に上限設けたくないならバッファサイズを倍々にしていくやつでメモリ確保の計算量をO(N^2)からO(N)に落とすのが普通かな
どっちもいいプログラミングの練習になると思う

Cなんもわからんからこれが何をしてるのか教えて

>>9
fputs("Please give an integer: ", stdout);

>>11
This is a pen．的なプログラムだ

>>12
それprintfと等価だけど置き換える意味あるの？