How do I stop this from re-running?

Question

0.00/5 (No votes)

See more:

For the code below, the Insert statement I only need to happen ONCE. When the while loop is performed and the items are displayed for user.

is there anyway to do this?

I need duplicate entries in the database so I cant do that. see code below

What I have tried:

$insertedMeal = false;
while($dbRow=$dbQuery->fetch(PDO::FETCH_ASSOC) and ($cnt < $max)) 
{
  echo "//displays information from table"
  $cnt++;
  $MealID=$dbRow["MealID"]
if (!$insertedMeal) {
    $insertedMeal = true;

$stmt=$conn->prepare("INSERT into Meals (UserID, MealID) VALUES (:UserID, :MealID) "); 
$stmt->execute(array(
':UserID' => $UserID,
':MealID' => $MealID
));
 } //end

Posted 1-Apr-18 23:39pm

Member 13637584

Updated 2-Apr-18 4:43am

v3

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Wendelius · Answer 1 · 2018-04-01T23:49:00

Solution 1

Not sure if I understand the question correctly, but with your current code the insert statement will be executed as many times as the loop iterates. So if you want to run the statement only once, move it outside the loop.

In the description you say that you need duplicate values in the database. That leaves some questions concerning the database structure. It could well be that the problem should actually be resolved by changing the database schema. But without further info this is merely guessing.

Posted 1-Apr-18 23:49pm

Wendelius

Comments

Member 13637584 2-Apr-18 5:58am

When i move it outside the loop, it doesn't insert into the database correctly?

Wendelius 2-Apr-18 6:09am

What do you mean by that? Is the insert using wrong values? If that is the case then you need to pick the values you want to use into some variables inside the loop and the use those variables outside the loop.

Member 13637584 2-Apr-18 6:16am

I did try that as i kept $MealID=$dbRow["MealID"] inside the loop then referred to it outside the loop but still doesnt work right?

Wendelius 2-Apr-18 6:27am

If you assign $MealID inside the loop without investigating anything then the $MealID will contain the last MealId when you exit the loop. Is this the problem?

Member 13637584 2-Apr-18 7:11am

Yes thats exactly it?

Wendelius 2-Apr-18 7:16am

Then you need to use some IF comparison inside the loop. I can't tell you what the correct condition is since I don't know the logic you want to use but think about this pseudo-code

while loop
 if some_condition is true then
 set $mealid to desired value
 end if
  advance to next record
end loop
use $mealid in insert statement

Member 13637584 2-Apr-18 7:20am

its basically when the user goes onto the page(for the first time) i want that INSERT to work, and thats it? Would IF statement work?

Wendelius 2-Apr-18 7:28am

That's quite impossible to answer without seeing all the code. But in general if you need to execute something conditionally then yes, IF is one way to do it.

Member 13637584 2-Apr-18 7:39am

is there an if statement that can state when user first goes on a page?

Wendelius 2-Apr-18 7:59am

You can store information about the page visits in a cookie. Have a look at PHP: Cookies - Manual[^]

Member 13637584 2-Apr-18 10:40am

Unfortunately I don't want to do that way for a few reasons. Any way you can help me with the IF statement? I have tried an if statement but it doesnt seem to work - see updated code above

Wendelius 2-Apr-18 11:00am

I'm sorry but I'm having a hard time understanding what you're after.

If you need to track page visits I think it's quite hard to do without utilizing cookies.

Or is the problem that you want to run the insert statement but you don't want the insert to happen if the record already exists?

Please describe the situation in more detail.

Member 13637584 2-Apr-18 11:29am

Basically, I need this to run once, as once it's in the database, i dont need it in again. I dont need to track page visits,when a user gets these items displayeed(first part of while loop) this is why the INSERT statement needs to happen, but finding it hard to stop that particular part of the loop from loop each refresh. I need it to do the INSERT once and stop - it puts values into the database.

Patrice T · Answer 2 · 2018-04-02T00:10:00

Solution 2

This code is just wrong and can't execute because of 2 missing ')'

PHP

$stmt->execute(array(
':UserID' => $UserID,
':MealID' => $MealID

Posted 2-Apr-18 0:10am

Patrice T

Comments

Member 13637584 2-Apr-18 6:09am

Sorry, i do have that in the code. I have updated the above.

Patrice T 2-Apr-18 6:15am

indeed it is better that way.

Member 13637584 4-Apr-18 8:07am

@ppolymorphe do you have any solution to this issue??

Patrice T 4-Apr-18 8:22am

Your code fragment is not complete and we can only guess what is going on.
My best option is to use a debugger to see what is going on in this code and with variables.