Could someone please critque my code?

Originally Posted by pbmods

Heya, Frank.

You'll like the empty() function better, as it combines isset() and == ''.

Incidentally, if you're looking for a better way to do this, the first thing that comes to mind is to make it OO :)

Originally Posted by fjm

Motoma, thanks for the help. I changed the brackets and now I have no more parse error although for some reason I cannot seem to get the code to loop any errors. The code is jumping right to the "insert into"..

I have also verified with print_r and can see all of the post vars but the code isn't seeing the $_POST[$index].

Thanks again!

Frank

Originally Posted by Motoma

Yup, you will need to increment errorCount inside the loop!

Originally Posted by fjm

Motoma,

Thanks for the code and the help. I got the code to work by incrementing the value as you suggested.

I am however getting a HUGE amount of "undefined index" errors on this line [php]if($_POST[$index] == '')[/php]

I thought I could circumvent the error by using isset but when I use that, the error messages stop working. I am assuming because the loop no longer works? Am I correct?

I would like to find out why because I don't get why the error messages stop. I am talking about the validation error messages for the user.

Could you please help me to understand?

Thanks,

Frank

Originally Posted by Motoma

Please remember that POST variables are case sensitive. You will need your values in your field array to appear exactly as they do in your HTML FORM. Also make sure you are using POST rather than GET!

Another thing you will want to do is change your test to check for existence first:
[php]
if(!isset($_POST[$index]) || $_POST[$index] == '')
[/php]

Originally Posted by fjm

Motoma,

Thank you for that. Can you please tell me:

Are these two variables equal? Are they the same?

[php]
$_POST[$index]

$_POST['index']
[/php]

No dollar sign and the use of single quotes. Are these the exact same thing?

Originally Posted by fjm

I couldn't get your code to run and got an T-ECHO error on line 11. I noticed you left off the semicolon. I don't know if that was on purpose. I also saw that you referred to print_r($arr_) (note the trailing underscore. Here is what I changed it to to get it to run:

[php]
echo '<pre>';
print_r($arr);
echo '</pre><hr />';
[/php]

Here was the output:
[php]Array
(
[2] => first element in $arr
[two] => second element in $arr
[three] => third element in $arr
)

second element in $arr
first element in $arr
second element in $arr
third element in $arr
[/php]

Here is what I guessed:

1. ?
2. 2
3. two
4. ?

Motoma, thanks for helping me understand this!

Originally Posted by Motoma

Yeah, my laptop keyboard has been giving out, so some of my code will be wrong. I write it all on the spot, so please bare with me as it is untested as of the moment I press the Submit button.

Originally Posted by Motoma

I hope the exercise helped you understand a little more about arrays. If you still need help, I would suggest checking out php.net as it is an excellent resource for programmers of all skill levels. Also remember, experimentation yields the greatest understanding. print_r and echos will help you though a lot of the learning process.