Beiträge von cottton

Zitat

Sie müssen mindestens 6 Zeichen für ihr Password angeben

Das ist ok. Aber das Formular ist wieder leer und ich muss alles noch mal eingeben =)

Das kannst Du ändern indem Du die Daten in die Felder setzt, die schon gesendet wurden und valid sind.

<?
// PHP ...

?>
<!-- html ... -->

<input type="text" value="<?php echo isset($_POST['name']) ? htmlspecialchars($_POST['name']) : '' ?>">

Und das "böse" F5.

Drück mal F5 bei erfolgreicher Registrierung.

Dann wird das gleiche Formular (die gleichen Daten) noch einmal via Browser abgesandt.

EDIT: bei Dir passiert btw alles richtig: die email ist schon vergeben - OK

Das kannst Du mit header('Location: ...') verhindern.

Wenn du per PHP also feststellst, dass alles io ist, dann leitest Du den User per header("Location: http://www.example.com/register_success"); um.

Im Prinzip einfach eine "nichts könnende" Seite, die alleine für die Erfolgsmeldung da ist.

Such mal im Netz nach erneute formular-übermittlung verhindern

Zitat von Jav

Für den PasswortHash können keine Variablen eingegeben werden, weil die $-Zeichen im Hash weitere Variablen auflösen wollen

Entweder verstehe ich Dich falsch, oder Du hast nen Denkfehler

Der hash ist ein String und steckt in der Variable. Wenn da ein $ im String steckt, dann heit das nicht, dass da versucht wird eine neue Variable zu erstellen.

Es sei denn Du würdest zB eval($hash) nutzen, aber wozu?

Der Hash ist immer ein anderer beim password_hash(), da der Salt bei jedem Durchgang neu erstellt wird.

Deshalb kann man zB nicht

$hash === $ddHash prüfen, sondern muss password_verify() nutzen.

(der Salt steck im erzeugten Hash. password_verify kann also den gleichen Salt zum hashen des eingegebenen Passwortes nutzen)

Lust ja, Zeit manchmal nicht so

Register Script sehe ich nichts falsches.

Aber im Login Script:

Könnte hier dran liegen:

$password = filter_var($password, FILTER_SANITIZE_STRING);

Du veränderst hier die Eingabe des Nutzers.

Das pw musst Du genau so nehmen, wie es der Nutzer eingibt.

Oder auch daran:

$fetchRow = $stmt->fetch(PDO::FETCH_OBJ);

fetch zieht eine row - die nächste.

Evtl sind schon mehrere Nutzer mit dem Namen in der db.

Das Script würde hier ggf richtig reagieren, wenn der Nutzername einzigartig sein soll.

Teste mal:

Den Filter auf dem pw weg: auskommentieren oder löschen

und

// ...
$fetchRow = $stmt->fetch(PDO::FETCH_OBJ);
var_dump($stmt->rowCount());

kommt da 1 oder > 1 ?

Wird evtl zu unübersichtlich.

Würde ein steuerscript (Controller) empfehlen (oder wie gehabt das Eingangs Script).

Der prüft halt eben ,ob es überhaupt Eingaben gibt und

Wenn ja, dann erstellt er ein neues Objekt von deiner Klasse.

Dann würde ich ein setData() nutzen, in das Du dann zb die Post Daten rein schickst.

kann leider kein bsp Code geben. Bin auf mobile momentan.

Da drin setzt Du alle Werte und nutzt die jeweiligen Setter.

Zb setName.

setName ist jetzt alleine verantwortlich den Wert auf die Eigenschaft name zu setzen. Der Setter prüft, ob der Wert valid ist, und setzt ihn dann.

Wenn nicht valid, dann Exception.

Wenn alle Setter in setData durch sind, dann kannst du von aussen die Berechnung anstoßen.

Wobei ich hier einen getter empfehlen würde.

Im bsp getCosts().

getCosts würde jetzt die Berechnung durchführen und den Wert returnen.

Statt setData könnte man auch eine Factory nutzen. Eine Klasse, die die Rohdaten bekommt, evtl paar Optionen,

Und dann das Objekt erstellt, alle Werte über die Setter setzt,

und das Objekt returnt.

Ich werde Mal sehen, ob ich Sontag wieder am PC bin. Dann kann ich auch Beispiele liefern ;D

EDIT: Antwort überschnitten Ja so ein Dreher passiert.

Simpler Fehler - zwei mal $this->gemietetVon genutzt.

test:

