Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#123 closed ожидается проверка (задача сдана)

Home assignment #4

Reported by: Andrey Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#4 lazy_string Version: 1.0
Keywords: Cc: andreypestov82@…

Description


Change History (23)

comment:1 Changed 7 years ago by cpptester

Type: ожидается проверкаожидаются исправления

comment:2 Changed 7 years ago by cpptester

Tested revision 726 by pestov.andrej.

Smoke tests

test

stage

result

info

log

run_smoke_test.cpp

gcc62

FAILURE

run_smoke_test-gcc62.log

Common header used in some tests: common.h

comment:3 Changed 7 years ago by cpptester

Tested revision 727 by pestov.andrej.

Smoke tests

test

stage

result

info

log

run_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_smoke_test-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_smoke_test-san.log

Common header used in some tests: common.h

comment:4 Changed 7 years ago by cpptester

Tested revision 730 by pestov.andrej.

Main tests

test

stage

result

info

log

ncomp_c_str_const.cpp

gcc62

FAILURE

run_aug_plus.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus-san.log

run_aug_plus_aug_plus.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_aug_plus-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_aug_plus-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_aug_plus-san.log

run_aug_plus_c_str_char_traits_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_c_str_char_traits_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_c_str_char_traits_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_c_str_char_traits_no_mem-san.log

run_aug_plus_c_str_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_c_str_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_c_str_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_c_str_no_mem-san.log

run_aug_plus_c_wstr_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_aug_plus_c_wstr_no_mem-run.log

run_aug_plus_char_char_traits_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_char_char_traits_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_char_char_traits_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_char_char_traits_no_mem-san.log

run_aug_plus_char_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_char_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_char_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_char_no_mem-san.log

run_aug_plus_lazy_str_char_traits_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_lazy_str_char_traits_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_lazy_str_char_traits_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_lazy_str_char_traits_no_mem-san.log

run_aug_plus_lazy_str_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_aug_plus_lazy_str_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_aug_plus_lazy_str_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_aug_plus_lazy_str_no_mem-san.log

Common header used in some tests: common.h

comment:5 Changed 7 years ago by cpptester

Tested revision 744 by pestov.andrej.

Main tests

test

stage

result

info

log

ncomp_c_str_const.cpp

gcc62

FAILURE

run_ctor_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_ctor_no_mem-run.log

run_ctor_repeat.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_ctor_repeat-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_ctor_repeat-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_ctor_repeat-san.log

run_ctor_repeat_char_traits_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_ctor_repeat_char_traits_no_mem-run.log

run_ctor_repeat_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_ctor_repeat_no_mem-run.log

run_empty.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_empty-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_empty-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_empty-san.log

run_empty_eq.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_empty_eq-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_empty_eq-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_empty_eq-san.log

run_empty_wide.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_empty_wide-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_empty_wide-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_empty_wide-san.log

run_index_assign_copy_no_mem.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_index_assign_copy_no_mem-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_index_assign_copy_no_mem-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_index_assign_copy_no_mem-san.log

run_plus.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_plus-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_plus-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_plus-san.log

Common header used in some tests: common.h

comment:6 Changed 7 years ago by cpptester

Tested revision 745 by pestov.andrej.

Main tests

test

stage

result

info

log

run_plus_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_c_str_no_mem-run.log

Common header used in some tests: common.h

comment:7 Changed 7 years ago by cpptester

Tested revision 746 by pestov.andrej.

All tests passed. Good job!

comment:8 Changed 7 years ago by Vladimir Rutsky

Type: ожидаются исправленияожидается проверка

comment:9 Changed 7 years ago by cpptester

Type: ожидается проверкаожидаются исправления

Tested revision 746 by pestov.andrej.

Main tests

test

stage

result

info

log

run_assign_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_assign_c_str_no_mem-run.log

run_char_traits_custom_eq.cpp

gcc62

FAILURE

run_char_traits_custom_eq-gcc62.log

