search

hno-guchi / refactor_tetris

0 stars 0 forks source link

review feedbacks #2

Open snara-42 opened 2 years ago

snara-42 commented 2 years ago

on -> is_on timer -> interval

snara-42 commented 2 years ago

各関数がどのファイルにあるのかちょっとわかりづらいかも

hno-guchi commented 2 years ago

printf()関数で出力する文字列を、別で定義しておくと、よりメンテナンスしやすくなるらしい。 関数名と同じファイル名を作成し、分割すると良いかも。

snara-42 commented 2 years ago

イベント駆動では on_something という関数にすることが多い

snara-42 commented 2 years ago

handle_lines -> drop_lines など

snara-42 commented 2 years ago

400000 -> #define INITIAL_INTERVAL 400000

hno-guchi commented 2 years ago

game.current -> game.tetromino print_gameover(); -> game_end();など game_loop.c -> game_flow.c key_handle -> react_key

関数名に動詞を入れると良い。 初期化で与える値は、defineで設定すると良い 　ex) Timer, timer_decrease, point=100など 関数の中で変更しない引数は、constを付けると良い。 Left, right. Downは、接頭辞付きのdefine定義が良い？？ handle_linesは、分けた方が良い。 shape座標系とfield座標系 long longは使わなくとも良い？？ ( t_timeval も long long にしてもいい) Main関数内の関数名の非対称性

hno-guchi commented 2 years ago

‘#’ ‘.’をdefineすること。 timerの命名について。（intervalなどに変更すること） rotate_mino()へのコメントが欲しかった。