issues
search
moznion
/
IdolSongVote
For CROSS 2014
4
stars
1
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
最後に、壊滅的にコメントがない
#31
yappo
opened
10 years ago
1
sum_up.pl 動かすと投票不能になる
#30
yappo
opened
10 years ago
2
別の場所で $initial_group って名前で扱われた値が別の所で for my $group (keys %$polled_data) { ってなっててわかりずらい
#29
yappo
opened
10 years ago
0
sum_up.pl の rename $song_file, $song_file_stash; とかで、二回実行すると過去のバックアップ消える
#28
yappo
opened
10 years ago
0
flock の OPERATION 引き数にマジックワード使わない
#27
yappo
opened
10 years ago
0
CGI::Simple の param の戻り値を毎回 decode_utf8 するのだるいから wrapper 作ったら?
#26
yappo
opened
10 years ago
0
open する時に encoding を指定して開けばいちいち print $fh encode_utf8(...) しなくてよい
#25
yappo
opened
10 years ago
0
open とか flock とか seek とかの error みてない
#24
yappo
opened
10 years ago
0
$nc とかぱっと見わからないし、そこの文字数節約するコストより可読性下がる方がデメリットだから $crlf でいい
#23
yappo
opened
10 years ago
0
vote.cgi で redirect するとこで status コードは数字しか入ってないのに無駄に encode_utf8 してる
#22
yappo
opened
10 years ago
0
sum_up.pl の名前がいみわかんない
#21
yappo
opened
10 years ago
0
IdolSongVote::Util の file load 処理が冗長
#20
yappo
opened
10 years ago
0
vote.cgi の投票成功したあとの画面に投票フォームは本来のには無い
#19
yappo
opened
10 years ago
1
vote.cgi の lock 処理について
#18
yappo
opened
10 years ago
1
投票完了後のメッセージを cookie に詰めて redirect するのは資源の無駄
#17
yappo
opened
10 years ago
0
vote.cgi の my $foo = $var ? encode_utf8($var) : '' って分岐させる必要あるの?
#16
yappo
opened
10 years ago
0
vote.cgi は GET と POST で CGI ファイル別にしてよかったのでは
#15
yappo
opened
10 years ago
0
retrieve_initial_group 関数で イニシャル->イニシャルグループ変換しないで、静的な変換ハッシュ作ったほうがいい
#14
yappo
opened
10 years ago
0
keys %{+GOJUON_MAP} って書かれると混乱するから use constant GOJUON_MAP じゃなくて my %GOJUON_MAP って明示的にしたほうがいいのでは
#13
yappo
opened
10 years ago
1
$song->{initial} or next; はあんまりお行儀よくない
#12
yappo
opened
10 years ago
0
init_songs_data.pl の NUM_OF_SONGS は songs.ltvs の中の曲数を可変的に採用したほうがいい
#11
yappo
opened
10 years ago
3
キーワード検索機能いらなくね？
#10
yappo
opened
10 years ago
0
spmgs/$initial_group.ltsv の中にあるイニシャルは冗長
#9
yappo
opened
10 years ago
1
songs/*ltsv で LTSV を使う必要性が 1mm も感じない
#8
yappo
opened
10 years ago
0
scraper がアプリケーションコードと違う言語なのは割りと運用めんどい
#7
yappo
opened
10 years ago
1
全体的に Text::Xslate 使う必要性が感じられない
#6
yappo
opened
10 years ago
0
HTTP Header を print するときに \r\n ですらなくて \n しか使ってないのは微妙
#5
yappo
opened
10 years ago
1
Content-Type ヘッダがおかしい
#4
yappo
opened
10 years ago
0
./cgi-bin/tmpl にテンプレート置いたら設定次第でリソース丸見え
#3
yappo
opened
10 years ago
0
index.cgi は静的ページでいいのに .cgi だしましてや Xslate とか使って HTML 作ってる
#2
yappo
opened
10 years ago
0
突っ込み所がなくてつまらないので CGI & no database server でやり直し
#1
yappo
closed
10 years ago
2