![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 15 Pomógł: 0 Dołączył: 10.04.2012 Ostrzeżenie: (0%) ![]() ![]() |
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: https://github.com/ToTamir/BBCode-Parser
Powód edycji: [nospor]: nie pozycjonuj sie tu
|
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 559 Pomógł: 6315 Dołączył: 27.12.2004 ![]() |
Ale naprawde taki problem zamiast
return preg_replace($patterns, $replacements, $string); dac return (string)preg_replace($patterns, $replacements, $string); (IMG:style_emoticons/default/questionmark.gif) I juz kod jest zabezpieczony. Tobie sie teraz wydaje ze ten NULL nie wystapi nigdy, moze i slusznie, ale nie jestes w stanie przewidziec co sie stanie u kogos za rok. Ja po "paru" latach programowaniu wiem ze po jakims czasie w skrypcie potrafia sie pojawic nieoczekiwane bledy. Naprawde wiec jest takim duzym narzutem danie rzutowanie na string, tak na wszelki wypadek? No nie jest. Skoro funkcja ma zwracac string to niech zwraca string na 100% zawsze. Co do obrazow to co ja ci poradze... No sprawdzaj czy to obraz czy nie (IMG:style_emoticons/default/smile.gif) Moze przez to bedzie dzialac to dluzej ale na pewno bedzie bezpieczniej. Od biedy mozesz patrzec na rozszerzenie ale w dzisiejszych czasach to juz dawno nie jest wyznacznikiem. |
|
|
![]() ![]() |
![]() |
Aktualny czas: 10.10.2025 - 12:06 |