Домашнее задание 4 (2я попытка)

[Максим]

Первая версия ленивой строки.

Есть несколько вопросов:

1. В задании указано, что требуется реализовать деструктор класса ленивой строки. Есть ли в этом смысл, если всюду используется vector + shared_prt? Или имелось в виду сделать его виртуальным, чтобы избежать утечек памяти при наследовании от него?
2. В этом решении есть недостаток - каждый экземпляр объекта содержит поле size_. В дальнейшем я его планирую убрать, можно ли будет пользоваться для этого методом size у вектора символов?

Максим.

[cpptester]

Tested revision 728 by petrov.maksim.

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

[Vladimir Rutsky]

Replying to petrov.maksim:

Первая версия ленивой строки.

Есть несколько вопросов:

1. В задании указано, что требуется реализовать деструктор класса ленивой строки. Есть ли в этом смысл, если всюду используется vector + shared_prt? Или имелось в виду сделать его виртуальным, чтобы избежать утечек памяти при наследовании от него?

Можно не реализовывать деструктор, если вам достаточно автоматически сгенерённого компилятором деструктора.
В классе строки не должно быть виртуальных функций.

2. В этом решении есть недостаток - каждый экземпляр объекта содержит поле size_. В дальнейшем я его планирую убрать, можно ли будет пользоваться для этого методом size у вектора символов?

Да, если вы используете vector для хранения буфера символов, то разумно использовать его size() И не хранить дублирующуюся информацию.

[cpptester]

Tested revision 799 by petrov.maksim.

Main tests

test	stage	result	info	log
​run_index_assign_assign.cpp	gcc62	SUCCESS
	run	FAILURE	exit code 139	​run_index_assign_assign-run.log

Common header used in some tests: ​common.h

[cpptester]

Tested revision 800 by petrov.maksim.

All tests passed. Good job!

[Максим]

Местные тесты проходят успешно.
Удалил из класса строки поле size_, вместо него теперь используется метод vector::size() у буфера символов. В буфер добавлен терминальный ноль для корректной работы метода c_str().
Жду проверки и замечаний.

Максим.

[Vladimir Rutsky]

Проверка автотестами не считается проверкой, вернул milestone на вторую попытку сдачи, скоро проверю.

[Максим]

Доброго времени суток, Владимир!

Я уже написал кураторам на почту об уходе из CS центра, не тратьте время на проверку этого задания.

Спасибо за помощь и ценные советы.

[Vladimir Rutsky]

Здравствуйте, Максим!

Хорошо, я тогда просто закрою тикет.

Успехов!

[Vladimir Rutsky]

Milestone ha4-milestone1 deleted