Confuse taking data form 2 tables in database

PHP
1

I want to create a online toefl test . Now I want to calculate the final score of the user. I want to take the data from the two tables of the database, that is table score_structure and table score_listening .

I would like to take the column score in the table score_structure and table score_listening and then count them. but when i run it, score from table score_Sturcute and table score_listening did not get in. please help mee

<? php

              $email = $_SESSION['email'];

              
              $final="SELECT score_listening.score_list,score_structure.score_struct FROM score_listening, score_structure     WHERE score_listening.email = score_structure.email
                      AND score_listening.email = $email";
                
                $ex=mysql_query($final);
                $data=mysql_fetch_array($ex);
                $ScoreListening=$data['score_list'];
                $ScoreStructure=$data['score_struct'];
                $SkorAkhir = (($ScoreListening+ $ScoreStructure+ $score)/3) * 10; 

              $simpan = "INSERT INTO score_reading VALUES ('$email', '$score')";
              $NA = "INSERT INTO final_score VALUES ('$email', '$ScoreListening', '$ScoreStructure', '$score', '$FinalScore')";
              if(mysql_query($simpan) && mysql_query($NA)){
                   header("location:index.php");
              }else {
                    echo mysql_error();
              }
              }

              ?>
2

When you jumble PHP field names inside of SQL queries you need them in quotes.

score_listening.email = '$email'"

Better is to use prepare statements to keep the data separated from the code.

3

ok, thank you

4

Also what @felgall forgot to mention as well as using prepared statements is that the mysql_* extension that you’ve used (which doesn’t support prepared statements) was removed from version 7 of PHP

5

I have lost count of how many times I’ve mentioned that in the last six months.

1 Like
6

And me - I was just replying to the other post from this user and decided not to turn yet another thread off its original topic.

7

This topic was automatically closed 91 days after the last reply. New replies are no longer allowed.