I'm not sure the name of this query.

Shygirl

http://i157.photobucket.com/albums/t80/kissykissy2007/1.jpg

astin is my account. under astin, I have the ID "12", and the wing im in "2"

This is the wing database. My wing is "lalala" Notice it is wing #"2", and the commander is ID "12"

notice under CC1, the value "21"

This is the actual website. Notice in the top left sector there is 21 command centers. That is CC1.

The form on the bottom is saposed to insert buildings. Like when i type 5 into the text field next to command centers, and press constuct, it should say "26" in the database.

Here is my code:

<?php

error_reporting(E_ALL);

$username="151296";
$password="puppypuppy";
$database="151296";
mysql_connect($localhost,$username,$password);
@mysql_select_db($database) or die( "Unable to select database");
$last = $_COOKIE['ID_my_site'];

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'";
$query2 = "SELECT Wing From users where users.Wing = 0";
$result = mysql_query($query) or die(mysql_error());
$result2 = mysql_query($query2) or die(mysql_error());


$sql = sprintf("INSET INTO wings (CC1,GB1,Radar1,Runway1,Factory1,NP1,Residence1,PP1,Oil1) VALUES ('".$_POST['amount1']."','".$_POST['amount2']."','".$_POST['amount3']."','".$_POST['amount4']."','".$_POST['amount5']."','".$_POST['amount6']."','".$_POST['amount7']."','".$_POST['amount8']."','".$_POST['amount9']."')");





$result = mysql_query($sql)or die (mysql_error());

header("Location: arming.php");

?>


Is this INSERT? UPDATE? IMPUT? I don't know what to use. :confused:

Shygirl

THIS SUCKS HOW TO I POST PICS??!

laserlight

Link to the images that you wish to show, like you did on the first line.

Shygirl


<?php

error_reporting(E_ALL);

$username="151296";
$password="puppypuppy";
$database="151296";
mysql_connect($localhost,$username,$password);
@mysql_select_db($database) or die( "Unable to select database");
$last = $_COOKIE['ID_my_site'];

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'";
$query2 = "SELECT Wing From users where users.Wing = 0";
$result = mysql_query($query) or die(mysql_error());
$result2 = mysql_query($query2) or die(mysql_error());







<?php

$add1=$_POST('amount1');
$add2=$_POST('amount2');
$add3=$_POST('amount3');
$add4=$_POST('amount4');
$add5=$_POST('amount5');
$add6=$_POST('amount6');
$add7=$_POST('amount7');
$add8=$_POST('amount8');
$add9=$_POST('amount9');

$query="UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET GB1=GB1+$add2 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET Radar1=Radar1+$add3 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET Runway1=Runway1+$add4 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET Factory1=Factory1+$add5 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET NP1=NP1+$add6 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET Residence1=Residence1+$add7 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET PP1=PP1+$add8 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";
$query="UPDATE wings SET Oil1=Oil1+$add9 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";





$result = $database->query($query);

header("Location: arming.php");

?>

White Screen. 😕

bretticus

Two PHP tips.

Bits of php code are always surrounded by php tags. That is, you begin with <?php and you end with ?>. if you start a php tag (<?php) you must end with a closing tag (?>)

Strings inside of double quotes means that all characters must be within the double quotes. Thus, the single quotes in your SQL queries must be within the double quotes before escaping to inject the php variables.

Incorrect:

$query="UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = " '. $last .' ";

Correct:

$query="UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";

Finally, if you read that article I gave you several posts back about printing out PHP errors, you will notice a part that mentions that syntax errors in your PHP script will prevent any error functions you may stick at the top from being parsed. It goes on to say that these same error setting can be accessed in your php.ini file. I suggest, you do this one thing and you will be able to kiss your grey and white screens goodbye! Here's another link more to the point (look at the bottom for error handling.)

Shygirl


<?php 

error_reporting(E_ALL); 

$username="151296"; 
$password="puppypuppy"; 
$database="151296"; 
mysql_connect($localhost,$username,$password); 
@mysql_select_db($database) or die( "Unable to select database"); 
$last = $_COOKIE['ID_my_site']; 

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'"; 
$query2 = "SELECT Wing From users where users.Wing = 0"; 
$result = mysql_query($query) or die(mysql_error()); 
$result2 = mysql_query($query2) or die(mysql_error()); 









