public? и порядок методов - Githubissues

Phibonoci / Groebner-Basis

Курсовая

0 stars 0 forks source link

public? и порядок методов #4

Open DimaTrushin opened 4 years ago

DimaTrushin commented 4 years ago

https://github.com/Phibonoci/Groebner-Basis/blob/2296d4834a1ca8280f906ebc631d03a3d3e9f587/overflow_detector.h#L37

По поводу функций проверки на overflow. У меня возникло два вопроса:

Нужны ли эти методы в публичном интерфейсе? Наверное да.
Не нужно ли их сгруппировать отдельно? А именно все методы проверки на overflow идут вместе, и операторы inplace тоже иду вместе.

Phibonoci commented 4 years ago

Да, нужны, без этого на assert-ах ничего не потестишь.
С точки зрения просмотра кода именно эти функции - основа имплементации операторов и мне кажется логичным располагать их вместе, хотя тут дело вкуса скорее (: