2008 project
Posted by bothajar
| Announcements | Last Post | |
|---|---|---|
 |
SoC Curricula | 09/30/2017 01:08PM |
|
Demarcation or scoping of examinations and assessment | 02/13/2017 07:59AM |
|
School of Computing Short Learning Programmes | 11/24/2014 08:37AM |
|
Unisa contact information | 07/28/2011 01:28PM |
|
May 14, 2009 10:37AM |
|
Re: 2008 project May 15, 2009 10:59AM |
Hi Johan
I have found your problem in amend2. Use the $_GET to pass the variables and then you should come right. Below I have inserted where you need to make the changes. Read up in chapter 2 on page 42. It explains how to use the $_GET and $_POST quite nicely!
Remember that you should also try and practice indentations and standards when writing your code. It makes it easier reading.
Note also that ?PHP and <? is incorrect - you MUST use <?php to be able to access the php code, otherwise it is just recognised as plain HTML text and does nothing.
<?php
session_start();
?>
<?php
$ud_cid=$_POST['ud_cid'];
$ud_cname=$_POST['ud_cname'];
if ($_GET['ud_cid']= ""
echo "! No identifier retrieved";
else
echo "Amending record" . $ud_cid;
Hope this helps!
I have found your problem in amend2. Use the $_GET to pass the variables and then you should come right. Below I have inserted where you need to make the changes. Read up in chapter 2 on page 42. It explains how to use the $_GET and $_POST quite nicely!
Remember that you should also try and practice indentations and standards when writing your code. It makes it easier reading.
Note also that ?PHP and <? is incorrect - you MUST use <?php to be able to access the php code, otherwise it is just recognised as plain HTML text and does nothing.
<?php
session_start();
?>
<?php
$ud_cid=$_POST['ud_cid'];
$ud_cname=$_POST['ud_cname'];
if ($_GET['ud_cid']= ""
echo "! No identifier retrieved";
else
echo "Amending record" . $ud_cid;
Hope this helps!
|
May 15, 2009 01:59PM |
The name=ud_cid part MUST be in.
The Form method is POST, so you must retrieve with $_POST. You use $_GET when you have a link like this <a href='page.php?ud_cid=$ud_cid'>Send ID</a>
The error checking order is - make sure $record exist on 1st page.
Then make sure the db query has retrieved the cid and cname.
Then add them to the form as follows: <INPUT TYPE="text" name="ud_cid" VALUE="<? echo "$record" ?>"> etc.
Send to the next page.
Catch them with &_POST and then echo them e.g. ud_cid on top of the page before you start using it.
Always use error checking like this to narrow the problem down.
$cname=cname, although wrong as pointed out is a leftover I imagine that is not used, and is correctly called lower down so it does not matter - but you should remove it since it is visual clutter. Loop is also 100% correct with his while comment - but you can use while for now - it will not have any effect on execution time. I always say to beginners - get the code working, and then you can get smart
However, you should not be defining you db connection on these pages - it should be in a single file which you include on every page you use it in. If you leave it like this, I will have to go and change your connection details on every pages before I can use it with my db. I don't want to do that. I want to go to one single page and change my connection details there, to be valid throughout your application. It figure that you can also include the mysql_select_db statement there since we are using a single datatbase.
The Form method is POST, so you must retrieve with $_POST. You use $_GET when you have a link like this <a href='page.php?ud_cid=$ud_cid'>Send ID</a>
The error checking order is - make sure $record exist on 1st page.
Then make sure the db query has retrieved the cid and cname.
Then add them to the form as follows: <INPUT TYPE="text" name="ud_cid" VALUE="<? echo "$record" ?>"> etc.
Send to the next page.
Catch them with &_POST and then echo them e.g. ud_cid on top of the page before you start using it.
Always use error checking like this to narrow the problem down.
$cname=cname, although wrong as pointed out is a leftover I imagine that is not used, and is correctly called lower down so it does not matter - but you should remove it since it is visual clutter. Loop is also 100% correct with his while comment - but you can use while for now - it will not have any effect on execution time. I always say to beginners - get the code working, and then you can get smart
However, you should not be defining you db connection on these pages - it should be in a single file which you include on every page you use it in. If you leave it like this, I will have to go and change your connection details on every pages before I can use it with my db. I don't want to do that. I want to go to one single page and change my connection details there, to be valid throughout your application. It figure that you can also include the mysql_select_db statement there since we are using a single datatbase.
|
Re: 2008 project May 15, 2009 10:30PM |
|
May 28, 2009 01:10PM |
|
Re: 2008 project May 28, 2009 02:24PM |
O thank you very much, it is working now. I looked in the book and copied the file from the publisher's site and did not notice the difference. I understand it well now and now it makes a lot of sense. It was short-sighted of me not to spot the difference :-(.. I either need new glasses or should not work late night or juts be more accurate
Thanks
Thanks
|
June 05, 2009 09:19AM |
|
June 09, 2009 04:02PM |
I have marked all (except 3 which I have sent back to fix one or two things related to the config.php page - they include connection details on every page and I am not going to change it on every page - they get -10% already) and will now put final marks in this Thursday - please, by 07h00 it must be in. You know how tedious the process is to get marks added after the cut-off, and I will not go in again before the cut-off date
Sorry, only registered users may post in this forum.