Thank you for taking the time to write to us, but this is not
a bug.

I will check the documentation and update accordingly.

This is the expected behavior.

I would like to propose a different solution.  Since notices 
are 
more for undefined variables, etc. It would be better to 
throw an 
exception that could be caught and dealt wit. Yes,  
geoip_record_by_name() does return false, but there is 
nothing 
you can do about the notice.

<?php
error_reporting(E_ALL);

try {
   $geo = geoip_record_by_name('10.10.10.10');
} catch (Exception $e) {
   $geo = // some default array
}

var_dump($geo);

GeoIP is procedural and not OO style.

You can always run PHP without NOTICES (production recommended) or append a @ in front of the function.

The fact that 10.10.10.10 is not in the DB is actual maybe a problem, as the GeoIP DB contains all "valid" IP. You might be passing an invalid IP, hence the NOTICE to help you debug.

Please see this talk by Rasmus Lerdorf as to why I'm trying to get rid of the notice, 
that GeoIP is throwing. http://talks.php.net/show/phpcon2011/4


I appreciate that you are trying to help people debug, but that fact of the matter is, 
that if GeoIP returns false instead of an array, when you go to try use the returned 
array, your code will fail because it isn't an array.  That is why I feel like the 
notice is unnecessary and harder to deal with, when you have notices turned on in dev.

10.10.10.10 is an internal development IP, so that is why it isn't in the DB. But even 
if you get the latest MaxMind DB there might still be missing IPs.

Putting a @ in front of GeoIP is extremely bad practice so I refused to do that.

Placing @ in front of the function will suppress the 
reporting of "real" errors due to a db missing or the module 
not loaded. This is not the right solution.

We have the same pain with geoip throwing that notice in 
this situation.  It returns false if IP not found so it's a 
clear and consistent process to check for this situation on 
the user side.
I second that it should NOT throw a notice on ip not found.

This is the _only_ function in our whole system that needs 
special treatment because of this notice behaviour.

One more vote for not having this throw a notice. Error suppression is bad for the 
reasons others have stated, and in development mode we have all errors set to 
display, to force developers to fix them. Well, our internal IP addresses are of 
course not found.

3 years have passed and still no solution to this bad practice? I agree with the guys that the function should throw an exception and not a notice.

Null returns at php7.2

Hi,friend.
My problem is: There is a mess code when I use this method to get the city name.
The result I want: Reykjavík,
but it gave me this result: Reykjav�k,
and I try to use the method mb_convert_encoding() or iconv(), but it still doesn't work.I don't think it's a charset problem.
So can you help me see what the problem is, thank you.

Hi,friend.
My problem is: There is a mess code when I use this method to get the city name.
The result I want: Reykjavík,
but it gave me this result: Reykjav�k,
and I try to use the method mb_convert_encoding() or iconv(), but it still doesn't work.I don't think it's a charset problem.
So can you help me see what the problem is, thank you.