wie unterscheide ich Einzelfirmen, AG, GmbH... und je nach Branchen?
Naja, z.B. by Type. Z.B.protected $type;
Wenn es dann zw. AG und GmbH ect. große Unterschiede geben wird,
dann kannst Du Abstrakte Klassen verwenden (wenn es sich "lohnt").
Firmen sind Klassen, Eigenschaften und Objekte?
Verstehe die Frage nicht.
"Kommt immer drauf an".
Du kannst alles als Objekt behandeln, bis es keinen Sinn mehr macht.
Soll am Bsp. heißen:
Addresse - haben sie alle. Also gibts ein Objekt "Address", das im Object "Organization" genutzt wird.
Firma (company (name)) - hm, macht keinen Sinn, gehört zur Addresse. Ist also eine Eigenschaft der "Address".
Dann gibts mehrere Addressen ... z.B. Billing und Shipping (hier evlt. nicht das Thema) ...
also kann man eine "abstract Address" nutzen.
Sowas fällt einem auch mal später auf, wenn Du schon Code geschrieben hast - dann änderst Du das einfach.
Bsp.:
class Organization {
protected $billingAddress;
protected $shippingAddress;
public function setBillingAddress(BillingAddress $address) {
$this->billingAddress = $address;
return $this;
}
public function setShippingAddress(ShippingAddress $address) {
$this->shippingAddress = $address;
return $this;
}
}
interface AddressInterface {
public function getFirstName();
// ...
public function toArray();
}
abstract AbstractAddress implements AddressInterface {
protected $company; // name
protected $firstname;
protected $middlename;
protected $lastname;
protected $postcode;
protected $city;
protected $region;
protected $street; // array of multiple lines (additional info ect)
// ...
// or set all on a property "protected $data;"
public function __construct(array $data) {
// sets data ...
}
}
class BillingAddress extends AbstractAddress {
const TYPE = 'billing';
}
class ShippingAddress extends AbstractAddress {
const TYPE = 'shipping';
}
$organization = new Organization();
$billingAddressArray = []; // some address data array loaded from db or so ...
$organization->setBillingAddress(new BillingAddress($billingAddressArray)) Es kommt also wirklich immer darauf an.
Abr wie gesagt - oft bemerkt man das beim Schreiben.
Ich nehm einfach mal das Bsp. "Verein" -
erst ist es nur eine Property - ein String, der Vereinsnahme.
Später irgendwann merkst Du, dass Du da eine|mehrere Vereinsaddresse(n) hast|willst, ...
oder einen Vereinstyp ... usw ...
dann wird der Verein eben ein Object.
Config.
Z.B.
// config/email/email-config.php
return [
'email' => [
'tool_name' => [
'driver' => 'smtp',
'host' => '',
'port' => '',
'encryption' => '',
'username' => '',
'password' => '',
'from' => [
'email' => '',
'name' => '',
]
],
],
];Im eigentlichen mailer Script holst Du Dir dann die config und go. =)
Kommt natürlich noch drauf an, wie Du mit configs umgehst. Also hast Du eine App, die alle configs sammelt und merged, oder holst Du Dir die config hier per require_once, ... .
Die Seite kann in einem nicht aktiven Tab noch offen sein und der Benutzer hat sie vergessen:
Hearbeat + Man könnte noch per JS abfragen, ob der Tab aktiv ist (also nicht im Hintergrund, Stichwort "focus").
(Evtl zu viel auf einmal. Wollte es nur erwähnen.)
Für genau diese Variante - nein.
Du kannst aber einfach eine Zeit eintragen.
Z.B. "last_seen" : "2021-11-06 14:33:23"
(column Typ Datetime)
Online wäre der User dann, wenn das letzte "last_seen" nicht älter als {n} Minuten ist.
Kann ja nicht funktionieren, wenn Du uns Code vorenthältst (ein blödes Wort).
Du bist in einem for-loop.
Da solltest Du nicht `header("Location: ...")` nutzen.
Durch das `exit` war dann natürlich Ende des Loops.
Eine Lösung wäre:
$failure = 0;
$success = 0;
for($i=0;$i<$countfiles;$i++) {
$dateiName = $_FILES["file"]["name"][$i];
$dateiTempName = $_FILES["file"]["tmp_name"][$i];
$safe_path = "../../../../media/Files/" . htmlspecialchars(ucfirst($userData["verzeichnis"])) . "$datei/";
$absolutePath = $safe_path . $dateiName;
if(file_exists($absolutePath)) {
// Später noch hinzufügen
}
if (move_uploaded_file($dateiTempName, $absolutePath)) {
$success++;
} else {
$failure++;
}
}
if ($success) {
$_SESSION['flash_messages']['Erfolgreich'][] = "{$success} Dateien wurden erfolgreich hochgeladen!";
}
if ($failure) {
$_SESSION['flash_messages']['Erfolgreich'][] = "{$failure} Dateien konnten nicht hochgeladen werden!";
}
header("Location: verzeichnis.php?folder=");
exit(0);Ok, versteh ich.
Passt trotzdem noch.
Egal wo - wenn Du eine Message ausgeben willst, dann pack das in die session.
Und die nächste Seite, die dann geladen wird, kann die Messages ausgeben (mit dem Session code).
Ne, Du setzt $_SESSION['messages']['info'][] = "Wurde hochgeladen!";
in dem upload (upload.php ?) script.
Und in dashborad.php nutzt Du die Ausgabe.
Bsp:
Das Upload - Script(upload . php ?)
// wenn upload erfolgreich -
if (move_uploaded_file($dateiTempName, $absolutePath)) {
// dann session info message setzen -
$_SESSION['flash_messages']['info'][] = "Wurde hochgeladen!";
// umleiten (script dashboard.php wird gestarted)
header("Location: dashboard.php");
// und hier stoppen.
exit(0); // exit(0) ^= kein fehler.
}dashboard . php
if (isset($_SESSION['flash_messages']) and is_array($_SESSION['flash_messages'])) {
// Wir erwarten ein array wie z.B:
// [
// 'flash_messages' => [
// 'info' => ['message 1', 'message 2', /* ... */],
// 'warning' => ['message 1', 'message 2', /* ... */],
// // ...
// ],
// ]
foreach ($_SESSION['flash_messages'] as $type => $messages) {
// $type ist z.B: "error", "info", "notice", "warning", ... .
// $messages z.B: ['message 1', 'message 2', /* ... */]
foreach ($messages as $message) {
// Hier dein code zum ausgeben der message.
// Z.B.:
// flash_msg_info, flash_msg_warning, ...
$cssClass = 'flash_msg_' . htmlspecialchars($type);
// "[info] Die Datei wurde erfolgreich hochgeladen."
$message = '[' . htmlspecialchars($type) . '] ' . htmlspecialchars($message);
$htmlMessage = '<div class="' . $cssClass . '">' . $message . '</div>';
// Ausgeben, oder sammeln, ... .
echo $htmlMessage;
}
}
// Alle Nachrichten (wenn es welche gab) wurden ausgegeben -
// reset session "flash_messages".
$_SESSION['flash_messages'] = [];
}Session.
Wenn erfolgreich, dann setzt Du z.B. $_SESSION['flash_messages'][] = 'Die Datei wurde ... .';.
Und immer, wenn die Webseite ausgegeben wird,
dann guckst Du einfach in $_SESSION['flash_messages'], ob da was drin ist, das angezeigt werden soll.
Wenn Du die Nachrichten angezeigt hast (oder diese evtl. zu alt sind), dann löscht Du sie aus der Session.
Bsp.
if (isset($_SESSION['flash_messages']) and is_array($_SESSION['flash_messages'])) {
foreach ($_SESSION['flash_messages'] as $message) {
// Hier dein code zum ausgeben der message.
}
// All messages were shown - reset|remove them.
$_SESSION['flash_messages'] = [];
}---
Wird oft "Flash messages" genannt.
Bsp: https://stackoverflow.com/a/33224494/3411766
---
EDIT: Du kannst, und solltest wohl auch, alle Nachrichten by type sammeln und ausgeben,
um z.B. Fehler "in Rot" darzustellen ect.
Bsp:
// Add a message.
$_SESSION['flash_messages']['info'][] = 'Die Datei wurde ... .';
// or
$_SESSION['flash_messages']['error'][] = 'Fehler: ... .';
if (isset($_SESSION['messages']) and is_array($_SESSION['messages'])) {
foreach ($_SESSION['messages'] as $type => $messages) {
foreach ($messages as $message) {
// Hier dein code zum ausgeben der message.
// $type ist z.B. "error", "info", "notice", "warning", ... .
}
}
// All messages were shown - reset|remove them.
$_SESSION['messages'] = [];
}mv /media/Files/Test123/TestNeu/Test1/Test2 /media/Files//media/Files/Test123/TestNeu/Test1NeuerOrdner
mv /media/Files/Test123/TestNeu/Test1/Test2 /media/Files/Test123/TestNeu/Test1NeuerOrdnerein mal "/media/Files/" war zu viel.
Ich wüsste ja zu gern, was Du da genau vor hast. Der Benutzer soll PHP-Code eingeben können und der soll dann ausgeführt werden? Von so etwas habe ich noch nie gehört und ich frage mich, ob es keine Alternative ohne solche sicherheitskritischen Aktivitäten gibt.
Stef when das so ist, dann rate ich definitiv zu Template engines.
Denn damit kannst Du loops ect umsetzen, ohne PHP.
Mit str_replace bist Du da sehr limitiert.
Bei meinem Projekt können die User über eine Schnittstelle den HTML- & PHP-Code ihres Formulars auslesen und dann über ihr Script ausführen/nutzen. Im HTML-Code steht noch PHP-Code der entweder die Erfolgs- oder Fehlermeldung ausgibt.
Templates.
Wenn ich das richtig verstanden haben, dann solltest Du dem User templates zur Verfügung stellen.
Bsp: twig
https://github.com/twigphp/Twig
Docu: https://twig.symfony.com/doc/3.x/
---
Deine User bekommen dann das template geladen, das sie verändern können.
Und Du kannst im script die templates laden und parsen und ausgeben lassen.
Und HTML ohne php ist auch besser für die Augen =)
Nur nebenbei: bei includes, bei denen Du nicht weißt (oder evtl. nicht wissen kannst)
ob session_start() schon aufgerufen wurde,
kannst Du prüfen, ob sie denn schon gestartet wurde.
Siehe: https://www.php.net/manual/de/function.session-status.php
Gegenfrage: warum musst Du die alle Löschen?
Werden die nicht mehr gebraucht? (könnte man in den /tmp/ ordner packen)
Temporär erstellt, um die Bilder z.B. an den Browser zu senden? (könnte man per <img> tag mit base64 string machen)
...
Ich frage, weil ich automatisiertes Dateien und Ordner löschen nicht mag.
Muss nicht, aber könnte viel schief gehen =)
Like: oops, wo ist den mein ganzes Projekt hin?
Da fehlt noch die Behandlung des URL-Kontexts (mit http_build_query() oder urlencode()).
Mir gings dabei um die Sicherheit. Hätte ich eigtl. mal hinschreiben müssen :x
Statt preg_replace würde ich nur den filename von dem eingehenden String holen.
Wenn jemand also "../../path/to/my_file.txt" gibt,
dann alles nach dem letzen "/".
Bsp.:
$file = '../../path/to/my_file.txt';
$filename = pathinfo($file,PATHINFO_BASENAME); // my_file.txtBestimmte Zeichen verbieten ist Mist. Dann musst Du ständig hinderher sein, mit updates ... und das will keiner.
Mit dem Filename kann nichts passieren (nichts, was ich kenne. Kann mich irren.), und Du brauchst nichts zu warten.
EDIT: Sicherheitsproblem:
alle vars mit htmlspecialchars() ausgeben. (EDIT: um z.B. XSS zu unterbinden. Alle Daten sind als Böse anzusehen. Auch die aus Deiner db.)
Also
<a href="includes/downloadFile.php?file=<?php echo $ausgabeDatei ?>" type="submit" class="btn btn-info btn-fill pull-right btn-sm">Download</a>zu
<a href="includes/downloadFile.php?file=<?php echo htmlspecialchars($ausgabeDatei) ?>" type="submit" class="btn btn-info btn-fill pull-right btn-sm">Download</a>Der bessere Weg ist es eine Möglichkeit für den Sysadmin zu haben das Passwort zurückzusetzen
- Das System erstellt ein neues Passwort und sendet es direkt dem Nutzer zu, sodass der Admin es nie sieht
- Der Admin kann ein neues Passwort eintragen und muss es dem Nutzer manuell übermitteln
Danach kann der Nutzer sich anmelden und sein Passwort wieder auf sein Wunschpasswort ändern
Find ich nicht so prikelnd.
New pw per email?
New pw per admin?
Nope. New pw by user - yes.
Was tk1234 sagt.
BTW: Komm jetzt nicht auf die Idee, das User pw noch extra irgendwo (verschlüsselt oder gar im Klartext) abzuspeichern. 
Das User pw kennt nur der User, und niemand sonst.
Was prodev sagt (prepared statements)
plus alle daten, die Du in HTML ausgibst maskieren.
Bsp:
<td>'.htmlspecialchars($row->score).'</td>
Auch Daten, die aus Deiner db kommen. Alle Daten sind als nicht sicher anzusehen.
Details auf Anfrage oder im Netz zu fniden.
GET, wenn Du etwas abfragst, ohne zu verändern
POST, wenn etwas verändert wird.
Gibt noch PATCH und PUT ... aber einfach mal im Netz suchen, wenn es interessiert.
Sensible Daten immer POST (login, ...).
Suchanfragen GET (z.B. www.[...].de/movie/{id} order www.[...].de/movie?genre=scifi&name_like=abcdef)
