c0a21135/projexd

プロジェクト演習テーマD

projexd's Issues

ほぼ同じオブジェクト

ProjExD/ex04/dodge_bomb.py

Line 66 in 62bde5a

b1obj_sfc = pg.Surface((150, 150)) #オブジェクト

同じオブジェクトを二つ作るのなら、関数やリストを用いてもっとすっきりかけるかなと思います。

importする順番を変えましょう。

ProjExD/ex05/fight_kokaton.py

Line 1 in f00736f

import pygame as pg

クラス名を変更しないPythonの機能をimportした後に改行し、asでクラス名を変更したクラスをimportしましょう。（講義資料P13分かりやすい資料参照）

if maze_list[mx][my-1] == 0:wo

ProjExD/ex03/maze.py

Line 16 in fdc34c7

if maze_list[mx][my-1] == 0: #移動先が床の場合は座標の変更と画像の差し替え

タプルではなくリストで管理してください。

ProjExD/ex04/dodge_bomb.py

Line 26 in 903063f

# bomb_list.append((bomb_sfc, bomb_rct, vx, vy))

中身の値のvxやvyが変更される値なのでタプルによってエラーになっています。
そのため、リストで管理すれば動作すると思います。

READMEのタイトルが第4回になっています

ProjExD/ex05/fight_kokaton.py

Line 1 in 792047b

import random

READMEのタイトルが第4回になっています！！

改行入れよう

ProjExD/ex01/alphabet.py

Line 6 in 59c10a8

if maze_list[mx][my-1] == 0:をif maze_list[mx][my-1] != 1:にする

ProjExD/ex03/maze.py

Line 16 in fdc34c7

if maze_list[mx][my-1] == 0: #移動先が床の場合は座標の変更と画像の差し替え

if maze_list[mx][my-1] != 1:にすれば、それぞれのスタートやゴールの色替えも容易になり、他の判定を入れる際に拡張性が増すと思います。

空行は2行入れましょう(12行目も同様)

ProjExD/ex05/fight_kokaton.py

Line 7 in 792047b

空行が1行になっています。12行目も同様です。
それぞれの空行を2行に変更しましょう。

==Trueならいらない？

ProjExD/ex03/maze.py

Line 66 in fdc34c7

if game_flag == True:

「 == True」を無くして、「 if game_flag: 」のみで充分な気がします。

7行の全く同じコード

ProjExD/ex02/calc.py

Line 105 in 55af261

button = tk.Button(root, text=ope, font=("", 30), width=4, height=2)

7行あってここまで同じだと、関数にしてもいいかもと思う。
(イテレータ, r,c )を引数にして、1行上からfor文ごと関数にするとか。

変数について

ProjExD/ex03/maze.py

Line 107 in 430e81c

sec, ms = 0, 0

変数の定義がばらばらで分かりにくいのでどこかにまとめておくと分かりやすいかもです

改行のみのprint()は避けましょう

ProjExD/ex01/alphabet.py

Line 26 in 59c10a8

print()

一つ前の文章に"\n"使いましょう

定義の上にはimportのみにしましょう

ProjExD/ex02/calc.py

Line 4 in 55af261

root = tk.Tk()

rootなどの記述が定義の上に設定されてしまっているため、定義の後にrootなどの記述を入れるようにしましょう。
その際に、if name == "main":を利用して他ファイルからimportされたときではなく，コマンドラインから実行されたときにのみに動くようにしてもいいと思います。

importするモジュールについて

ProjExD/ex01/alphabet.py

Line 2 in 59c10a8

import time

資料にはtimeモジュールではなくdatetimeモジュールが使われていたので変更しておくと良いと思いまする。