cottton nachname sie haben vom 2017-06-23 18:09:36 bis zum 2017-06-25 18:09:36 unser Auto, VW Käfer, mit dem Kennzeichen GR-123ABC gemietet. Die Gesamtkosten liegen bei 240€.

class A
{
    const KOSTEN = 5.00;
    protected $gemietetVon;
    protected $gemietetBis;
    protected $vorname;
    protected $nachname;
    protected $auto;
    protected $kennzeichen;
    protected $kosten;

    public function prepare()
    {
        // simulate data ...
        $this->gemietetVon = (new \DateTime('now'))->format('Y-m-d H:i:s');
        $this->gemietetBis
            = (new \DateTime('now + 2 days'))->format('Y-m-d H:i:s');
        $this->vorname = 'cottton';
        $this->nachname = 'nachname';
        $this->auto = 'VW Käfer';
        $this->kennzeichen = 'GR-123ABC';
        return $this;
    }

    public function setRueckgabe()
    {
        if (empty($this->gemietetVon)
            || empty($this->gemietetBis)
            || empty($this->vorname)
            || empty($this->nachname)
            || empty($this->auto)
            || empty($this->kennzeichen)
        ) {
            return false;
        } else {
            $leiheStart = new DateTime($this->gemietetVon);
            $leiheEnde = new DateTime($this->gemietetBis);
            //$tage = $leiheEnde->diff($leiheStart);

            /** @var DateInterval|bool $interval */
            $interval = $leiheStart->diff($leiheEnde);

            $tage = $interval->format('%a');

            // Ein tag hat 24 stunden
            $gesamteDauer = $tage * 24; //Stundenanzahl
            $gesamtPreis = $gesamteDauer * self::KOSTEN;
            $this->kosten = $gesamtPreis;

            return printf(
                "%s %s sie haben vom %s bis zum %s unser Auto, %s, mit dem Kennzeichen %s gemietet. Die Gesamtkosten liegen bei %d€.",
                $this->vorname,
                $this->nachname,
                $this->gemietetVon,
                $this->gemietetBis,
                $this->auto,
                $this->kennzeichen,
                $this->kosten
            );
        }

    }

    public function getRueckgabe()
    {
        return $this->kosten;
    }
}

(new A)->prepare()->setRueckgabe();
echo PHP_EOL;

DateTime http://php.net/manual/de/datetime.diff.php

$datetime1 = new DateTime('2009-10-11 01:00:00');
$datetime2 = new DateTime('2009-10-13 02:00:00');
$interval = $datetime1->diff($datetime2);

// %a für tage
echo "unterschied tage: " . $interval->format('%a') . "\r\n";
// %R für +/-
echo "unterschied tage: " . $interval->format('%R%a') . "\r\n";

// oder
echo $interval->format('Es sind %a Tage und %h Strunden unterschied.') . "\r\n";
//order
echo $interval->format('%Y-%M-%D %H:%I:%S') . "\r\n";

// format siehe http://php.net/manual/de/dateinterval.format.php

Wenn das nicht funktioniert - wie sieht denn das Format aus von den 2 die Du vergleichen willst?

Basi is css so "schlecht" (lahm)? Ich dachte immer anders herum. Hätt ich nicht gedacht.

Ich mag empty nicht so richtig. Ist nicht falsch. Ich vermeide es nur so oft es geht,

da 0, "", false, null,.... use alles empty ist.

Zum preg_match - sorry. Ich hab das + vergessen:

preg_match("/^[a-zA-Z]+$/", $vName)

Pattern:

/                    delimiter (start pattern)
    ^                start zeile
        [a-zA-Z]     groß und kleinbuchstaben
        +            und davon 1 oder mehrere
    $                ende zeile
/                    delimiter (end pattern)

is_numeric prüft nicht, ob eine Zahl im Wort ist.

Du hast vergessen Vorname usw zu setzen. Da kann noch kein Fehler sein.

if (isset($_POST['formularAutoverleihAbgesendet'])) {
    //neue Instanz zu autoverleih erstellen
    $auto1 = new Autoverleih();
    $auto1->setVorname($_POST['vorname']);
    $auto1->setNachname($_POST['nachname']);
    // ...
    // und alle anderen felder hier noch setzen ...

    
    if (count($auto1->error) > 0) {
        echo implode('', $auto1->error);
    } else {
        echo $auto1->getVorname();
    }
}

Für den Vorname-setter würde ich das hier vorschlagen:

