Menu
Community /Pin to ProfileBookmark
@NogDogJul 01.2021 — #
Note, there is no reason to cast a literal
Seehttps://www.php.net/manual/en/language.operators.comparison.php#language.operators.comparison.ternary for info on the
That's a different use case than you implied in the original post. You'd probably have to use an
How To Default Page Number To 1 By Writing As Minimum Amount of Code ?
Folks,
I want to check if the page number exists in url or not. If not then by default the page nu8mber should be “1”.
(Imagine a SERP. They usually contain page number in their urls).
Notice my questions in the comments of my two codes. And try answering them.
Is this code ok or faulty ?
“`
if(!empty($_REQUEST[‘page’]) && is_int($_REQUEST[‘page’]))
{
$page_no = strip_tags(stripslashes($_REQUEST[‘page’])); //Is strip_tags() or stripslashes() necessary here ?
}
else
{
//If Page Number not in Url then default the page number to: 1.
$page_no = INT(1); //Which one is correct ?
$page_no = INTVAL(1); //Which one is correct ?
}
How to shorten the code ?
My attempt to shorten it:
Is this code ok or faulty ?
“`
if(!empty($_REQUEST[‘page’]) && is_int($_REQUEST[‘page’]))
{
$page_no = strip_tags(stripslashes($_REQUEST[‘page’])) : $page_no = 1; //Is strip_tags() or stripslashes() necessary here ?
//Is this part valid or not: “: **$page_no = 1;**”
}
else
{
//If Page Number not in Url then default the page number to: 1.
$page_no = INT(1); //Which one is correct ?
$page_no = INTVAL(1); //Which one is correct ?
}
Show us how you would write it.
Sign in
to post a comment7 Comments(s) ↴
0
@NogDogJul 01.2021 — #[code=php]
$page_no = (!empty($_REQUEST['page']) and intval($_REQUEST['page']) > 0)
? intval($_REQUEST['page']) : 1;
[/code]Note, there is no reason to cast a literal
1intval(1)See
? :0
@NogDogJul 05.2021 — #>@developer_web#1633737 How to ternary whip this one ?
That's a different use case than you implied in the original post. You'd probably have to use an
if/else