1

私は次のPHPスクリプトを持っています:

<?php
$vote_type = $_GET['type'];
$book = $_GET['book'];
$id = $_GET['id'];


include 'pagehead.php';

$tracker_table = $book.'VoteTrack';
$username = $_SESSION['username'];

session_start();
if ($_SESSION['username'] == null) {
echo 'You must be logged in to vote';
echo '<br>';
echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
echo 'Return to lesson';
echo '</a>';
die();
}

include 'mysqlserver.php';
$con = mysql_connect($mysql_host, $mysql_username, $mysql_password);
if (!$con){
die ('Failed to connect to the database');
}

mysql_select_db("a6595899_s", $con);


$data_query = "SELECT * FROM $book WHERE id=$id";
$lesson_data = mysql_query($data_query);
$lesson_array = mysql_fetch_assoc($lesson_data);

$vote_cop_query = "SELECT * FROM $tracker_table WHERE user='$username' AND id=$id";
$vote_cop_data = mysql_query($vote_cop_query);
$vote_cop = mysql_fetch_assoc($vote_cop_data);

if (mysql_num_rows($vote_cop_data) != 0 && $vote_type == 'up' && $vote_cop['has'] == 1) {
echo 'You have already upvoted this lesson.';
echo '<br>';
echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
echo 'Return to lesson';
echo '</a>';
die();
} elseif (mysql_num_rows($vote_cop_data) != 0 && $vote_type == 'down' && $vote_cop['has'] == 2) {
echo 'You have already downvoted this lesson.';
echo '<br>';
echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
echo 'Return to lesson';
echo '</a>';
die();
}

$vote_count = $lesson_array['votes'];
if ($vote_type == 'up') {
$vote_count++;
$has_type = 1;
} elseif ($vote_type == 'down') {
$vote_count--;
$has_type = 2;
} else {
die('Vote type not specified.');
}

$new_or = mysql_num_rows($vote_cop_data);

if ($new_or == 0) {
$track_query = "INSERT INTO $tracker_table (user, id, has)
VALUES ('$username', $id, $has_type)";
} else {
$track_query = "UPDATE $tracker_table SET has=$has_type WHERE user='$username' AND id=$id";
}
mysql_query($track_query);


//actually cast vote..
$update_query = "UPDATE $book SET votes=$vote_count WHERE id=$id";
mysql_query($update_query);

echo 'Your vote has been submitted!';
echo '<br>';
echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
echo 'Return to lesson';
echo'</a>';


?>

これは非常に単純な投票/および投票システムです。残念ながら、特定のシナリオでは故障します。良いと思うレッスンを読んでいるとしましょう。投票します。後で、レッスンが実際にひどいことに気づいたので、私はそれを否定します。初めて賛成した後、レッスンは1点ありました。反対票を投じた後、再び0になりました。ロジックは、レッスンに-1ポイントを与えて、レッスンに再度反対票を投じることができるはずだと指示しています。私のコードはこれを許可しません。私のスクリプトは、同じアクションが2回続けて許可されないことを単に示しているからです。これを修正するためにどの数学を使用しますか?

4

1 に答える 1

7

問題は、ユーザー自身の賛成票に反対票を投じた後、ユーザーのアクティビティを更新する場合です。

$track_query = "UPDATE $tracker_table SET has=$has_type WHERE user='$username' AND id=$id";

あなたがすべきことは、レコードを更新するのではなく、テーブルから削除してから、スコアを変更することです。そうすれば、次に行う投票は、行った「最初の」投票になります。

または、「nullify」または「revoke」などと呼ばれる3番目のvote_copタイプを使用して、それに応じて投票警官を変更することもできます。

以下の私の提案を参照してください:

$hasVotedBefore = mysql_num_rows($vote_cop_data) != 0;

if ($hasVotedBefore) {

  switch ($vote_cop_data['has']) {
  case 0:
    $vote_cop_type = 'revoked'; // Not really neccessary to do this, but just here for show.
    break;    
  case 1:
    $vote_cop_type = 'up';
    break;
  case 2:
    $vote_cop_type = 'down';
    break;    
  default:
    break;

  if ($vote_type == $vote_cop_type) { // We're here because we voted before and our new vote is the same as the old one.

    if ($vote_type == 'up') {

      echo 'You have already upvoted this lesson.';
      echo '<br>';
      echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
      echo 'Return to lesson';
      echo '</a>';
      die();

    } elseif ($vote_type == 'down') {

      echo 'You have already downvoted this lesson.';
      echo '<br>';
      echo '<a href="lesson.php?book='.$book.'&id='.$id.'">';
      echo 'Return to lesson';
      echo '</a>';
      die(); 
    }

  } else { // Were here because we've voted before, and our new vote is the opposite of the old vote.

     // Update vote_cop row in the database so the 'has' column is 0 (value of a revoked vote)
     // This way, for future votes, we know the user has voted before, but revoked their vote.
     $track_query = "UPDATE $tracker_table SET has=0 WHERE user='$username' AND id=$id";
     mysql_query($track_query);
  }
} else { // We're here because we never voted before.

  $track_query = "INSERT INTO $tracker_table (user, id, has) VALUES ('$username', $id, $vote_type)";
  mysql_query($track_query);
}

// TODO: actually cast vote..
于 2012-06-17T04:28:29.317 に答える