Beiträge von tk1234

  • Quiz mit Javascript erstellt - bitte um Code-Beurteilung

    Zitat von Failix

    Ist es sinnvoll, die Struktur so anzulegen?

    Nein. Nummerierte Spalten sind idR ein deutlicher Hinweis darauf dass das Datenbankdesign defekt ist. So auch hier: die Antworten gehören in eine extra Tabelle in der dann noch die ID der Frage steht (ggf. eine Spalte mit einem Sortier-Wert wenn die Reihenfolge der Antwortmöglichkeiten festgelegt werden soll). Wo du speicherst welche Antwort richtig ist, hängt etwas davon ab ob es (zukünftig) möglich sein soll dass auch mehr als eine Antwort richtig ist: wenn nur eine richtig ist kannst du sie in der Fragen-Tabelle speichern, sonst in der Antworttabelle.

    Und: Datenbankdesign immer als SQL-Querys (CREATE TABLE für das Design und INSERT für die Daten) posten, nie als Bild - hat auch den Vorteil dass man evtl. falsche Spaltentypen gleich mit erkennt.

  • IF Abfrage alle 3 Stunden ausführen

    Zitat von MarvinTLA

    Habe also den Befehl "crontab -e" genutzt auf meinem Raspberry PI und habe ganz unten folgendes hinzugefügt.

    /1 * * * * root /usr/bin/php /var/www/html/includes/verbindung.php


    Auch nach längerer Wartezeit ist nichts passiert.

    Logisch. Da zum einen ist »/1« als Angabe für die Minuten falsch, vor »/1« muss ein Bereich oder ein »*« stehen - wobei »*/1« zwar richtig, aber wenig sinnvoll ist: »*« macht das gleiche (siehe »man crontab.5«). Zudem gehört das »root« da nicht rein - zum einen soll das ganze sicher nicht als root ausgeführt werden und zum anderen gibt es in der Datei die mit »crontab -e« bearbeitet wird (im Gegensatz zu /etc/crontab) keine Spalten für den Benutzernamen.

  • IF Abfrage alle 3 Stunden ausführen

    Zum Cronjob-Problem hat m.scatello ja schon geschrieben: einfach ein Script unabhängig von den schon existierenden Dateien erstellen (den ersten Punkt im Folgenden beachten!)

    Zitat von MarvinTLA

    In den Dateien createFolder.php und createUser.php steht soweit immer das gleiche drin.

    Sehe ich das richtig? Der Teil zum prüfen der Verbindung ist in beiden Dateien identisch (bis auf den Dateinamen halt?). Wenn du Code durch kopieren von einer Datei in eine andere duplizieren musst, ist das ein deutlicher Hinweis darauf dass du den Code in eine Funktion/Klasse auslagern und die Methode/Funktion in beiden Dateien aufrufen möchtest.

    Zitat von MarvinTLA

    ":datum" => date('d.m.y'),

    Das Datenbankdesign scheint mir defekt zu sein: das ist kein Datumsformat mit dem MySQL was anfangen kann. Der Parameter ist aber überflüssig, NOW() in den Query zu schreiben reicht (Voraussetzung ist natürlich ein korrekter Datentyp was bei dir nicht gegeben scheint).

    Zitat von MarvinTLA

    $passwort = $_POST['passwort'];

    Das umkopieren von Werten aus $_POST usw. war schon immer Unsinn, du kannst bei execute auch direkt die Werte aus $_POST verwenden. Speziell bei der Zeile erhebt sich allerdings noch die Frage: wo kommt $password_gehasht weiter unten dann auf einmal her?

    Zitat von MarvinTLA

    $createFolder = ssh2_exec($connection, "mkdir " . escapeshellarg("/media/Files/" . htmlspecialchars(ucfirst($userData["nachname"])) . "/" . $pfad . "/" . $nameNeu));

    Das htmlspecialchars hierdrin ist Käse, einen Kontextwechsel nach HTML gibt es hier nicht. Und wo kommt $userData auf einmal her? Und wo wird sichergestellt dass in $userData['nachname'] kein ../ steht?

  • Skript tut nicht was es soll

    Zitat von newnexo

    Es soll dazu führen, dass ein Video startet, sobald es auf dem Bildschirm beim Scrollen sichtbar wird.

    Nein, soll es nicht. Du hättest das vielleicht gerne, aber dein Benutzer eher nicht - und für die ist die Seite ja schließlich. Ich bin mir jetzt nicht sicher, meine aber mich zu erinnern dass zumindest einige Browser das von dir gewünschte Autoplay unterbinden.

  • Infobox

    Zitat von basti1012

    Inline-style versuchen zu vemeiden

    richtig. Klassennamen nach der damit gesetzten Eigenschaft zu benennen ist auch nicht wirklich besser … Die Klassen sollten immer nach dem Einsatzzweck benannt werden, nie nach der Darstellung - die kann sich auch ändern.

  • Infobox

    Die Codefetzen schau ich mir jetzt nicht genauer an (v.a. bastel ich mir nicht alles so zusammen dass ich es testen kann - das wäre deine Aufgabe das zu ermöglichen ohne x Dateien händisch anlegen zu müssen), eines ist mir aber aufgefallen:

    Zitat von h3llo

    if (strlen($ean) == 13 AND is_numeric($ean) AND !strpos($ean, '.') AND !strpos($ean, 'e')) {

    du suchst die Funktion ctype_digit() (statt is_numeric()), damit kannst du die beiden strpos() rauswerfen.

    Außerdem: gewöhn dir an das ?> wegzulassen wenn anschließend kein HTML kommt - dann riskierst du nicht dass dir irgendwelche Leerzeichen vielleicht mal in die Suppe spucken.

  • Ersetzen eines exakten Ausdrucks mit replace

    Zitat von Failix

    Wie bringe ich der Funktion bei, dass sie ab 13 stoppt?

    Einfach im Ausdruck nur die Zahlen 1-12 erlauben: »\b([1-9]|1[0-2])\b«. Alternativ abfragen ob numberWordMap[nr] existiert:

    JavaScript
    '13'.replace(/(\d+)/g, (_, nr) => numberWordMap[nr] ? numberWordMap[nr] : nr

    (ob das wirklich korrekt so ist muss jemand beurteilen der in JS besser ist als ich, gibt es evtl. einen ??-Operator wie in PHP?)

  • Popup Fenster in HTML5

    Zitat von casi4711

    ich möchte ein Popupfenster erstellen welches ich über einen Desktoplink aurfufe.

    Was hast du genau vor? Was soll das --ssb im Kommando bewirken? Firefox kennt zwar eigentlich den Parameter --headless - der scheint aber aktuell einen Bug zu haben und Firefox startet mit dem Parameter garnicht.

    Mint kennt Webapps was wohl genau das machen würde was du vor hast - ob es für Windows sowas auch gibt weiß ich nicht.

  • Suchfeld im Header

    Zitat von basti1012

    Wenn man diesen Code … so machen würde … der bessere Weg

    Nein. $variable existiert nur wenn das Formular abgeschickt wurde, die Daten sollen aber ja auch bei späteren Aufrufen der Seite noch existieren weswegen die erste Variante schon richtig ist, nur eben ohne htmlspecialchars (bzw. erst im zweiten if-Block bei der Ausgabe).

    Und warum eigentlich $_SESSION['ausgabe']='true'; und nicht $_SESSION['ausgabe']=true;?

  • Suchfeld im Header

    Zitat von basti1012

    Es gebe auch noch eine möglichkeit mit Php.

    Da kommen zwar gleich wieder blöde Kommentare , aber was solls.

    Einfach einmal auf der gleiche Seite umleiten.

    Die Idee ist durch auch ok, gleich GET zu verwenden ist aber imho sinnvoller: die Abfrage verändert keine Daten, damit ist GET die Methode der Wahl - es wird schon seinen Grund haben dass Google (andere aber auch) GET-URLs verwenden.

    Zitat von basti1012

    $_SESSION['ausgabe']=htmlspecialchars($_POST['textfeld']);

    *das* aber ist auch Käse: das Speichern des Wertes in der Session ist kein Kontextwechsel (einer nach HTML schon gleich garnicht) womit htmlspecialchars() hier falsch ist - die Funktion darf erst verwendet werden wenn der Wert tatsächlich im HTML an den Browser geschickt wird.

  • Error: Failed opening required

    Zitat von Dekiblago

    __construct( $dsn = 'mysql:host=127.0.0.1;dbnameshop,charset=UTF8', $username = 'DB_USERNAME', $password = 'DB_PASSWORD' )

    Die Fehlermeldung passt nicht zum Code und der dsn ist immer noch Käse. Einer der drei Fehler darin ist das falsche charset (richtig wäre utf8mb4), den Rest siehst du hoffentlich selbst - wenn nicht solltest du überlegen ob das Programmieren wirklich das richtige für dich ist, simple Syntaxfehler sollte man auch selbst finden, dafür sind Foren nicht da.

  • PSR4 Autoloader findet Klasse nicht

    Zitat von Stef

    Ich habe mein Projekt gerade ohne jegliche Änderungen auf mein Webserver hochgeladen und über den Browser aufgerufen. Nun kommt der gleiche Fehler wie zu Beginn: Die Klasse kann nicht gefunden werden, obwohl der Pfad zur PHP-Datei stimmt.

    Da der Webserver vmtl. unter Linux läuft könnte es an einem Klassiker liegen: Groß- und Kleinschreibung bei Dateinamen und Verzeichnissen nicht konsequent beachtet (Linux unterscheidet hier, Windows nicht) - mehr lässt sich so völlig ohne Kenntnis von Code und Datenstruktur nicht sagen.

    Zum Thema Dateien außerhalb vom document-root ablegen muss ich auch noch was nachtragen: dass große Shop-System alle ihre Dateien innerhalb des Root-Verzeichnisses hätten (-> #9) stimmt nicht: Shopware z.B. hat da mit der Version 6 umgestellt, wenn die Domain nicht auf das public-Verzeichnis zeigt sondern auf die Ebene darüber, bekommt man lediglich eine Fehlermeldung. Das muss natürlich nicht repräsentativ sein, oft wird es aber historische Gründe haben da eine Umstellung der Verzeichnisstruktur immer einen gewissen Aufwand mitbringt.