The error can be avoided by using the isset() function. This function will check whether the index variables are assigned a value or not, before using them.
Fix Notice: Undefined Variable by using isset() Function This notice occurs when you use any variable in your PHP code, which is not set. Solutions: To fix this type of error, you can define the variable as global and use the isset() function to check if the variable is set or not.
Notice Undefined Index in PHP is an error which occurs when we try to access the value or variable which does not even exist in reality. Undefined Index is the usual error that comes up when we try to access the variable which does not persist.
Figure out what keys are in the $output array, and fill the missing ones in with empty strings.
$keys = array_keys($output);
$desired_keys = array('author', 'new_icon', 'admin_link', 'etc.');
foreach($desired_keys as $desired_key){
if(in_array($desired_key, $keys)) continue; // already set
$output[$desired_key] = '';
}
You can use isset() without losing the concatenation:
//snip
$str = 'something'
. ( isset($output['alternate_title']) ? $output['alternate_title'] : '' )
. ( isset($output['access_info']) ? $output['access_info'] : '' )
. //etc.
You could also write a function to return the string if it is set - this probably isn't very efficient:
function getIfSet(& $var) {
if (isset($var)) {
return $var;
}
return null;
}
$str = getIfSet($output['alternate_title']) . getIfSet($output['access_info']) //etc
You won't get a notice because the variable is passed by reference.
A variation on SquareRootOf2's answer, but this should be placed before the first use of the $output variable:
$keys = array('key1', 'key2', 'etc');
$output = array_fill_keys($keys, '');
If you are maintaining old code, you probably cannot aim for "the best possible code ever"... That's one case when, in my opinion, you could lower the error_reporting
level.
These "undefined index" should only be Notices ; so, you could set the error_reporting
level to exclude notices.
One solution is with the error_reporting
function, like this :
// Report all errors except E_NOTICE
error_reporting(E_ALL ^ E_NOTICE);
The good thing with this solution is you can set it to exclude notices only when it's necessary (say, for instance, if there is only one or two files with that kind of code)
One other solution would be to set this in php.ini (might not be such a good idea if you are working on several applications, though, as it could mask useful notices ) ; see error_reporting
in php.ini.
But I insist : this is acceptable only because you are maintaining an old application -- you should not do that when developping new code !
Set each index in the array at the beginning (or before the $output
array is used) would probably be the easiest solution for your case.
Example
$output['admin_link'] = ""
$output['alternate_title'] = ""
$output['access_info'] = ""
$output['description'] = ""
$output['url'] = ""
Also not really relevant for your case but where you said you were new to PHP and this is not really immediately obvious isset()
can take multiple arguments. So in stead of this:
if(isset($var1) && isset($var2) && isset($var3) ...){
// all are set
}
You can do:
if(isset($var1, $var2, $var3)){
// all are set
}
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With