php conditional problem

Question

Goal: User only selects one, only one option at a given time for the denomination, either some value of Nickels only, some value of Dimes only, or some value of Quarters only.

Problems: Currently the code only defaults to Nickels only?

Pre-existing condtion: If I select a different denomination, for example, Quarters only, I only get the values for Nickels, the same applies for Dimes?

Code Snippet:

if($denomination["Nickels"] != NULL)

{
$value = $denomination["Nickels"];
echo $value . " is the value of selected Nickels";
}

else if ($denomination["Dimes"] != NULL)
{
$value = $denomination["Dimes"];
echo $value . " is the value of  selected Dimes";
}

else if ($denomination["Quarters"] != NULL)
{
$value = $denomination["Quarters"];
echo  $value . "is the value of selected Quarters";
}

Dear me you like asking questions about nickels and dimes: stackoverflow.com/search?q=user:170622+dime — Dominic Rodger
– Dominic Rodger, Commented Dec 3, 2009 at 13:09
I believe that the three sets of questions; (a) tagged "php", (b) asked by "Newb", and (c) related to a problem with denominations, are rapidly converging. — jensgram
– jensgram, Commented Dec 3, 2009 at 13:19

hubbl · Accepted Answer · 2009-12-03 13:13:57Z

2

To make it short:

<?php
$value = array_shift(array_filter($denomination));
echo  $value . "is the value of selected Quarters";

With this you don't have to put in a if for every new element.

answered Dec 3, 2009 at 13:13

hubbl

3621 gold badge3 silver badges9 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Pekka · Accepted Answer · 2009-12-03 13:07:55Z

2

The "elseif" stops processing when the first condition is true (i.e. "Nickels"). Remove the "else"s and leave plain "if"s.

answered Dec 3, 2009 at 13:07

Pekka

451k150 gold badges989 silver badges1.1k bronze badges

Comments

Chris Gutierrez · Accepted Answer · 2009-12-03 13:09:47Z

0

Are you explicitly setting the values in the $denomination array to null?

Have you trying something like...

if($denomination["Nickels"])
{
$value = $denomination["Nickels"];
echo $value . " is the value of selected Nickels";
}

else if ($denomination["Dimes"])
{
$value = $denomination["Dimes"];
echo $value . " is the value of  selected Dimes";
}

else if ($denomination["Quarters"])
{
$value = $denomination["Quarters"];
echo  $value . "is the value of selected Quarters";
}

answered Dec 3, 2009 at 13:09

Chris Gutierrez

4,75522 silver badges18 bronze badges

1 Comment

Newb Over a year ago

No, but I assure you, I will be in the code cave with all this good stuff experimenting, until I get this baby of the ground, so yes I will by trying your code Chris Gutierrez, and thank you.

Kshitij Parajuli · Accepted Answer · 2009-12-03 15:58:43Z

0

Try not using the elseifs, replace them with ifs

if($denomination["Nickels"] != NULL)

{
$value = $denomination["Nickels"];
echo $value . " is the value of selected Nickels";
}

if ($denomination["Dimes"] != NULL)
{
$value = $denomination["Dimes"];
echo $value . " is the value of  selected Dimes";
}

if ($denomination["Quarters"] != NULL)
{
$value = $denomination["Quarters"];
echo  $value . "is the value of selected Quarters";
}

answered Dec 3, 2009 at 15:58

Kshitij Parajuli

3492 silver badges12 bronze badges

Collectives™ on Stack Overflow

php conditional problem

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related