kwLogmakoto_kwのひとりごと。programingCodeReadingActionScript2のライブラリが少ないなかせっかく見つけたライブラリのソースコードがいまいちなので結局自分で書くことにした。個人じゃなくてどこぞの会社のライブラリっぽいんだけどなー。100人いれば100通りの実装があり、自分の書き方がもっとも正しいとは思わないのだけど、そのまま使うに値しない、なんかいまいちだなーと思っているのか以下の用な点。データ(モデル)が存在しないオブジェクト指向なんだから、ある程度オブジェクトはデータでないといけない。MVCで云うところのController的なクラスも必要なこともありこのあたりは判断が難しい。MusicFileというデータに当たるクラスがあったときにMusicFile.Playで再生を行うのか、MediaPlayerというコントローラに当たるクラスを持ってきてMediaPlayer.Play(MusicFile)とするのかみたいな。今みてるライブラリはどっちでもなく全部がコントローラ的なクラス名がつけられてる。PlayingFileクラスみたいな。わかりにくいんだよ、どこでデータを格納してるのかが。変数名が気に入らないこれは、好みの問題もある。配列の変数にわざわざArrayとかつけるの好きじゃない。自分の場合は単に複数形にする。あと、currentという接頭頭をつける変数もちゃんと取捨選択したほうがいい。実行中はみんなcurrentなんだから。関数の結合度が密モジュールやクラスももちろんのこと関数も祖であることが望ましい。最小限の役割を果たすだけのものがよい。最小限の役割の関数を結合して別の機能を組み立てていくべき。いう祖で簡単な関数をつくる場合、できるだけクラスのメンバ変数を参照させたくない。たとえると下のaのようなコードがいいのだが、このライブラリはbのコードが多いのだよ。クラスメンバの責任がわかりにくい、なんとかして欲しい。Engineer人見知りソフトウェアエンジニアです。ビジュアル系、お笑い、Pixarが好き。勢いで吉本超合金おたけびBOTを作った。オールザッツ漫才が放送されない東京在住。趣味や日常からアウトプットの場としてブログを書いています。MarkdownやAmazonJSなど。