This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
| Summary: | RE Operation: Nested switch blocks are parsed incorrectly | ||
|---|---|---|---|
| Product: | uml | Reporter: | Alexandr Scherbatiy <sunflower> |
| Component: | Reverse Engineering | Assignee: | issues@uml <issues> |
| Status: | NEW --- | ||
| Severity: | blocker | ||
| Priority: | P3 | ||
| Version: | 5.x | ||
| Hardware: | All | ||
| OS: | All | ||
| Issue Type: | DEFECT | Exception Reporter: | |
| Attachments: |
nested switch blocks
nested switch blocks (Latest IDE) NPE when RE the operation with switch block The sequence diagram in NB6 is incorrect too. It is different from nb5.5.1. Attached the diagram which is generated with build 070718 |
||
Created attachment 31376 [details]
nested switch blocks
Created attachment 38898 [details]
nested switch blocks (Latest IDE)
Created attachment 38976 [details]
NPE when RE the operation with switch block
Created attachment 45353 [details]
The sequence diagram in NB6 is incorrect too. It is different from nb5.5.1. Attached the diagram which is generated with build 070718
as for graphics issue: it's much better now(look), but not yet perfect, at least nested cfs are located inside correct operands, but still may be sometimes in wrong ordering with messages. in fact it's the issue with combined fragments which do not contain messages, usually with primitive types. yet have [n] instead of correct expressions, it's second issue is model and both amy need to be separated. |
Steps to reproduce: - Do Reverse Engineering of 'void test()' method from 'Test' class and create a Sequence diagram: --------------------------------------------------------------- public class Test { enum DAY { SUNDAY, MONDAY, TUESDAY, WEDNESDAY, THURSDAY, FRIDAY, SATURDAY } public void test(){ int n = 20; String name = ""; DAY day = DAY.MONDAY; switch(day){ case SUNDAY: name = day.toString(); switch(n){ case 0: day = DAY.SUNDAY; case 1: day = DAY.MONDAY; break; default: day = DAY.TUESDAY; } case MONDAY: name = day.name(); switch(n){ case 2: day = DAY.TUESDAY; case 3: day = DAY.MONDAY; break; default: day = DAY.SUNDAY; } break; default: name = day.getClass().getName(); } } } --------------------------------------------------------------- The outer Combined fragment has 3 interaction operands on the Sequence diagram. The fist operand has '[n]' expression instead of '[ day = SUNDAY ]'. The second operand has '[n]' expression instead of '[ day = MONDAY ]'.