Как только мы идентифицировали входные и выходные данные, мы тут же оказываемся в ситуации, в которой уже побывали, тестируя binary_search()
do_dependent(), мы могли бы начать с очень большого значения переменной val и отрицательного значения переменной val, чтобы увидеть, что произойдет. Было бы лучше, если бы массив vec оказался вектором, предусматривающим проверку диапазона (иначе мы можем очень просто сгенерировать действительно опасные ошибки). Конечно, мы могли бы поинтересоваться, что сказано об этом в документации, но плохие функции, подобные этой, редко сопровождаются полной и точной спецификацией, поэтому мы просто “сломаем” эту функцию (т.е. найдем ошибки) и начнем задавать вопросы о ее корректности. Часто такое сочетание тестирования и вопросов приводит к переделке функции.26.3.3.2. Управление ресурсами
Рассмотрим бессмысленную функцию.
void do_resources1(int a, int b, const char* s) // плохая функция
// неаккуратное использование ресурсов
{
FILE* f = fopen(s,"r"); // открываем файл (стиль C)
int* p = new int[a]; // выделяем память
if (b<=0) throw Bad_arg(); // может генерировать исключение
int* q = new int[b]; // выделяем еще немного памяти
delete[] p; // освобождаем память,
// на которую ссылается указатель p
}
Для того чтобы протестировать функцию do_resources1()
Перечислим очевидные недостатки.
• Файл s
• Память, выделенная для указателя p
b<=0 или если второй оператор new генерирует исключение.• Память, выделенная для указателя q
0.Кроме того, мы всегда должны рассматривать возможность того, что попытка открыть файл закончится неудачей. Для того чтобы получить этот неутешительный результат, мы намеренно использовали устаревший стиль программирования (функция fopen()
void do_resources2(int a, int b, const char* s) // менее плохой код
{
ifstream is(s); // открываем файл
vector
if (b<=0) throw Bad_arg(); // может генерировать исключение
vector
}
FILE* do_resources3(int a, int* p, const char* s) // плохая функция
// неправильная передача ресурса
{
FILE* f = fopen(s,"r");
delete p;
delete var;
var = new int[27];
return f;
}