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
- Rejestracja:około 13 lat
- Ostatnio:prawie 5 lat
- Postów:11
0
PHP edytowany 1x, ostatnio: ToTamir
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)
edytowany 3x, ostatnio: Patryk27
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.