Can this short pharse be better?, if(emtpy($foo))

hayato1

I Thinking of a better pharse than this
if "checked" display value with "1"
if NOT checked "IS False" and display "ALL"

working, but i don't think this is good code.

$mvp = $_GET['mvp'];
	if (empty($mvp)) {
		$mvp = "%";
	}

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

Page nagvigate
<a href="<?="$PHP_SELF?mvp=$mvp&.......">PreviousPage</a>

will not project this "%"
index.php?mvp=%&..............

is there a better way than this?

intenz

Can you please explain a little better? Thanks.

laserlight

?mvp=%25
should do the trick, since ASCII 0x25 is '%'
Sort of like manually urlencoding it.

hayato1

hey, thanks for answering

to laserlight:
do you mean?
Edited i tried but still came out .php?mvp=%&...

$mvp = $_GET['mvp'];
if (empty($mvp)) {
$mvp = "%25";
}
<input type="checkbox" name="mvp" value="1">

to intenz:
I'm thinking of a way writing
if NOT checked DISPLAY all RESULT
IF CHECKED display CHECKED Value
so that will get
newreply.php?s=&action=&threadid=
Rather than
newreply.php?s=%&action=%&threadid=%

Thanks again~
*Edited i trying to learn writting in other way to avoide the % appear in address bar

because ever i used this
$key = $_GET['key'];
if (empty($key)) {
$key = "%";
}elseif ($key=="$key") {
$key = "%key%";
}

writting this way
1st time press next page, will appear
.php?key=%&.........
2nd time next page will get
.php?key=%%%&........
3rd next page
.php?key=%%%%%

i avoide this by writing
$key = $_GET['key'];
if(isset($key));

but same method won't be able to use in other _GET
my URL now is like this
mob.php?catergory=mob_name&keywords=&race=%&abc=%&sort=ASC&size=%&mvp=%&m_lv=%&n=8&page=8

laserlight

No, I mean:
newreply.php?s=%25&action=%25&threadid=%25

hayato1

isnt there a way to write something like

$mvp = $_GET['mvp'];
elseif (empty($mvp) {
$mvp = "";
}
this way will get no result if not checked, but i want default result is ALL

i couldnt get this correct below
$mvp = get if from form checked = TRUE
if not don't include it at all, treat as = FALSE = ALL