public function setVorname(string $vName)
{
    if (strlen($vName) === 0) {
        $this->error[]
            = "<p class='fehlermeldung'>Bitte geben sie ihren Vornamen an!";
        return;
    }
    if (preg_match("/^[a-zA-Z]$/", $vName) !== 1) {
        $this->error[]
            = "<p class='fehlermeldung'>Bitte geben sie Buchstaben an!";
        return;
    }
    $this->vorname = $vName;
}

Ach sowas

Das gibts doch Haufenweise.

Hier zB css only: http://www.daymedia.co.uk/blog/2011/07/m…ounter-in-css3/

oder einfach eins aussuchen - such mal nach "counter animation on scroll down"

Aso.

Also 20 Projekte per Seite. Das aber auf der OnePage per scroll getriggert?

Meine Seite ... [20 Projekte]
.
.
.
Meine Seite ... [40 Projekte]
.
.
.
Meine Seite ... [60 Projekte]

Das sollte sogar per css only funktionieren.

Aber das ist nicht meine Welt. Wird aber sicherlich hier jemand wissen.

Live? Wenn Du oder irgendetwas den Wert ändert, dann soll es live auf der Webseite aktualisiert werden?

Da fallen mir 2 Sachen ein:

Ajax call alle n Sekunden,

oder websocket.

Websocket ist aber evtl overkill.

Daher empfehle ich den ajax call.

Bsp:

index.html

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8">
    <title></title>
    <script src="http://code.jquery.com/jquery-latest.min.js" type="text/javascript"></script>
    <script type="text/javascript">
        function doAjax() {
            $.ajax({
                type: 'GET',
                url: 'path/to/url', /* hier die daten url reingehauen */
                success: function (data) {
                    $("#counter").html(data);
                },
                complete: function (data) {
                    // Schedule the next
                    setTimeout(doAjax, interval);
                }
            });
        }

        var interval = 1000;  // 1000 = 1 second, 3000 = 3 seconds

        $(document).ready(function () {
            setTimeout(doAjax, interval);
        });
    </script>
</head>
<body>

<h1>Test</h1>
<div id="counter">0</div>
</body>
</html>

data.php

<?php
echo rand(0, 11111);

Ajax Bsp von hier genutzt: https://stackoverflow.com/a/20371831/3411766

Ja. Bei kleinen Code Stücken kann man das ja schnell mal über fliegen.

Aber wenn es so viele Zeilen sind, dann ist Syntax highlighting schon angebracht.

Zum Fehler:

"Call to undefined method database::FetchFirst() in" -- welche Zeile?

Nope. So wird keiner weiterhelfen (können).

Ändere mal deinen Beitrag und setze den Code in einen Codeblock.

Im Editor das </> Zeichen klicken und dann dort den Code einfügen.

Das kann nicht sein. Denn in meinem 2ten Bsp Code gibt the keine Nutzung der Varibale

Zeig doch nochmal den code. Ich meine, ich kann ja raten, aber das dauert dann länger :p

Was funktioniert nicht?

Gibts ne Fehlermeldung?

Wie sieht der Code jetzt aus?

Die variable $show_errors ist nicht gesetzt.

Die kommt nicht in die Methode, und Du setzt sie auch nicht.

Woher kommt, oder soll denn der Wert kommen?

Wenn von außen, dann brauchst Du einen zusätzlichen Parameter wie zB

 function database ($host, $name, $user, $pass, $show_errors)

oder besser noch Du setzt den Wert über eine eigene Methode (einen "setter"):

class database
{
    var $connection = '';
    var $db = array();
    var $result = '';
    var $show_errors = null;

    function database($host, $name, $user, $pass)
    {
        $this->db['host'] = $host;
        $this->db['name'] = $name;
        $this->db['user'] = $user;
        $this->db['pass'] = $pass;
    }

    public function setShowErrors($bool)
    {
        $this->show_errors = (bool)$bool;
    }
}

Du prüfst als erstes, ob if (isset($_FILES['bild'])).

Die abfrage, ob if (empty($_FILES['bild'])) liegt innerhalb des isset.

Wenn kein Bild hochgeladen würde, käme das Script an dem ersten isset gar nicht erst vorbei.

Probier mal das hier:

<?php


if (isset($_FILES['bild'])) {
    // ...
} else {
    $error[]
        = '<p class="fehler">Bitte fügen sie ein Bild zum hochladen ein!</p> ';
}
?>
<!DOCTYPE html>
<!-- ... -->

Das else ist neu.

Der $error[] = ... wurde in dieses else verschoben.

Versuch mal Chrome neu zu installieren

Siehe: https://stackoverflow.com/questions/3855…-how-to-stop-it

Ich bin davon ausgegangen, dass Du die Bilder in die db schreiben sollst, für dein Studium