Правилно подреждане на функции

[eddyy93]

Как е по правилно:

Вариянт 1:

echo rr(stripslashes(substr_replace($row['title'], '...', 25))) . "<br>\n";

Вариянт 2:

echo substr_replace(rr(stripslashes($row['title'])), '...', 25) . "<br>\n";

 

[relax4o]

Функциите се изреждат отвътре навън, така че ти реши кое би ти свършило работа. И всъщност, ти какво се опитваш да направиш :?



ЕДИТ: Безразборното ползване на функции е все едно да играеш футбол, да не знаеш коя е твоята врата и да си вкараш автогол.

 

[eddyy93]

Ами функцията rr() маха " < " ," >"
stripslashes знаеш.
Другото ми вади текста, направено е така защото кирлица съдържа 2 символа от колкото латиницата 1 символ, за да не може последният символ да става " квадрат ".

 

[relax4o]

Прочети ми ЕДИТ поста.

Ти вадиш от базата данни, какви са тези < > вътре ? Като цяло структурата ти куца нещо.

За кирилицата имаш http://php.net/manual/en/function.mb-substr.php

 

[eddyy93]

Да де mb_substr без да искам съм постнал това. Както и да е.

Иначе тези < > ги махам заради xss.

 

[relax4o]

Да де mb_substr без да искам съм постнал това. Както и да е.

Иначе тези < > ги махам заради xss.

Има още какво да четеш. Щом си допуснал в базата данни да ти влязат тези < > значи е минал XSS. Прави всичко да се спира преди да влезе някъде.

 

[eddyy93]

Ами ще сложа функцията и на входните данни.

 

[relax4o]

http://web-tourist.net/forum/viewtopic.php?p=598422#598422 това е напълно достатъчно против XSS. Няма нужда всяка променлива да я минаваш през определена твоя функция.

function stripslashes_deep($value)
{
        $value = is_array($value) ?
                    array_map("stripslashes_deep", $value) :
                    stripslashes($value);
                    
        return $value;
}
    
$_GET = array_map("stripslashes_deep", $_GET);
$_POST = array_map("stripslashes_deep", $_POST);

Това ползвам, ако магическите кавички са включени, като пак не се занимавам всичко отделно да го минавам през функцията.

Имам си и други особености, но тях не ми се занимава да ти ги дообяснявам.

Седни, почети коя функция какво прави и какво връща, виж си хубаво структурата на сайта и определи къде, какво е нужно да направиш.

 

[eddyy93]

Чесно казано да го напиша на ново. :idea:

 

[relax4o]

Ами според това, което ми даваш, ти пишеш някакви спагети и пълни безсмислици. Какво в строителството, така и в програмирането, трябва да има някаква структура. Не може оттук пляснеш нещо, оттам изтупкаш нещо и хайде, кода готов.
Като взимате от някъде готов код, то се уверявайте, че е добър за вашия сайт. Уверявайте се, че наистина върши работа.
Това, че някой е казал, че еди си кой код го предпазвал, не значи, че наистина го предпазва.

Така мога да си повтарям едни и същи изречения, но с практиката ще свикнете.