TDD vs не TDD

Пятница, 04 Сентября 2020 г. 14:40 + в цитатник

Fester:

Цитата korvin @ 04.09.20, 05:13

В смысле «я знаю»? класс Request публичный с публичными полями. Все знают, как оно реализовано.

Все знают публичные поля класса Request (имя и тип), но никто не знает какие из этих публичных полей используются функцией submit. А значит, для вызова submit тебе надо либо инициализировать все поля класса Request, либо знать какие поля класса Request использует submit, чтобы не создавать лишние объекты.

Цитата korvin @ 04.09.20, 05:13

может мне ещё и JDK мокать? java.lang.Math.sin, например?

java.lang.Math.sin мокать не нужно, а, скажем, доступ к файловой системе нужно.

Цитата korvin @ 04.09.20, 05:13

А то ж вдруг я его вызываю, да синус считаю. Ты сам-то мокаешь стандартную библиотеку?

Я мокаю то, что мне нужно. Например доступ к реестр или доступ к файлам - тоже вполне себе стандартные штуки.

Цитата korvin @ 04.09.20, 05:13

Также как и любое другое поле: передам нужное мне значение в конструктор

Ты понимаешь, что вместо вызова некой функции (в данном случае canWithdraw) ты предлагаешь создать некое поле, которое должно заполняться гезультатом работы этой функции. При этом мало того, что функцию все равно придется вызывать (в продуктивном коде), так еще у тебя и Request'e добавится куча полей.

Цитата korvin @ 04.09.20, 05:13

С хера ли? Ты что, Егора Бугаенко начитался/насмотрелся? Plain data там, никаких объектов, никаких абстракций.

Не знаю кто такой Егор Бугаенко.
Никаких plain data там нет. Смотри свой собственный код, у тебя у Account есть как минимум две функции: hasSameOwnerAs и canWithdraw плюс к этому read-only проперти owner. Ясен хрен, что тут должна быть абстракция.

Цитата korvin @ 04.09.20, 05:13

1) у каждой функции своя область определения, значит у каждой будет свой Request

Отлично! Т.е. Request - это тоже нифига не plain data объект!
Это должен быть интерфейс типа

public interface IRequest

{

Money Amount {get;} // тут хз, возможно тоже должна быть абстракция

IAccount Source {get;}

IAccount Destination {get;}

Action CheckCompliance{ get; set; }

}

Цитата korvin @ 04.09.20, 05:13

2) нафига перегружать модуль Transfer ещё десятком какимх-то функций? Про Single Responsibility Principle слышал?