Есть ли более эффективный способ запустить эту функцию?

Question

Есть ли более эффективный способ запустить эту функцию?

Есть ли более удобный способ отформатировать созданную мной функцию, чтобы получить клиентские IP-адреса для чего-то в моем TeamSpeak?

    function getClientIp() {
if (!empty($_SERVER['HTTP_CLIENT_IP']))
return $_SERVER['HTTP_CLIENT_IP'];
else if(!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
return $_SERVER['HTTP_X_FORWARDED_FOR'];
else if(!empty($_SERVER['HTTP_X_FORWARDED']))
return $_SERVER['HTTP_X_FORWARDED'];
else if(!empty($_SERVER['HTTP_FORWARDED_FOR']))
return $_SERVER['HTTP_FORWARDED_FOR'];
else if(!empty($_SERVER['HTTP_FORWARDED']))
return $_SERVER['HTTP_FORWARDED'];
else if(!empty($_SERVER['REMOTE_ADDR']))
return $_SERVER['REMOTE_ADDR'];
else
return false;
}

Немного новичка

1

php teamspeak

Решение

Другие решения

Вы можете использовать? : выражение с более эффективным.

function getClientIp() {
return  !empty($_SERVER['HTTP_CLIENT_IP']) ? $_SERVER['HTTP_CLIENT_IP'] :
!empty($_SERVER['HTTP_X_FORWARDED_FOR']) ? $_SERVER['HTTP_X_FORWARDED_FOR'] :
!empty($_SERVER['HTTP_X_FORWARDED']) ? $_SERVER['HTTP_X_FORWARDED']:
!empty($_SERVER['HTTP_FORWARDED_FOR']) ? $_SERVER['HTTP_FORWARDED_FOR']:
!empty($_SERVER['HTTP_FORWARDED']) ? $_SERVER['HTTP_FORWARDED']:
!empty($_SERVER['REMOTE_ADDR']) ? $_SERVER['REMOTE_ADDR'] : false;
}

0

Источник

Accepted Answer

Это делает то же самое, что и ваш код, но это более кратко, и вы можете добавлять заголовки в массив, а не расширять if / else.

function getClientIp() {
$headers = ['HTTP_CLIENT_IP','HTTP_X_FORWARDED_FOR','HTTP_X_FORWARDED','
HTTP_FORWARDED_FOR','HTTP_FORWARDED','REMOTE_ADDR'];
foreach ($headers as $h) {
if (!empty($_SERVER[$h])) {
return $_SERVER[$h];
}
}
return false;
}

1