Project

General

Profile

Actions
Copy link

Task #5210

closed

[expression] Переименование некоторых сущностей

Added by Alexander Kamkin over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Andrei Tatarnikov
Category:
-
Target version:
0.3
Start date:
08/21/2014
Due date:
% Done:

100%

Estimated time:
Detected in build:
svn
Published in build:
140915

Description

Название некоторых классов представляются неудачными, например,

  1. NodeExpr - выражение это не только (x + 1), но и x, и 1 - предлагаю переименовать NodeExpr в NodeOperation (EXPR -> OPERATION).
  2. OperationReducer - упрощается не операция (оператор), а выражение - ExpressionReducer (или ExprReducer) представляется более адекватным названием.
  3. Нужно внимательнее посмотреть код на предмет подобных несоответствий.
Actions
Copy link
 #1

Updated by Andrei Tatarnikov over 9 years ago

  • Status changed from New to Open
  • % Done changed from 0 to 30

Class NodeExpr was renamed to NodeOperation - r572.

Actions
Copy link
 #2

Updated by Andrei Tatarnikov over 9 years ago

OperationReducer - упрощается не операция (оператор), а выражение - ExpressionReducer (или ExprReducer) представляется более адекватным названием.

Здесь как раз все так. OperationReducer и Transformer принимают параметры типа NodeOperation и упрощают их (переменные и константы упрощать нет смысла). Если бы OperationReducer работал с объектами типа Node, тогда переименование имело бы смысл. Нужно ли так сделать?

Actions
Copy link
 #3

Updated by Andrei Tatarnikov over 9 years ago

OperationReducer остался как есть, но сделан package-private. А Transformer.reduce может принимать теперь объекты типа Node (если не операции, то возващаются без изменений).

Actions
Copy link
 #4

Updated by Andrei Tatarnikov over 9 years ago

  • Status changed from Open to Resolved
  • % Done changed from 30 to 100

Пока не вижу других проблем. Пакет transformer пока сырой, но это отдельная проблема.

Actions
Copy link
 #5

Updated by Andrei Tatarnikov over 9 years ago

  • Status changed from Resolved to Closed
  • Published in build set to 140915
Actions
Copy link

Also available in: Atom PDF