Interface Cron

[Raik]

Hallo,
da gibt es so Interface cron datein für Forced banner oder paidmails von webmasterlose ect..
da steht nun eine Zeile gleich unter

// Forcedbanner updaten oder eintragen

und zwar

$int_menge = round($int_menge/$interfacedaten['restklicks']);
if ($ext_verguetung >=$interfacedaten['pmindestverguetung'] and $int_menge > 1)

hat jemand eine Ahnung warum die menge der Banner / die menge restklicks geteilt wird
und wenn das dann noch gröser als 1 ist wird es zugelassen und in die db geschreiben?

das macht für mich derzeit keinen Sinn da das die ware menge an restklicks verfälscht.

mfg Tharkosh
webmaster losesol.de

[VMS1]

Ich weiß ja nicht welches Interface du hast. Aber du hast Recht. So wie das da steht, ist das Blödsinn. Das kommt meist dabei raus, wenn Leute keine Ahnung von dem haben, was sie machen, sondern einfach nur vorhandene Interfaces umbauen und dabei ein wenig herumraten und ausprobieren, bis das Ding endlich auf Teufel komm raus Kampagnen einbucht.

Richtig müßte die Zeile heissen:
$int_menge = round($int_menge/100*$interfacedaten['restklicks']);

Dann werden von der vorhanden Menge soviel % eingebucht, wie man im Adminforce angegeben hat.

[Gremlin]

Yoa das ist eindeutig Schmu, die komplette Zeile kann gelöscht werden... woher hast das Iface?

[Raik]

danke dir für deinen beitrag...

Ja die habe ich hier aus dem forum, handelt sich zum größten teil um diese Paidmail Interface wo dann auch mal die zeile ganz weg ist und die nächste so aus sieht

if ($ext_verguetung >=$interfacedaten['pmindestverguetung'] and $int_menge > $interfacedaten['puebrig'])

ist nur doof das man die erst noch mal korektur lesen muss
naja denke mal ich schreibe mir die anderen selber.


mfg Raik

[Raik]

Original von Gremlin
Yoa das ist eindeutig Schmu, die komplette Zeile kann gelöscht werden... woher hast das Iface?

ähm Addon_Auto_Paidmail da stekt das drin

[Raik]

ähm ich noch mal...
wer ist eigendlich auf die idee gekomemn so komplexe sachen wie das in der For schleife in den Paidmail Interfacen zu schreiben

$kampagne[$u]['id'] = $code[($u-1)*7];
$kampagne[$u]['name'] = $code[(($u-1)*7)+1];
$kampagne[$u]['text'] = $code[(($u-1)*7)+2];
$kampagne[$u]['bid'] = $code[(($u-1)*7)+3];
$kampagne[$u]['verguetung'] = $code[(($u-1)*7)+4];
$kampagne[$u]['uebrig'] = $code[(($u-1)*7)+5];
$kampagne[$u]['mindestaufenthalt'] = $code[(($u-1)*7)+6];

das sieht extrem intelligend aus !?... aber das tut im auge weg ...
das hätte ein einfaches $code[$uu]; $uu++; auch gemacht ...

mfg Raik

[breaker]

Original von Raik
ähm ich noch mal...
wer ist eigendlich auf die idee gekomemn so komplexe sachen wie das in der For schleife in den Paidmail Interfacen zu schreiben

$kampagne[$u]['id'] = $code[($u-1)*7];
$kampagne[$u]['name'] = $code[(($u-1)*7)+1];
$kampagne[$u]['text'] = $code[(($u-1)*7)+2];
$kampagne[$u]['bid'] = $code[(($u-1)*7)+3];
$kampagne[$u]['verguetung'] = $code[(($u-1)*7)+4];
$kampagne[$u]['uebrig'] = $code[(($u-1)*7)+5];
$kampagne[$u]['mindestaufenthalt'] = $code[(($u-1)*7)+6];

das sieht extrem intelligend aus !?... aber das tut im auge weg ...
das hätte ein einfaches $code[$uu]; $uu++; auch gemacht ...

mfg Raik

Wieso komplex ? Es macht durchaus Sinn, die Daten vom Netzwerk zur DB-Losen weiterverarbeitung erstmal in ein Array zu schreiben .

[Raik]

ja sicher sinn macht das aber wer denkt so kompliziert um das zu schreiben ich habe mir das angesehen und mir ist das dazu eingefallen

$kampagne[$u]['bid'] = $code[$uu]; $uu++;
$kampagne[$u]['name'] = $code[$uu]; $uu++;
$kampagne[$u]['uebrig'] = $code[$uu]; $uu++;
$kampagne[$u]['verguetung'] = $code[$uu]; $uu++;
$kampagne[$u]['mindestaufenthalt'] = $code[$uu]; $uu++;
$kampagne[$u]['betreff'] = $code[$uu]; $uu++;
$kampagne[$u]['text'] = $code[$uu]; $uu++;

das sieht doch besser aus
das geht natürlich auch ohne die Daten in einem Array abzulegen

[MK75]

Ja, ist schon komisch wie manche Leute proggramieren, aber das weiß nur jeder selbst, wozu er was, wie macht.

[SebbyPHM]

http://www.totaltoll.de/shared/if.php

Codeausschnitt, wo die Kampagnen abgearbeitet werden:
// Alle Kampagnen abarbeiten
for ($i=0;$i<count($kampagnen);$i++) {
$clickurl = 'http://www.test.de/click.php?kid='.$kampagnen[$i]['kid'].'&bid='.$kampagnen[$i]['bid'];
//$db->query ( 'UPDATE tabelle SET payout = ' .$kampagnen[$i]['payout']. ' hier der rest ... ' );
}

Fazit: Es kann sooooo einfach sein ... Habe jetzt in 10 Mins nen komplettes Interface sauber und leicht anpassbar programmiert :) Das zeigt mal wieder, dass sich saubere Programmierung lohnt ...

[MK75]

Hy Sebby,

das kennen wir doch eigendlich gar nicht anders von Dir. :D

[SebbyPHM]

Original von MK75
Hy Sebby,

das kennen wir doch eigendlich gar nicht anders von Dir. :D
Hehe, danke :) Das nehm ich doch jetzt glatt mal als Lob auf ;)

[MK75]

Original von SebbyPHM
Original von MK75
Hy Sebby,

das kennen wir doch eigendlich gar nicht anders von Dir. :D
Hehe, danke :) Das nehm ich doch jetzt glatt mal als Lob auf ;)

Jupp, gerne doch, so war es nämlich auch durchaus gemeint. :D