Холиварка пыхыпешников (общая)

Rakovskiy

26 фев 2015 г., 19:27

The Fast, Secure and Professional - Yii2

# AlkatraZ (26.02.2015 / 18:51)
Ну, чем городить огород, намного проще насильно привести к строке: $var = "$var";
Но я не знаю, откуда у тебя берутся данные и зачем регулярка?

Допустим есть метод addSite(host, domain)

А вдруг тот кто будет им пользоваться передаст в host вместо строки массив, полетит всё к чертям. Я поставил проверку, если host или domain не строка выкидываем исключение. Как бы проблем нет, но красивее бы было в аргументе к методу задать тип String

addSite(String host, String domain)

Но увы, такой тип пока не поддерживается.

AlkatraZ

26 фев 2015 г., 19:35

╭∩╮ (`-`) ╭∩╮

# Rakovskiy (26.02.2015 / 19:27)
А вдруг тот кто будет им пользоваться передаст в host вместо строки массив, полетит всё к чертям.

Ну и правильно.
А для чего существует PhpDoc где описываются все типы передаваемых параметров?

Если кто-то, кто попытается передать массив, настолько крут, чтоб игнорировать описания, надеюсь у него хватит крутизны, чтоб отлавливать ошибки в разрабатываемых скриптах. Передача вместо строки - массива, вызовет Runtime ошибку.

Не заморачивайся с этим, правильно пиши PhpDoc каменты и будет тебе счастье.

Rakovskiy

26 фев 2015 г., 19:44

The Fast, Secure and Professional - Yii2

# AlkatraZ (26.02.2015 / 19:35)
Не заморачивайся с этим, правильно пиши PhpDoc каменты и будет тебе счастье.

PhpDoc пишу, но всё же хотелось бы окончательно застраховаться и выловить все-возможные ошибки...

Koenig

26 фев 2015 г., 20:12

(\/)____o_O____(\/)

# Rakovskiy (26.02.2015 / 19:21)
Ты имеешь ввиду типа этого?
[php]$site = ['site', 'ru'];
$text = 'Hello, welcome to my site_ru or site . ru';
echo preg_replace('#' . $site[0] . '(.?){1,3}' . $site[1] . '#ui', '*****', $text);[/ph

ага, типо того

Koenig

26 фев 2015 г., 20:14

(\/)____o_O____(\/)

Rakovskiy, на напиши уже в теле (string) $host

Rakovskiy

26 фев 2015 г., 20:31

The Fast, Secure and Professional - Yii2

# Koenig (26.02.2015 / 20:12)
ага, типо того

Будет иногда случайные слова резать

Rakovskiy

26 фев 2015 г., 20:34

The Fast, Secure and Professional - Yii2

# Koenig (26.02.2015 / 20:14)
Rakovskiy, на напиши уже в теле (string) $host

Notice: Array to string conversion

Да нафиг чёт мудрить, нет типа string для аргумента, запилю условие

ramzes

26 фев 2015 г., 23:01

# AlkatraZ (26.02.2015 / 19:35)
Ну и правильно.
А для чего существует PhpDoc где описываются все типы передаваемых параметров?

Если кто-то, кто попытается передать массив, настолько крут, чтоб игнорировать описания, надеюсь у не

ты злоумышленникам тоже будешь предлагать доку читать и не пихать бяку куда ее доком не прописали?

AlkatraZ

27 фев 2015 г., 0:00

╭∩╮ (`-`) ╭∩╮

# ramzes (26.02.2015 / 23:01)
ты злоумышленникам тоже будешь предлагать доку читать и не пихать бяку куда ее доком не прописали?

Ну я не видел его скрипт, потому не могу судить о безопасности, да и вообще, что он там делает с той строкой и откуда она вообще берется.
Из формы, или из URL ?

Blade

27 фев 2015 г., 0:00

The Frontend-Warrior

# Gaucer (26.02.2015 / 19:00)
Вмешаюсь не по теме.. зачем нужны тесты вроде понятно, а как выглядит тест?

http://habrahabr.ru/post/130086/