Function I created...

[Anon]

function checkval($a, $b) {
if ($a = "1") {
echo "$b ";
} else {
return false;
}
}

I created this just minute ago...

I want it to check this submit:
Fet
<input type="checkbox" name="fet" value="1">
&nbsp;&nbsp;Tölt
<input type="checkbox" name="tolt" value="1">
&nbsp;&nbsp;Brokk
<input type="checkbox" name="brokk" value="1">
&nbsp;&nbsp;Stökk
<input type="checkbox" name="stokk" value="1">
&nbsp;&nbsp;Skeið
<input type="checkbox" name="skeid" value="1">

When this i submitted the info goes to another document, which performs this:

if ($fet || $tolt || $brokk || $stokk || $skeid) {
checkval($fet, Fet);
checkval($tolt, Tölt);
checkval($brokk, Brokk);
checkval($stokk, Stökk);
checkval($skeid, Skeið);
} else {
echo "<b>ERROR:</b> Veldu gangtegund.";
}

What am I doing wrong? I get all echoed, not just selected onces... Please help.

Thanks in advance.. Arni

[Anon]

A rookie mistake:

if ($a="1")

should be

if ($1=="1")

the single "=" is an operator. It sets the value of $a to "1".
the double "==" is a comparator. It compares $a to "1".

[Anon]

Thanks ! 🙂

[Anon]

How about, if I want to set a , between $b that echoes... How would I do that? 🙂

btw, you don't have to answer that one... I guess it's a few line code...

thanks, arni

[Anon]

echo "$b,";

or
print $b.",";

[Anon]

Then it prints , even on the last one, and that is not what I wanted 🙂 but thanks anyway...

[Anon]

Okie, well then just add

Return TRUE;

right after

echo "$b ";

and change the first 4 checkval lines to:
if (checkval(your_variables_here)) { print ","; };

[Anon]

So it would be...

function checkval($a, $b) {
if ($a == "1") {
echo "$b "; return true;
} else {
return false;
}
}

if ($fet || $tolt || $brokk || $stokk || $skeid) {
//checkval($fet, Fet);
//checkval($tolt, Tölt);
//checkval($brokk, Brokk);
//checkval($stokk, Stökk);
if (checkval($fet, Fet)) { print ","; };
checkval($skeid, Skeið);
} else {
echo "<b>ERROR:</b> Gangtegund hestsins vantar.";
}

but I know this is crap, because I didn't understand you... :\

[Anon]

:-)

what I meant was that youd make it like this:

if (checkval($fet, Fet)) { print ","; };
if (checkval($tolt, Tölt)) { print ","; };
if (checkval($brokk, Brokk)) { print ","; };
if (checkval($stokk, Stökk)) { print ","; };
checkval($skeid, Skeið);

[Anon]

Thanks 🙂

Vincent wrote:

:-)

what I meant was that youd make it like this:

if (checkval($fet, Fet)) { print ","; };
if (checkval($tolt, Tölt)) { print ","; };
if (checkval($brokk, Brokk)) { print ","; };
if (checkval($stokk, Stökk)) { print ","; };
checkval($skeid, Skeið);