Closed nekr0z closed 1 year ago
position.IsValid имело более говорящее название, чем validation.IsLegal, как мне кажется.
Может хотя бы validation.IsBoardLegal ? Чтобы какой-то контекст был в названии что это про доску?
Чтобы какой-то контекст был в названии что это про доску?
Оно принимает board.Board
в качестве аргумента, чем не контекст про доску? ;)
Чтобы какой-то контекст был в названии что это про доску?
Оно принимает
board.Board
в качестве аргумента, чем не контекст про доску? ;)
Пусть @sadmadrus выскажется
validation.IsLegal - вполне норм. И название пакета вполне соотвтетствует содержимому
название пакета вполне соотвтетствует содержимому
Мне не очень нравится validation.ThreatsTo(), но не приходит в голову, как это лучше сформулировать
название пакета вполне соотвтетствует содержимому
Мне не очень нравится validation.ThreatsTo(), но не приходит в голову, как это лучше сформулировать
в термина я так понимаю это "битое поле" - bat box. Но не логично ли было его сделать логическим?
Но не логично ли было его сделать логическим?
Там возвращается список полей, на которых стоят фигуры, которые держат под боем это поле. Там не просто «битое/не битое».
ThreatenBy CheckedBy UnderCheck
CheckedBy
Годно. Исправил.
Надо освободить имя для пакета, который будет оценивать шах/мат/пат.