Webtoon Paint

Final Presentation

Team 2

200911371 김민철

200911381 김진현

200911417 정명권

1. Unit Test Report

2. System Test Respond Report

3. Static Analysis Respond Report

4. Epilogue

I N D E X

Writing on OSP1000ver4, OSP2030ver3, OSP2040ver2

Unit Test Report

1. Unit Test Report Complementing 6 Unit Test Case

requestErasing(), requestSetColor() requestDelArea(), requestCopyArea() requestChangeCut(), requestSetCutSize()

1. Unit Test Report

System Test Respond Report

2. System Test Respond Report Communication problem

→ Change to “PNG로 저장한다”

2. System Test Respond Report Document refine mistake

→ Remove from OSP 2040

2. System Test Respond Report Lack of exception handling

→ Add Exception handling

2. System Test Respond Report Funtion Error

→ Make program not act in condition that width or height is 0

Static Analysis Respond Report

3. Static Analysis Respond Report

About ‘Name’ • Change field names to lowercase.

• Remove under bar(_) in field that contain under bar(_)

• Change the method name starts with a lowercase letter.

3. Static Analysis Respond Report

About ‘Name’ • Change field names to lowercase.

• Remove under bar(_) in field that contain under bar(_)

• Change the method name starts with a lowercase letter.

3. Static Analysis Respond Report

About ‘Name’ • Change field names to lowercase.

• Remove under bar(_) in field that contain under bar(_)

• Change the method name starts with a lowercase letter.

3. Static Analysis Respond Report

About ‘Name’ • Change field names to lowercase.

• Remove under bar(_) in field that contain under bar(_)

• Change the method name starts with a lowercase letter.

3. Static Analysis Respond Report

Name After Modifying

btnNewBUtton loadbutton btnNewButton_1 savebutton btnNewButton_2 brushbutton btnNewButton_3 eraserbuton btnNewButton_4 areabutton btnNewButton_5 delareabutton btnNewButton_6 cutoffbutton btnNewButton_7 copybutton btnNewButton_8 pastebutton btnNewButton_9 addcutbutton btnNewButton_10 delcutbutton btnNewButton_11 mergebutton btnNewButton_13 sharpbutton btnNewButton_14 thickbutton

3. Static Analysis Respond Report

Name After Modifying

Area area bef_x befx bef_y befy BrushConf brushconf C c CanvasList canvaslist CanvasPanel canvaspanel CB cb CI ci ColorGroup colorgroup CurrentGraphic currentgraphic CurrentImage currentimage Cut cut CutListGroup cutlistgroup

CutListPanel cutlistpanel

Draw draw

EnabledCutNum enabledcutnum

3. Static Analysis Respond Report

Name After Modifying

File file

flowLayout_1 flowlayout1

flowLayout_2 flowlayout2

IF inf

SelectedTool selectedtool

SizeGroup sizegroup

ToolGroup toolgroup

ToolLeft toolleft

ToolTop tooltop

3. Static Analysis Respond Report

Name After Modifying

button c1button button_1 c2button button_2 c3button button_3 c4button button_4 c5button button_5 c6button button_6 c7button button_7 c8button button_8 c9button button_9 c10button button_10 c11button button_11 c12button button_12 c13button button_13 c14button button_14 c15button button_15 c16button ButtonList buttonlist

3. Static Analysis Respond Report

Name After Modifying

LoadCut loadCut

SaveCut saveCut

3. Static Analysis Respond Report

About ‘Visibility Modifier’

Using private instead public for class encapsulate. • Main class does not encapsulate the fields.

Generate accessor functions to access internal fields. • Do not generate accessor function to fields that used only inside class.

Do not generate accessor function to fields that used only inside method.

Do not generate accessor function to fields that provided by Java.

Main class does not encapsulate the fields.

Do not generate accessor function to fields that used only inside method or class.

Do not generate accessor function to fields that provided by Java.

3. Static Analysis Respond Report

About ‘IF’ Remove unnecessary comparisons including boolean variables.

IF statements enclosed in braces.

Else statement enclosed in braces.

3. Static Analysis Respond Report

If(comparison == true) -> if(comparison)

If(comparison == false) -> if(!comparison)

3. Static Analysis Respond Report

About ‘Unused & Comment’

Remove unused field, method, imports

Remove testing code

3. Static Analysis Respond Report Cyclomatic Complexity

→ remove switch case and add new field to supplant.

3. Static Analysis Respond Report

Cyclomatic Complexity → Shift out method.

5.5. NCSS Method Count

NCSS Method Count

→ Make new function for reduce line.

4. Epilogue

Thanks