$add1=$_POST('amount1'); 
$add2=$_POST('amount2'); 
$add3=$_POST('amount3'); 
$add4=$_POST('amount4'); 
$add5=$_POST('amount5'); 
$add6=$_POST('amount6'); 
$add7=$_POST('amount7'); 
$add8=$_POST('amount8'); 
$add9=$_POST('amount9'); 





$query="UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET GB1=GB1+$add2 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET Radar1=Radar1+$add3 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Runway1=Runway1+$add4 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET Factory1=Factory1+$add5 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET NP1=NP1+$add6 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET Residence1=Residence1+$add7 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";


$query="UPDATE wings SET PP1=PP1+$add8 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Oil1=Oil1+$add9 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'";





$result = $database->query($query); 

header("Location: arming.php"); 

?>

I did everything you said. I still get just a white screen.......

djjjozsi

change this:
@mysql_select_db($database) or die( "Unable to select database");

into

mysql_select_db($database) or die( "Unable to select database");

change this:
$result = $database->query($query);

// ( you did not create a $database object... )
into this:

$result=mysql_query($query) or die ("error: ".mysql_error());

change these:

$query2 = "SELECT Wing From users where users.Wing = 0"; //what was your aim with this query?
$result = mysql_query($query) or die(mysql_error());
$result2 = mysql_query($query2) or die(mysql_error());

into:
$result = mysql_query($query) or die(mysql_error());

Shygirl



<?php 

error_reporting(E_ALL); 

$username="151296"; 
$password="puppypuppy"; 
$database="151296"; 
mysql_connect($localhost,$username,$password); 
mysql_select_db($database) or die( "Unable to select database");
$last = $_COOKIE['ID_my_site']; 

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'"; 
$result = mysql_query($query) or die(mysql_error());









$add1=$_POST('amount1'); 
$add2=$_POST('amount2'); 
$add3=$_POST('amount3'); 
$add4=$_POST('amount4'); 
$add5=$_POST('amount5'); 
$add6=$_POST('amount6'); 
$add7=$_POST('amount7'); 
$add8=$_POST('amount8'); 
$add9=$_POST('amount9'); 





$query="UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET GB1=GB1+$add2 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Radar1=Radar1+$add3 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Runway1=Runway1+$add4 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Factory1=Factory1+$add5 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET NP1=NP1+$add6 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Residence1=Residence1+$add7 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET PP1=PP1+$add8 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 


$query="UPDATE wings SET Oil1=Oil1+$add9 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'"; 





$result=mysql_query($query) or die ("error: ".mysql_error()); 

header("Location: arming.php"); 

?>

I did what you said and still white screennnnnnnnnnnnnnnnnn

screams into pillow

laserlight

At the moment, the most important thing is to be able to get error message feedback from the PHP interpreter. This means setting error_reporting to at least E_ALL and setting display_errors to on in php.ini.

Incidentally, try not to reveal your database username/password.

I suggest trying out something simple first:

<?php

error_reporting(E_ALL);

$username = '*****';
$password = '*****';
$database = '*****';
mysql_connect($localhost, $username, $password)
    or die('Could not connect to database server: ' . mysql_error());
mysql_select_db($database)
    or die('Could not select database: ' . mysql_error());

echo 'The script exited successfully.';

?>

I note that the $localhost variable was not defined. It probably should be renamed to $host and then given the value of 'localhost'.

Shygirl

but its just a simple imput codeeeeeeeeeeeeeeeeee why is it so hardddddddddddddddddddddddddddddddddddddddddd

laserlight

Shygirl wrote:
but its just a simple imput codeeeeeeeeeeeeeeeeee why is it so hardddddddddddddddddddddddddddddddddddddddddd

It isn't. Nonetheless, you should have your development environment properly setup for development, i.e., you must be able to see any error messages that are emitted. Then, you must follow proper development practices, one of which is incremental development: write some code, test, write more code, test, etc.

Weedpacket

The more you write before you start testing what you've written, the more you end up having to dig through when you try to fix it.
But you really should pay attention to what you're writing while you're writing it; which includes keeping it tidy and watching your punctuation. As an example of what I mean in that last part:

$add1=$_POST('amount1');

does not mean the same thing as

$add1=$_POST['amount1'];

