Werktage zu einem Datum addieren

SinnlosS

Ok, also von der Laufzeit her ist deine Funktion tatsächlich nen Eck schneller als die strtotime()-Variante.
Du hattest allerdings noch einen kleinen Fehler drin. Bei 6 Werktagen pro Woche darfst du auch nur durch 6 dividieren, nicht durch 7, um auf die Anzahl der Wochen zu kommen.
An meine Bedürfnisse (= 5 Werktage pro Woche) angepasst sieht es dann so aus:

PHP

function getEndDate($start, $tage) {
  $w = date('w', $start);
  $start += 86400 * floor($tage + (2*(($w + $tage) / 5)));
  return date("d.m.Y",$start);
}

Liefert jetzt korrekte Ergebnisse und ist von der Laufzeit her die schnellste Lösung.
Danke.

The User

Also bei mir klappt das nicht. Wenn das so einfach wäre...
Aber mit dem / 6 hattest du Recht. Übrigens hatte ich neben der 7 noch einen Fehler: Ich hatte den Samstag an Stelle des Sonntags genommen.

So, habe es jetzt richtig hinbekommen, musste sehr lange grübeln:

PHP

// Endgültige Implementierung
function getEndDate($start, $tage) {
  $w = date('w', $start);
  return $start += 86400 * ($tage + 2*(floor(($tage + 2* $w)/5)-1));
}
// strtotime
function getStrEndDate($start, $tage) {
  return strtotime('+'.((int)$tage).' weekday',(int)$start);
}
// Anzahl der Wochenenden
function getNumWe($start, $tage)
{
  $w = date('w', $start);
  return floor(($tage + 2* $w)/5)-1;
}
// Samstag wird übersprungen
function getSamDate($start, $tage) {
  $w = date('w', $start);
  return $start += 86400 * floor($tage + ((($w + $tage) / 6)));
}
// Sonntag wird übersprungen
function getSonDate($start, $tage) {
  $w = date('w', $start);
  return $start += 86400 * floor($tage + ((($w + $tage-1) / 6)));
}
// Test-Ausgaben, die zeigen, dass richtig Übersprungen wird
$t = time();
for($i = 0; $i != 21; ++$i)
echo date('w', getEndDate(time(), $i)) .
' ' . date('w', getStrEndDate(time(), $i)) .
' ' . date('w', getSamDate($t, $i)) .
' ' . date('w', getSonDate($t, $i)) .
' ' . date('w', getNumWE($t, $i)) . '<br/>';
$t = time();
$t1 = microtime(true);
for($i = 0; $i != 100; ++$i)
  getEndDate($t, $i);
$t2 = microtime(true);
echo 'Numerische Weg: ' . ($t2 - $t1) . '<br/>';
$t1 = microtime(true);
for($i = 0; $i != 100; ++$i)
  getStrEndDate($t, $i);
$t2 = microtime(true);
echo 'strtotime: ' . ($t2 - $t1) . '<br/>';

Alles anzeigen

Ich habe mal den ganzen Weg hingeschrieben, dann versteht man es auch. Es werden auf jeden Fall zwei Dinge klar:
-Die Funktion getEndDate ist stets drei mal schneller als die Variante mit strtotime (Ich habe es für 1, 10, 100 und für 1000 ausprobiert ;))
-Die Variante mit strtotime erzeugt keine richtige Ausgabe

Viele liebe Grüße
The User

SinnlosS

Zitat von The User

Also bei mir klappt das nicht. Wenn das so einfach wäre...
[...]
-Die Variante mit strtotime erzeugt keine richtige Ausgabe

Viele liebe Grüße
The User

Bei den beiden Punkten verstehe ich noch nicht ganz was du meinst.

Ich hatte mir das Testscript hier mal auf die schnelle zusammengeschustert gehabt, da passen alle Ausgaben und Ergebnisse für alle Funktionen:

PHP

