Task #5210
closed
[expression] Переименование некоторых сущностей
Added by Alexander Kamkin over 10 years ago.
Updated about 10 years ago.
Assignee:
Andrei Tatarnikov
Published in build:
140915
Description
Название некоторых классов представляются неудачными, например,
NodeExpr - выражение это не только (x + 1), но и x, и 1 - предлагаю переименовать NodeExpr в NodeOperation (EXPR -> OPERATION).OperationReducer - упрощается не операция (оператор), а выражение - ExpressionReducer (или ExprReducer) представляется более адекватным названием.- Нужно внимательнее посмотреть код на предмет подобных несоответствий.
- Status changed from New to Open
- % Done changed from 0 to 30
Class NodeExpr was renamed to NodeOperation - r572.
OperationReducer - упрощается не операция (оператор), а выражение - ExpressionReducer (или ExprReducer) представляется более адекватным названием.
Здесь как раз все так. OperationReducer и Transformer принимают параметры типа NodeOperation и упрощают их (переменные и константы упрощать нет смысла). Если бы OperationReducer работал с объектами типа Node, тогда переименование имело бы смысл. Нужно ли так сделать?
OperationReducer остался как есть, но сделан package-private. А Transformer.reduce может принимать теперь объекты типа Node (если не операции, то возващаются без изменений).
- Status changed from Open to Resolved
- % Done changed from 30 to 100
Пока не вижу других проблем. Пакет transformer пока сырой, но это отдельная проблема.
- Status changed from Resolved to Closed
- Published in build set to 140915
Also available in: Atom
PDF