How do I simplify this

[JustinN]

I have 2 classes (Graphics circle, Graphics Item) Than I have a form that calls these classes and code to draw on this form. what do I need to do to use this code through out my program without using all of this code.

 

Public Class frmdraw

Inherits System.Windows.Forms.Form

' enums....

Public Enum GraphicsTools As Integer

CirclePen = 0

End Enum

Public Enum GraphicsSizes As Integer

Small = 4

Medium = 10

Large = 20

End Enum

' members.....

Public GraphicsItem As New ArrayList

Public GraphicTool As GraphicsTools = GraphicsTools.CirclePen

Public GraphicsSize As GraphicsSizes = GraphicsSizes.Large

Public GraphicColor As Color = Color.Black

' DoMousePaint - respond to a mouse movement....

Private Sub DoMousePaint(ByVal e As MouseEventArgs)

' store the new item somewhere...

Dim newItem As GraphicsItem

' what tool are you using?

Select Case GraphicTool

' circlepen?

Case GraphicTool.CirclePen

' create a new graphics circle...

Dim circle As New GraphicsCircle

circle.SetPoint(e.X, e.Y, GraphicsSize, GraphicColor, True)

' store this for addition...

newItem = circle

End Select

' where you given an item?

If Not newItem Is Nothing Then

' add it to the list...

GraphicsItem.Add(newItem)

'invalidate...

Invalidate()

 

End If

 

End Sub

Protected Overrides Sub OnMouseDown(ByVal e As System.Windows.Forms.MouseEventArgs)

 

' is the button down?

If e.Button = MouseButtons.Left Then

DoMousePaint(e)

End If

 

 

 

End Sub

Protected Overrides Sub OnMouseMove(ByVal e As System.Windows.Forms.MouseEventArgs)

' is the button down?

If e.Button = MouseButtons.Left Then

DoMousePaint(e)

End If

End Sub

Protected Overrides Sub OnPaint(ByVal e As System.Windows.Forms.PaintEventArgs)

' go through the list...

Dim item As GraphicsItem

For Each item In GraphicsItem

'ask each item to draw itself

item.Draw(e.Graphics)

Next

End Sub

[JustinN]

Maybe im going about this all wrong. I have a small form with a textbox and an add button. The user can type in a number in the textbox and then click the add button. I want this to add the number to the database (which I already can do) and then load a form in which the user can draw on. This form needs to be linked with the number that the user ented in the textbox. so that the user can select the number later and it will bring up that form with the drawing on it.

What do you think??

[Iceplug]

You can declare an array of forms on your main form... or use an ArrayList to collect your forms. :)

[Mothra]

Try having the user draw in a control in the form (a picbox perhaps) and then saving the drawing as an image in the database. Then you can have a "blank" form and load the delected "drawing" into the picture box for viewing/editing, what-have-you.

[PlausiblyDamp]

You could serialize the GraphicsItem ArrayList and store that in the DB - just associate it with the number the user entered.

This could then be deserialized when the form is 'loaded' from the DB - if you need to keep the items editable this should work - if you just store the resulting image then you wouldn't be able to edit the items later.