function get_enddate($start, $tage) {
    $wochen   = floor($tage / 5);
    $rest       = $tage % 5;
    $tage    += ($wochen * 2);
    $ende       = $start + ($tage * 86400);
    $ende    += (date("w", $ende)==6 || !date("w",$ende) || (date("w",$ende)-$rest)<1) ? 172800 : 0;
    $enddatum = date("d.m.Y",$ende);
    return $enddatum;  
} 
function addDays($time, $days) {
    for($i = 0; $i < $days; $i++) {    
        $time += 86400;
        while(date("w", $time) < 1 || date("w", $time) == 6) 
            $time += 86400;
    }
    $enddatum = date("d.m.Y",$time);
    return $enddatum;     
}
function get_endday($start, $tage) {
  $time= strtotime('+'.((int)$tage).' weekday',$start);
    $enddatum = date("d.m.Y",$time);
    return $enddatum;  
}
function getEndDate($start, $tage) {
  $w = date('w', $start);
  $start += 86400 * floor($tage + (2*(($w + $tage) / 5)));
  return date("d.m.Y",$start);
}
echo "<form method='post' action=''>Tage: <input type='text' name='tage' value='$days'><input type='submit' value='ok'></form>";


$time = mktime(1,0,0,4,14,2009);
$days = isset($_POST['tage']) ? $_POST['tage'] : 0;


$time_start = microtime(true);
echo "Funktion 1: ".get_enddate($time,$days)."<br>";
$time_end = microtime(true);
$time1 = $time_end - $time_start;
echo "Laufzeit: ".$time1;
$time_start = microtime(true);
echo "<br><br>Funktion 2: ".addDays($time,$days)."<br>";
$time_end = microtime(true);
$time1 = $time_end - $time_start;
echo "Laufzeit: ".$time1;
$time_start = microtime(true);
echo "<br><br>Funktion 3: ".get_endday($time,$days)."<br>";
$time_end = microtime(true);
$time1 = $time_end - $time_start;
echo "Laufzeit: ".$time1;
$time_start = microtime(true);
echo "<br><br>Funktion 4: ".getEndDate($time,$days)."<br>";
$time_end = microtime(true);
$time1 = $time_end - $time_start;
echo "Laufzeit: ".$time1;

Alles anzeigen

War halt nur zum schnellen testen gedacht, nichts schönes.

Ich verwende jetzt deine Funktion, ist halt am schnellsten.

The User

Probiers doch einfach mal aus:
In 4 Werktagen ist der 22. 4. 2009.
Ich vergleiche mit verschiedenen Funktionen:
Nr.1 strtotime:

PHP

function get_endday($start, $tage) {
  $time= strtotime('+'.((int)$tage).' weekday',$start);
    $enddatum = date("d.m.Y",$time);
    return $enddatum;  
}
echo get_endday(time(), 4);

Ausgabe:

Zitat

20.04.2009

=> falsch

Nächste Version, deine zur Zeit bevorzugte:

PHP

function getEndDate($start, $tage) {
  $w = date('w', $start);
  $start += 86400 * floor($tage + (2*(($w + $tage) / 5)));
  return date("d.m.Y",$start);
}
echo getEndDate(time(), 4);

Ausgabe:

Zitat

23.04.2009

=> falsch

Die Version in meinem letzten Post (nur die erste Funktion, der Rest war zum probieren und um alte Sachen klarzustellen):

PHP

function getEndDate($start, $tage) {
  $w = date('w', $start);
  return date('d.m.Y', $start += 86400 * ($tage + 2*(floor(($tage + 2* $w)/5)-1)));
}
echo getEndDate(time(), 4);

Ausgabe:

Zitat

22.04.2009

=> richtig, auch wenn die Funktion etwas komplizierter ist.
Nur das wollte ich sagen.

Viele liebe Grüße
The User

PS:
Sorry, das stimmt auch nicht. Für Morgen klappt es nicht.
Ich muss die Wochenendfunktion nochmal überarbeiten.

SinnlosS

Das ist komisch.
Mit meinem geposteten Script kriege ich bei allen 4 Funktionen immer das richtige Ergebnis, auch bei 4 Tagen.
Da ist das Ergebnis überall der 20.04. was auch korrekt ist, da ich als Startdatum ja fix den 14.04. habe und nicht das heutige Datum.

Tags