Diese Funktion haut einfach nicht hin.

[compuboy1010]

Guten Morgen,

warum haut diese Funktio nicht hin:

## IP FUNCTIONS ##
function assignClientIP()
{
                if (getenv('HTTP_X_FORWARDED_FOR') == '')
    {
         $this->aCustomer->setClientIP((!empty($HTTP_SERVER_VARS['REMOTE_ADDR']) ) ? $HTTP_SERVER_VARS['REMOTE_ADDR'] : ( ( !empty($HTTP_ENV_VARS['REMOTE_ADDR']) ) ? $HTTP_ENV_VARS['REMOTE_ADDR'] : $REMOTE_ADDR ));
    }            
    
    else
    {
        $this->aCustomer->setClientIP("Proxy");
    }
} 

Das Attribut $clientIP bleibt immer leer.

Steckt der Wurm da drin?

Danke

EDIT:

---

php-tags by schmalle

---

[wahsaga]

Re: Diese Funktion haut einfach nicht hin.

dann lass dir doch als allererstes mal zur kontrolle ausgeben, was dein ausdruck

((!empty($HTTP_SERVER_VARS['REMOTE_ADDR']) ) ? $HTTP_SERVER_VARS['REMOTE_ADDR'] : ( ( !empty($HTTP_ENV_VARS['REMOTE_ADDR']) ) ? $HTTP_ENV_VARS['REMOTE_ADDR'] : $REMOTE_ADDR )) 

überhaupt für einen rückgabewert liefert.

[compuboy1010]

function assignClientIP()
{
if (getenv('HTTP_X_FORWARDED_FOR') == '')
{

//0$this->aCustomer->setClientIP
echo ((!empty($HTTP_SERVER_VARS['REMOTE_ADDR']) ) ? $HTTP_SERVER_VARS['REMOTE_ADDR'] : ( ( !empty($HTTP_ENV_VARS['REMOTE_ADDR']) ) ? $HTTP_ENV_VARS['REMOTE_ADDR'] : $REMOTE_ADDR ));
}

else
{
$this->aCustomer->setClientIP("Proxy");
}
}

ergibt jetzt:


INSERT INTO besucher (sid,ip) VALUES (''e24599cfa4cabf5df48b5eaa40b7925b',''')
You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'e24599cfa4cabf5df48b5eaa40b7925b',''')' at line 1
1064
Warning: Failed to write session data (files). Please verify that the current setting of session.save_path is correct (c:\php\tmp_cookies) in Unknown on line 0


weißß aber echt nicht wo der Fehler liegt. Und diese blöde Meldung mit dem Session Cookie Path ist auch neu. habe es in der ini eingetragen. Hat auch immer prima funktioniert.

[Titus]

Re: Diese Funktion haut einfach nicht hin.

Also ich find das hier um einiges übersichtlicher:

function assignClientIP()
{
  if (!($ip = $_SERVER['HTTP_X_FORWARDED_FOR']))
    if (!($ip = $_SERVER['REMOTE_ADDR']))
      $ip = 'Proxy';
  $this->aCustomer->setClientIP($ip);
} 

[compuboy1010]

;-)

Danke!

Sag mal wie macht man das so eine einfache Logik zu finden?
Man muss doch nach irgenwelchen Gesichtspunkten vorgehen können?

Ich habe immer das Problem, dass meine Funktionen (selbst einfache)
immer hochkompliziert werden und ich sie nach einer Weile selber nicht mehr verstehe.

Wie kann man so etwas lernen?


Danke

[schmalle]

Original geschrieben von compuboy1010
Wie kann man so etwas lernen?

die natur liefert hierfür ein sehr gutes tool: das menschliche gehirn. einfach mal benutzen

ansonsten logischen verstand trainieren, in mathe aufpassen, alles tun was das denken auch "um die ecke" trainiert!

[mrhappiness]

Original geschrieben von compuboy1010
Wie kann man so etwas lernen?

üben?

nicht drauflos programieren, sondern sich erst ein konzept überlegen?

[compuboy1010]

Ja, das augearbeitete Konzept/Flussdiagramm muss aber noch umgestzt werden. Da kann der Code aber auf ganz verschiedene Weise programmiert werden.

Welche Technik verwendet Ihr dort?