連続投票の制限
投票cgiを設置したいのですがうまくいきません、集計を1日一回だったのを少し改造して即時集計にしました
一応動くのですが、連続投票できてしまいます。
連続投票を一定時間制限できるようにしたいのですがうまくいきません
60秒くらい間をおいてから投票できるようにできますか？
ip制限もあるみたいなのですがこれも機能してないです・・・
改善できるなら元のソースがかなり変わってもかまいません
連続投票制限だけでも機能できればと思っています。よろしくお願いします
local $times = time();
sub vote{
if($FORM{id}){
my $fl=0;
if(!$CK_ref){$fl=1;}
else{ if(index($ENV{'HTTP_REFERER'},index.html>=0){$fl=1;} }
if($fl){
$FORM{id}=~s/\n//g;
my $vote = $FORM{vt2}?1:0;
open(OUT,">>vote_temp.cgi"); #時間用のログファイルに書き込み
print OUT "$FORM{id}<>$ENV{'REMOTE_ADDR'}<>$FORM{vt}<>$vote<>".$times."<>\n";
close(OUT);
&reset_vote if $VT_RESET < $times;
my(@log_lines,%pt,%pt2,%cnt,$name,$value,%CKIP);
open(IN,"vote_temp.cgi");
my @log = <IN>;
close(IN);
if(@log){
my $cktime = $times - 60; #60秒制限
open(IN,"vote_ck_IP.cgi"); #投票した人のIPを記録したログ
while(<IN>){
my @ck = split('<>');
next if $_[1] < $cktime;
$CKIP{$_[0]} = $_[1];
}
close(IN);
foreach(@log){
chop;
my @rank = split('<>'); #ID<>IP<>評価<>おすすめ<>時間\n
next if $CKIP{"$rank[0]_$rank[1]"};
$pt{$rank[0].'_'.$rank[2]}++; $pt2{$rank[0]}++ if $rank[3]; $cnt{$rank[0]}++;
$CKIP{"$rank[0]_$rank[1]"} = $rank[4];
}
open(OUT,">vote_temp.cgi");
close(OUT);
open(OUT,">vote_ck_IP.cgi");
while(($name, $value) = each(%CKIP)){
print OUT "$name<>$value<>\n";
}
close(OUT);
open(IN,"log.cgi");
my @data = <IN>;
close(IN);
foreach(@data){
#集計処理長かったので省略
}
open(OUT,">vote_bf.cgi");
open(IN,"vote_log.cgi");
while(<IN>){ print OUT $_; }
close(IN);
close(OUT);
open(OUT,">vote_log.cgi");
print OUT @log_lines;
print OUT "\n1;\n";
close(OUT);
}
&make_vote_ck('set');
}
}
sub reset_vote{
my @log;
foreach my $i(1..$LAST_ID){
next if !@{$VT[$i]};
$VT[$i][18] = $VT[$i][7];
$VT[$i][19] = $VT[$i][8];
$VT[$i][20] = $VT[$i][0];
foreach my $j(0..8){ $VT[$i][$j] = 0; }
push(@log,'$VT['.$i.'] = ['.(join(',',@{$VT[$i]}))."];\n");
}
open(OUT,">vote_log.cgi");
print OUT @log;
print OUT "\n1;\n";
close(OUT);
&make_vote_ck('reset');
}
sub make_vote_ck{
if($_[0] eq 'set'){
my @t =localtime($times + 86400);
$VT_TIME = timelocal(0,0,5,$t[3],$t[4],$t[5]);
$VT_RANK = $VT_RUI = $VT_RECO = $VT_RCRUI = $VT_COUNT = $VT_CTRUI = $VRK_CK = 1;
}
if($_[0] eq 'reset'){
my @m =localtime($times);
$m[4] += 1;
if($m[4] > 12){ $m[4] = 1; $m[5] += 1; }
$VT_RESET = timelocal(0,0,2,1,$m[4],$m[5]);
$VT_RANK = $VT_RECO = $VRK_RS = $VRK_CK = 1;
}
if($_[0] eq 'restore'){ $VT_RANK = $VT_RUI = $VT_RCRUI = $VT_RECO = $VT_COUNT = $VT_CTRUI = $VRK_CK = 1; }
open(OUT,">vote_ck.cgi");
print OUT "\$VT_TIME = '".$VT_TIME. "';\n";
print OUT "\$VT_RESET = '".$VT_RESET. "';\n";
print OUT "\$VT_RANK = '".$VT_RANK. "';\n";
print OUT "\$VT_RUI = '".$VT_RUI. "';\n";
print OUT "\$VT_RECO = '".$VT_RECO. "';\n";
print OUT "\$VT_RCRUI = '".$VT_RCRUI. "';\n";
print OUT "\$VT_COUNT = '".$VT_COUNT. "';\n";
print OUT "\$VT_CTRUI = '".$VT_CTRUI. "';\n";
print OUT "\$VRK_CK = '".$VRK_CK. "';\n";
print OUT "\$VRK_RS = '".$VRK_RS. "';\n";
print OUT "\$LAST_ID = '".$LAST_ID. "';\n";
print OUT "\n1;\n";
close(OUT);
}
お礼
有難うございます。 言われてみれば、確かにそうですね。簡単なコードを書くだけでもいつもミスだらけなのですが、今回は原因を特定することができずに困り果てていたため大変助かりました。