[SOLVED] help with simplifying

[MDanz]

i got like 874 lines of code..

 

 

is there a way i can simplify these, since there are 4 duplicates with different changes inside if and else statements..

 

e.g.

if{

  if{

    code 1

    }

  else {

    code 2

    }

}

else{

  if{

    code 3

    }

    else {

    code 4

    }

 

}

 

 

 

do i use class or include?.. it will be way easier since i'm lost in brackets..

[Bricktop]

Hi MDanz,

 

It's hard to say without seeing more of your code, there are numerous ways of streamlining code, some better than others depending on the source code in question.

 

If you post more code we'll be able to give a more definitive answer.

[MDanz]

this is what i attempted but not working

 

<?php
$radiobutton = $_GET['radiobutton'];
if($radiobutton == 'on')
{
include("searchtopic.php");
}
else
{
include("searchnormal.php");
}

?>

 

i want if radiobutton is checked on the submit form then use searchtopic code otherwise use searchnormal code..

 

 

this was just an attempt i saved the code in different files.. if another way is easier i'm welcome to do that..

 

 

 

[RichardRotterdam]

What does your form look like?

[MDanz]

What does your form look like?

 

 <form action="test.php" method="get">
<center>
   <input name="search" type="text" value='' size="25" />
   <input type="submit" name="submit" value="search">

   <input name="radiobutton" type="radio" value="radiobutton">
</center>

</form>

[RichardRotterdam]

For your php code to work your radio button should have a value "on"

<input name="radiobutton" type="radio" value="on">
<input name="radiobutton" type="radio" value="off">

[MDanz]

thx alot got it working... is there a way to do it without radiobutton like checkbox.. because i prefer just one item showing not two.

 

e.g.

<input name="checkbox" type="checkbox" value="checkbox">

if($checkbox == 'checked')

 

 

[RichardRotterdam]

That's just a matter of changing type="radio" to type="checkbox" and removing the off radio button.

 

edit Again your value should match the if condition you have your value set to "checkbox" not "on"