Mirasoft Php Framework, Тестирование

[Andkorol]

Сообщение от dem66

А что если например нужно получить string? Например когда в гет параметрах только один елемент /?text ???

Тут всё просто - если функция даже чисто теоретически может вернуть несколько значений - всегда нужно возвращать массив.
А уж сколько в этом массиве будет элементов - неважно.
Нужный вам элемент получить потом из массива несложно - зато никогда не запутаетесь.

В противном случае - через пару месяцев вы сами и помнить не будете, в каком случае у вас возвращается строка, а в каком массив.
Что уж тут говорить о других программистах, которым возможно придется использовать/сопровождать ваш код.

На самом деле даже куча нативных функций в PHP страдает этой фигней - возвращают т.н. mixed.
А ещё в куче функций перепутаны needle и haystack - что заставляет постоянно лезть в мануал за уточнениями.
С нативными функциями уже ничего не поделаешь - ну а при написании пользовательских функций желательно избавлять себя от непоняток.

[dem66]

Сообщение от Andkorol

Тут всё просто - если функция даже чисто теоретически может вернуть несколько значений - всегда нужно возвращать массив.
А уж сколько в этом массиве будет элементов - неважно.
Нужный вам элемент получить потом из массива несложно - зато никогда не запутаетесь.
.

Не подскажите пример?

То есть возвращать вот так:

PHP код:

foreach($output as $k=>$v)  {
                     $k = GetParams::xss_clean($k);   
                        if(empty($v)){   
                             $output[] = $k; 
                             $data = $output;
                        }else{   
                             $v = GetParams::xss_clean($v); 
                             $output[$k] = $v; 
                             $data = $output; 
                        } 
} 


[Andkorol]

Сообщение от dem66

Не подскажите пример?

Пример чего - как из массива получить нужный вам элемент по ключу?

Это - бессмысленный фрагмент:

if(empty($v)){
$output[] = $k;
$data = $output;
}else{
$v = GetParams::xss_clean($v);
$output[$k] = $v;
$data = $output;
}

Вы выполняете это всё в цикле - и фактически при каждой итерации просто переопределяете $data - мне трудно понять зачем...

[dem66]

а ну да. нужно:

PHP код:

$out = array();
foreach($output as $k=>$v)  { 
                     $k = GetParams::xss_clean($k);    
                        if(empty($v)){    
                             $out[] = $k;  
                        }else{    
                             $v = GetParams::xss_clean($v);  
                             $out[$k] = $v;  
                        }  
} 
return $out; 


??? или я не прав

[Andkorol]

Ну типа того...

Только криво как-то вот сама идея насчет "если нет значения - возвращаем просто ключ".
В том смысле, что вы добавляете в массив то ассоциативный элемент(если есть значения), то элемент с числовым ключом(если значения нет).

Если у вас вдруг будет GET вида ?name=Sam&test - ваша функция вернет массив такого вида:

array
'name' => string 'Sam' (length=3)
0 => string 'test' (length=4)

Что с этим делать дальше - малопонятно.

[dem66]

вот вот. и ка это разрулить?

[Andkorol]

Сообщение от dem66

вот вот. и ка это разрулить?

Так это вам виднее - вы же пишете код...

По-хорошему - таких противоречивых ситуаций в коде быть не должно.
Т.е. либо нормальное ЧПУ с корректным роутингом - либо нормальный корректный GET в формате ключ=значение.
Всё, что в GET без значения - в топку.

[dem66]

а если реализовать GET вида /?text/privet

array('text'=>'privet');

[Andkorol]

Сообщение от dem66

а если реализовать GET вида /?text/privet

array('text'=>'privet');

Это всё изврат - и выглядит соответственно...
Сделайте нормальный ЧПУ, если не хотите нормальный GET.

[dem66]

да хватит уже цитировать.
Почему изврат это ведь и есть ЧПУ настоящий. Ну не знаю, везде в интернете такой ЧПУ на форумах пишут, и в разных статьях так рекомендуют.
То есть делать только вида /?text=privet ?