Показать сообщение отдельно
Старый 14.03.2008, 16:22
Iv вне форума Посмотреть профиль Отправить личное сообщение для Iv Посетить домашнюю страницу Iv Найти все сообщения от Iv
  № 25  
Iv
 
Аватар для Iv

Регистрация: Apr 2001
Адрес: Moscow
Сообщений: 1,475
По умолчанию Тестирование нового метода

Прежде чем заменять вызовы старого метода на новый, мы обязательно(!!!) должны протестировать наш новый метод split.
Это ключевой и, как вы убедитесь, совсем не лишний шаг в выбранном способе рефакторинга.
Создавая новый метод мы имеем право делать всё что нам заблагорассудится с новым кодом, но старый код не имеем права трогать. И, когда новый метод готов, мы тестируем его. Тестирование нового метода обязательно производится заменой всей логики старого метода на вызов нового метода.

Продублируем метод Math2.bezierSplit и закомментируем верхний, чтобы было легко откатиться в случае ошибки.
После чего удалим всю старую логику и заменим на новую:
Код AS3:
private static function bezierSplit(p1x:Number, p1y:Number, c1x:Number, c1y:Number, c2x:Number, c2y:Number, p2x:Number, p2y:Number):Array {
	var curve : CubicBezierSVG = new CubicBezierSVG(
		new Point(p1x, p1y), 
		new Point(c1x, c1y),
		new Point(c2x, c2y),
		new Point(p2x, p2y)); 
 
	var halves:Array = curve.split();
	var firstCurve : CubicBezierSVG = halves[0] as CubicBezierSVG; 
	var secondCurve : CubicBezierSVG = halves[1] as CubicBezierSVG; 
 
	return [
			[firstCurve.start, firstCurve.startControl, 
			firstCurve.endControl, firstCurve.end], 
 
			[secondCurve.start, secondCurve.startControl, 
			secondCurve.endControl, secondCurve.end] 
	];
}
Тестируем. Работает! Только после этого мы можем приступать к следующему шагу: замене вызовов старого метода на новый.
__________________
http://realaxy.com


Последний раз редактировалось iNils; 20.12.2010 в 13:24.