Menu

#147 Typos when typing in initial values for T for a Column crash DWSIM

4.1
closed
None
2017-03-29
2016-12-15
Alex
No

Typing in "370]" instead of "370" in the Initial Estimates tab of the Column Configuration group of the Distillation Column object editor results in an unhandled error, Conversion from string "370]" to type 'Double' is not valid.

The trace mentions DWSIM.UnitOperations.EditingForm_Column_InitialEstimates.dgvv_CellValueChanged.

This shouldn't happen. Typos should just be ignored, or cause the cell to reset to a default value.

Discussion

  • Daniel Medeiros

    Daniel Medeiros - 2016-12-31
    • status: open --> wont-fix
    • assigned_to: Daniel Medeiros
     
  • Daniel Medeiros

    Daniel Medeiros - 2016-12-31

    I believe that the error message which is displayed in the exception window is enough to tell the user what went wrong.

     
  • Alex

    Alex - 2017-01-03

    No, the problem is that the original value is deleted and replaced with the new invalid string.

    An exception message is not user friendly, but even if you display one, you should throw out the new value and keep the original one.

     
  • Daniel Medeiros

    Daniel Medeiros - 2017-01-03
    • status: wont-fix --> accepted
     
  • Daniel Medeiros

    Daniel Medeiros - 2017-01-08
    • status: accepted --> pending
     
  • Daniel Medeiros

    Daniel Medeiros - 2017-03-29
    • status: pending --> closed