alarm_sound(self) 関数の中のpygame.mixer.music.play(loop_count)が二個ある理由

pusaitou commented 4 years ago

ここのpygame.mixer.music.playを二個置くことには意味があります pygame.mixer.music.play()で流された曲がpygame.mixer.music.stop()を使わずに終了した後に一回だけpygame.mixer.music.play()が呼ばれても曲を流すことはありません pygame.mixer.music.play()が二回目に呼ばれると音を流します pygame.mixer.music.play()で流された曲が終了したあとpygame.mixer.music.stop()を呼び出してもやはり最初の一回目はpygame.mixer.music.play()で曲を流すことはありませんよってpygame.mixer.music.play()を二回呼び出すことで思った通りの動作をしてくれます

pusaitou commented 4 years ago

issueに解説を書いていいのかわかりませんが、ソースコードに書くとめちゃ文章ながくなっちゃったのでこちらに書かせていただきます大昔にリーダブルコードにコメントは書きすぎないほうがいい、変数名関数名で表現しろみたいなこと見たことあったかなー(勘違いかも)という感じなのでこちらに書きました

kobi32768 commented 4 years ago

実はある程度のコメントは残したほうがいいとも書かれてます

pusaitou commented 4 years ago

そうなんですね！それじゃあこれについても書いておこうと思います

# 二回呼び出すことには意味があって、正常動作をしてくれます。詳しくはリポジトリーのissueでみれます

くらいは書いておきます

pusaitou / mikochiku_alarm

alarm_sound(self) 関数の中のpygame.mixer.music.play(loop_count)が二個ある理由 #42