Proszę o ocenę kodu pod względem optymalności oraz bezpieczeństwa. Parser miał być prosty i bezpieczny. Udało się cały kod zmieścić w 66 linijkach także dużo sprawdzania nie będzie ;-)
Tutaj znajduje się cały kod wraz z opisem:
Simple and secure BBCode parser for PHP7
BBCode parser dla PHP7
Wątek przeniesiony 2018-02-13 09:19 z PHP przez Patryk27.
- Rejestracja: dni
- Ostatnio: dni
- Postów: 11
0
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Wrocław
- Postów: 13042
0
-
To nie jest parser, tylko bardziej pewien transformator ;-)
-
patternsorazreplacementspowinny stanowić pary, a nie dwie osobne tablice - aktualnie nie wiadomo co z czym się łączy. -
Po co te multum spacji w phpdocu?
-
Wykorzystuj nową, zwięzłą składnię do tworzenia tablic (
$foo = [];) ponad starą ($foo = array();). -
Gdzie są testy? Co łączy się bezpośrednio z następnym punktem:
-
W jaki sposób sprawdziłeś, że Twój kod jest bezpieczny? (masz to wszak w nazwie)