Fortress

    @Override
    public int calculate()
    {
        switch(this.getData().getType().getType())
        {
            case LOGIC_INTEGER:
                return (Integer) this.getData().getValue();
            case BIT_VECTOR:
                return ((BigInteger)this.getData().getValue()).intValue();
            case LOGIC_BOOLEAN:
                if ((Boolean) this.getData().getValue())
                    return 1;
                else
                    return 0;
            case LOGIC_REAL:
                return ((Double)this.getData().getValue()).intValue();
            case UNKNOWN:
                break;
        }
        return 0;
    }

Думаю, правильнее вместо этого switch добавить в интерфейс IData метод intValue, осуществляющий конвертацию значения в тип int.

Возможно правильнее использовать не int, а BigInteger.

Кстати, схлопывание констант делается только для параметров (для repeat и подобных конструкций)? Для HDL-кода нужно аккуратно делать вычисление констант, поскольку так в основном битовые вектора, причем с поддержкой неопределенных значений (X, Z). Лучше вычисления делать только для параметров.

Я подозреваю (не проверял), что NullPointerException происходит из-за того, что в моей реализации IVariable метод getData возвращает null (такая задумка - у переменной нет значения). Если в выражении присутствует переменная, схлопывания не должно происходить. Схлопываться должны только выражения, состоящие из значений.

Еще у моих данных getType() возвращает null.

switch(this.getData().getType().getType()) {...}

Это я как-нибудь доделаю. Но причина не только в этом.

Даже если избавиться от NPE возникнет CastException. Так как мой getValue() возвратит объект моего класса, а не Integer и т.п.

Возможное решение у IData сделать метод, получающий значение нужно типа. Нужно обсудить с Андреем.

См. таск #4246

Я написал калькулятор для основных стандартных операций EStandardOperation (MINUS, PLUS, ADD, SUB, MUL, DIV, REM, MOD) и API-классы для создания/расширения калькуляторов (см. класс syntax.Calculator). А у класса Operation появился метод для схлопывания подвыражений:

public ISyntaxElement reduce(ICalculator calculator, int flags)

Его можно использовать следующим образом

syntax.addFormula(
    new Formula(
        new Operation(
            EStandardOperation.EQ,
            a,
            new Operation(
                EStandardOperation.ADD,
                new Value(intType.valueOf("2", 10)),
                new Value(intType.valueOf("3", 10))).reduce(CalculatorForStandard.getInstance(), 0)
            )
        )
    );

Данный код помогает упростить выражение (+ 2 3) в число 5.

Сергей, посмотри старую релизацию (старые методы calculate в операциях и других классов). Возможно, они больше не нужны.