which would be what you want.
Also, all of your update queries except the last one have no effect. But then, all of them including the last one could be combined to a single query anyway.

laserlight wrote:
I note that the $localhost variable was not defined. It probably should be renamed to $host and then given the value of 'localhost'.

I think it was originally

localhost

but Shygirl changed it to

$localhost

when PHP complained about it not being

"localhost"

. Punctuation, see...

laserlight

Weedpacket wrote:
The more you write before you start testing what you've written, the more you end up having to dig through when you try to fix it.

That reminds me: although I talked about "write some code, test, write more code, test, etc.", another school of thought says that you should write the tests before you code the functionality, hence it would become: "write a test, write the code to make the test pass, write another test, write more code to make the new test pass while not causing existing tests to fail, etc". However, this means more than just running the code to see if there are no parse errors and looks ok; it means having a suite of repeatable tests.

Shygirl

Guys........... I'm 15. I don't know code enough to write a little code, and test it and stuff. Basicly, I loook at guids in google... find a code that looks similar to what I want... copy/paste it to my site... tweak it to what i GUESS it should be, and then I play with it for a week untill I get lucky, and it works.

The next time I need a code similar.. I copy what worked before, and put it into my new code.. tweak it.. rinse and repeat... That is why some code is like.. why is that there... its becasue it was for another page, and I copied and pasted it.

Granted, it sounds like i'm not learning this way, but you should know that EVERYTIME I tweak it to try to make it work better, I am manually tweaking it.

So please dont whine, and tell me to go to college 1st. Becasue I firmly believe I can do this. I just need a good push from you guys from time to time. 🙂

And for those of you skeptics that think this won't work, go to http://planez.orgfree.com

It's coming along ok.

laserlight

Shygirl wrote:
Guys........... I'm 15. I don't know code enough to write a little code, and test it and stuff.

No matter how old and experienced you are, there will always be something you do not know, and when you need to work with a new thing, you should still try to start with something simple.

Shygirl wrote:
The next time I need a code similar.. I copy what worked before, and put it into my new code.. tweak it.. rinse and repeat... That is why some code is like.. why is that there... its becasue it was for another page, and I copied and pasted it.

Ideally though, you should, over time, come up with a personal library of functions and classes that you can reuse instead of copying and pasting code all over the place. For example, if you notice that you are always reusing a particular piece of code, think of how to generalise it enough so that it can be a function or class that you can reuse.

Shygirl

But the problem im having now is that, in my site.. the only time i have actually imputted something into the database, was for signup, and create a wing.

Sign up made a new column inside the "users" table, and create a wing made a new column inside the "wing" table. you can see how those might be similar in code

Now im asking the database to check the cookie of the client to see who the user is.. then im asking it to look under the user's column inside the user database to see the number of their wing. Then im asking the database to look inside the wing database for that number. Then im asking it to modify the value of a row in an already existing column.

I have been trying to make this work for months now. I have litterally given up on this exact part of my site at least 4 times in the last year. notice i made thsi account in septermber. 2008, more then a year ago... it was for this exact problem.

My problem is, that this kinda of query is too specific to google... and its to advanced to ask casual php users.... and when i go on this forum and ask... it seems like no1 else can get it to work either..

so this must be some HARD code or something...........

but yeah.... id like to see some statistics on the souicide rate of PHP programers.... >.>

laserlight

Shygirl wrote:
Now im asking the database to check the cookie of the client to see who the user is.. then im asking it to look under the user's column inside the user database to see the number of their wing. Then im asking the database to look inside the wing database for that number. Then im asking it to modify the value of a row in an already existing column.

So, start by not using a cookie. Instead, start by getting the SQL statement correct. From your description, what I would do is to join the user and wing tables based on a given user id, and perform an update on the corresponding row in the wing table. Once this part works (e.g., you create the necessary test data in the database, then you use the known user id and then check the database to see if the value was changed correctly), then you can think about how to get the user id from a cookie.

Shygirl

I already got the user id from a cookie though!

Somehow google made it so that, the site can read who you are when you login.

I even have it display your name, statistics, and all that!

go to http://planez.orgfree.com

username: energy
password: puppy

you will see what im talking abuot..........

btw have you seen my code... for the most part.. the code is already doing what im telling it to.... i just got to get it to stop white screening........... thats where you experts come in 😃