Common header used in some tests: common.h

comment:10 Changed 7 years ago by cpptester

Tested revision 906 by pestov.andrej.

Smoke tests

test

stage

result

info

log

run_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_smoke_test-asan.log

clang-san

SUCCESS

san

FAILURE

exit code 1

run_smoke_test-san.log

Common header used in some tests: common.h

comment:11 Changed 7 years ago by cpptester

Tested revision 907 by pestov.andrej.

Main tests

test

stage

result

info

log

run_index_const_wstr.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_index_const_wstr-run.log

run_index_last_const.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_index_last_const-run.log

Common header used in some tests: common.h

comment:12 Changed 7 years ago by cpptester

Tested revision 908 by pestov.andrej.

All tests passed. Good job!

comment:13 Changed 7 years ago by Andrey

Milestone: ha4-milestone1ha4-milestone2
Type: ожидаются исправленияожидается проверка

comment:14 Changed 7 years ago by Vladimir Rutsky

Milestone: ha4-milestone2ha4-deadline
Type: ожидается проверкаожидаются исправления

Замечания:

  1. Вместо явного обращения к оператору:
 template <typename T, typename S>
 bool operator==(const T * l, const lazy_basic_string<T, S> & r)
 {
     return (r.operator == (l));
 }

лучше используйте более интуитивное обычное сравнение:

 template <typename T, typename S>
 bool operator==(const T * l, const lazy_basic_string<T, S> & r)
 {
     return r == l;
 }
  1. Группу операторов сложения было бы проще реализовать через один operator+= и конструкторы от символа, Си-строки. Например:
lazy_basic_string operator+(const CharT * s2) const
{
  lazy_basic_string temp(*this);
  temp += lazy_basic_string(s2);
  return temp;
}

это должно существенно сократить и упростить код.

  1. Используйте в качестве типов для объектов-исключений либо готовые классы-исключения из STL (например, std::runtime_error) или создавайте свои производные классы от std::exception (или других, вроде std::runtime_error):
throw std::string("Bad index");

Исправьте, пожалуйста, в ближайшее время.

comment:15 Changed 7 years ago by Andrey

Type: ожидаются исправленияожидается проверка

comment:16 Changed 7 years ago by cpptester

Type: ожидается проверкаожидаются исправления

Tested revision 1037 by pestov.andrej.

Main tests

test

stage

result

info

log

run_aug_plus_c_str_char_traits_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_aug_plus_c_str_char_traits_no_mem-run.log

run_aug_plus_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_aug_plus_c_str_no_mem-run.log

run_aug_plus_wchar_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_aug_plus_wchar_no_mem-run.log

run_plus_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_c_str_no_mem-run.log

run_plus_char_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_char_no_mem-run.log

run_plus_lazy_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_lazy_str_no_mem-run.log

Common header used in some tests: common.h

comment:17 Changed 7 years ago by cpptester

Tested revision 1046 by pestov.andrej.

All tests passed. Good job!

comment:18 Changed 7 years ago by cpptester

Tested revision 1048 by pestov.andrej.

Main tests

test

stage

result

info

log

run_plus_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_c_str_no_mem-run.log

run_plus_char_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_char_no_mem-run.log

run_plus_lazy_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_plus_lazy_str_no_mem-run.log

Common header used in some tests: common.h

comment:19 Changed 7 years ago by cpptester

Tested revision 1049 by pestov.andrej.

All tests passed. Good job!

comment:20 Changed 7 years ago by Vladimir Rutsky

Смените тип тикета на "ожидается проверка", когда ваше решение будет готово для проверки.

comment:21 Changed 7 years ago by Andrey

Cc: andreypestov82@… added
Type: ожидаются исправленияожидается проверка

comment:22 Changed 7 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

Решение зачтено.

comment:23 Changed 7 years ago by Vladimir Rutsky

Milestone: ha4-deadline

Milestone ha4-deadline deleted

Note: See TracTickets for help on using tickets.