Reload Tabelle -> INET_ATON()

[jpwfour]

Was mir grad noch einfällt was immer nervig ist nachträglich machen zu müssen, wär praktisch, wenn das Grundscript schon gleich die IP Adressen als Zahl in der Datenbank behandeln würde, dementsprechend die topframe_forced.php und die content/verdienen/klick4.php angepasst wäre.

Nur mal so als Anhaltspunkt:

[tableold=head]|Größe in KiloByte|Anzahl Einträge
vorher:|6.316,84|53.750
nachher:|3.386,08|53.458[/tableold]

Man kann da also mächtig einsparen, nicht nur an Platz, sondern auch an Geschwindigkeit gewinnen, da ein Index auf Zahlen einfach schneller durchsucht werden kann.

--

CREATE TABLE IF NOT EXISTS `vms_reloads` (
  `ip` int(11) UNSIGNED NOT NULL default '0',
  `uid` int(7) NOT NULL default '0',
  `tan` varchar(32) NOT NULL default '',
  `bis` int(10) NOT NULL default '0',
  KEY `klick4` (`tan`,`uid`,`ip`)
) ENGINE=MyISAM;

für umsteiger:

ALTER  TABLE  `vms_reloads`  CHANGE  `ip`  `ip` INT( 11  ) UNSIGNED NOT  NULL

##

topframe_forced.php

ON (t1.tan=t2.tan AND (t2.uid=".$_SESSION['uid']." or t2.ip=INET_ATON('".$ip."') ))

...

db_query("INSERT INTO ".$db_prefix."_reloads (ip,uid,tan,bis) VALUES (INET_ATON('".$ip."'),".$_SESSION['uid'].",'".$forced['tan']."',".$new_reload.")"); 

##
content/verdienen/klick4.php

ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = INET_ATON("'.$ip.'") ) AND t2.bis >= '.time().')

Also eh nur minimale Veränderungen mit denk ich doch recht großem Effekt.

Gibt zwar dann natürlich Probleme mit den Addons, die darauf Aufbauen (Paidlink Bereich, Highforced-Bereich usw.) da der "Umbau" dort aber auch mit 2 kleinen Änderungen zu machen ist, kann man ja darauf einfach in der readme.txt verweisen

Btw: lässt sic auch in laufendem Betrieb gut umstellen, indem man
1. die Änderungen in den Dateien macht und erst dann
2. den SQL Query :

UPDATE vms_reloads SET ip = INET_ATON(ip)

ausführt und erst dann
3. den obigen alter table query ausführt

[Gremlin]

Im Prinzip guter Vorschlag hatte ich auch schon dran gedacht, aber es gibt sehr viele Addons welche auch die vms_reloads nutzen, diese wären dann alle erst einmal inkompatibel.

[eRaaaa]

ich hatte das vor einiger zeit auch schon ansprechen wollen, weil ich auf phpperformance im blog (http://phpperformance.de/ip-adressen-optimal-speichern/) das schon gelesen hatte. gerade auch bei dem ländercode addon würd ichs umstellen...

[jpwfour]

Man köntne es so machen, dass man gleich schon in der functions.lib.php in der Variable
$ip
den "long" Wert speichert, somit müsste man in den Addons etc. nichts anpassen.

Außer wenn es dann wiederum Addons gibt, die auf $ip zugreifen und das in eigenen Tabellen verwenden, wobei dann halt in varchar(15) nur die zahl gespeichert wird, was aber ja immerhin funktioniert (wenn auch platzverschwendung ist)

Also in der functions.lib.php dann:

$ip             = sprintf("%u", ip2long($_SERVER['REMOTE_ADDR'])); 

[DJschatz27]

Tach ne frage gibts hierzu noch eine andere Lösung

weil ich habs eben mal getestet aber dann bin ich wegen der ip fürs klicken gesperrt oder was muß man alles ändern?

[Hardy]

Vielleicht INET_ATON() aus der klickdatei entfernen