<?php 

error_reporting(E_ALL); 

$username="151296"; 
$password="puppypuppy"; 
$database="151296"; 
mysql_connect("localhost",$username,$password); 
mysql_select_db($database) or die( "Unable to select database");
$last = $_COOKIE['ID_my_site']; 

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'"; 
$result = mysql_query($query) or die(mysql_error());

$add1=$_POST('amount1'); 
$add2=$_POST('amount2'); 
$add3=$_POST('amount3'); 
$add4=$_POST('amount4'); 
$add5=$_POST('amount5'); 
$add6=$_POST('amount6'); 
$add7=$_POST('amount7'); 
$add8=$_POST('amount8'); 
$add9=$_POST('amount9'); 




mysql_query("UPDATE wings SET CC1=CC1+$add1 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET GB1=GB1+$add2 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET Radar1=Radar1+$add3 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET Runway1=Runway1+$add4 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET Factory1=Factory1+$add5 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET NP1=NP1+$add6 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET Residence1=Residence1+$add7 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET PP1=PP1+$add8 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());

mysql_query("UPDATE wings SET Oil1=Oil1+$add9 WHERE wings.Numero = users.Wing AND users.username = '" . $last ."'") or die ("Error:" . mysql_error());





$result=mysql_query($query) or die ("error: ".mysql_error()); 

header("Location: arming.php"); 

?>

djjjozsi

how about this?

$_POST[ ] and NOT a ( )

lets learn from this code below...


<?php

error_reporting(E_ALL);

$username="151296";
$password="puppypuppy";
$database="151296";
mysql_connect("localhost",$username,$password);
mysql_select_db($database) or die( "Unable to select database");
$last = $_COOKIE['ID_my_site'];

$query = "SELECT * FROM wings, users where wings.Numero = users.Wing AND users.username = '" . $last ."'";
$result = mysql_query($query) or die(mysql_error());
$rows=mysql_fetch_assoc($result);
$wing=$rows["Wing"];

function update($field,$number)
{
global $last;
global $wing;

$add=$_POST['amount'.$number];
mysql_query("UPDATE wings SET $field=$field+$add WHERE wings.Numero='".mysql_real_escape_string($wing)."' AND users.username = '" . mysql_real_escape_string($last) ."'") or die ("Error:" . mysql_error());
}



update("CC1","1");
update("GB1","2");
update("Radar1","3");
update("Runway1","4");
update("Factory1","5");
update("NP1","6");
update("Residence1","7");
update("PP1","8");
update("Oil1","9");

header("Location: arming.php");

?>

laserlight

Shygirl wrote:
I already got the user id from a cookie though!

Yes, and if you are confident that that part of the code is correct because you already tested it then perhaps you can just go ahead. However, I find it easier to test when I can more directly control the input values by setting them in the script itself.

Shygirl wrote:
btw have you seen my code... for the most part.. the code is already doing what im telling it to....

However, there is one obvious big problem: your code does not properly escape incoming variables for use in SQL statements. You should be using [man]mysql_real_escape_string/man on $last.

Two less severe problems would be the failure to check that incoming variables exist with say, [man]isset[/man], before using them and executing so many SQL statements when one will do (you can update multiple fields with one SQL statement).

Shygirl wrote:
i just got to get it to stop white screening........... thats where you experts come in

The problem is that it is impossible to tell what this "white screening" actually means. It is often symptomatic of a case where the display_errors configuration setting in php.ini is set to off, and there is an error of some kind. However, if you are not actually printing anything in arming.php, then perhaps it is perfectly fine 🙂

A quick way to check is to replace:

header("Location: arming.php");

with:

echo 'Hello world!';

Now, when you run that script and get 'Hello world!', it is clear that the problem lies with "arming.php". If you still get a "white screen of death", it means that you have to change your PHP configuration settings to set display_errors to on (and check that error_reporting is at E_ALL while you are at it so you don't have to place calls to error_reporting() all over the place). Then, when you restart your server and run the script again, post the error messages that you get.

EDIT:
djjjozsi actually repeated what Weedpacket stated, and although it is clear to me that they are right, I would not apply the fix just yet. The point is, you have to learn how to view error messages and interpret them. In the long run, you simply cannot rely on others to debug your "white screens of death" for you.