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
0
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)