![]() |
|
||||||||||
|
|||||
|
Регистрация: Apr 2001
Адрес: Moscow
Сообщений: 1,475
|
Прежде чем заменять вызовы старого метода на новый, мы обязательно(!!!) должны протестировать наш новый метод split.
Это ключевой и, как вы убедитесь, совсем не лишний шаг в выбранном способе рефакторинга. Создавая новый метод мы имеем право делать всё что нам заблагорассудится с новым кодом, но старый код не имеем права трогать. И, когда новый метод готов, мы тестируем его. Тестирование нового метода обязательно производится заменой всей логики старого метода на вызов нового метода. Продублируем метод Math2.bezierSplit и закомментируем верхний, чтобы было легко откатиться в случае ошибки. После чего удалим всю старую логику и заменим на новую: 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. |
![]() |
Часовой пояс GMT +4, время: 16:42. |
|
|
« Предыдущая тема | Следующая тема » |
|
|