|
What?????
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
which line is wrong in my code?
|
|
|
|
|
All of it. Please read the suggestions you have been given
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
|
Then we can't help you further
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
good one
|
|
|
|
|
As others have said, your connection is open already. You could wrap the line c.Open() with
if (c.ConnectionState != ConnectionState.Open)
c.Open();
|
|
|
|
|
|
We are here to help but you must also put forth some effort. Copy & Paste is not a valid coding technique. You need to understand the objects you are working with. This error is very easily corrected by simply using the Intellisence built in to Visual Studio.
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
if (c.State != ConnectionState.Open)
c.Open(); ---- accept that this code ***** but "c.ConnectionState" wrong
|
|
|
|
|
For goodness sake, I just typed into the textbox here. I'm not opening things up in Visual Studio and creating a whole project just to test things for you.
|
|
|
|
|
trying to figure out a way to get it right...
1. items in a collection fall in one of the category parent or orphan
2. items in a collection may have one or more independent parent / one or more orphan
3. Both (Parent and Orphan) may or maynot have Individual behaviour
4. If a parent has one or null behaviour it should be applied to children. Individual behaviour of children item be hidden
5. If a orphan has one or null behaviour it should be applied to it
6. An item will have only one or null behaviour
trying to rephrase the following code...
private static void ApplyBehaviour(IEnumerator items, Behaviour behaviour = null)
{
while(items.MoveNext())
{
ItemModel item = (ItemModel)items.Current;
if(item.type = parent)
{
parentModel parentModel = (parentModel)item;
if(parentModel.HasBehaviour)
{
behaviour = parentModel.Behaviour;
}
ApplyBehaviour(parentModel.Children.GetEnumerator(), behaviour);
if(parentModel.HasBehaviour)
{
behaviour = null;
}
}
item.Behaviour = behaviour;
}
}
}
inital call will be ApplyBehaviour(items);
if two items with common parent and different behaviour, then the above code gets the
first item and children right...takes the parent behaviour but the second item retains its behaviour and applie it to children.
not sure whether i have explained in a way that someone can read and understand
- Regards -
J O N
A good thing is a bad thing if it keeps you from the best thing. - Dr. Adrian Rogers
modified on Wednesday, June 1, 2011 8:37 AM
|
|
|
|
|
It looks like the problem is that once you finish the first sibling with custom behaviour, you reset the value of behaviour to null. In other words, the logic as coded starts ignoring the value passed in after discovering the first sibling with parentModel.HasBehaviour == true.
Changing the code as follows avoids this problem:
if(item.type = parent) {
ParentModel parentModel = (parentModel)item;
ApplyBehaviour(
parentModel.Children.GetEnumerator()
, parentModel.HasBehaviour ? parentModel.Behaviour : behaviour
);
}
|
|
|
|
|
What is the reason of memory consumption in the DrawRoi function leading to hundreds of Mb?
There is a Frame bitmap and its FrameWithRoi copy to assign to PictureBox.
On the mouse move event new ROI coords are estimated and DrawRoi() is called.
DrawRoi()
{
RectangleF roi = Roi;
RectangleF rect = new RectangleF()
{
X = (float)Frame.Width * roi.X,
Y = (float)Frame.Height * roi.Y,
Width = (float)Frame.Width * roi.Width,
Height = (float)Frame.Height * roi.Height
};
if (rect.Left > 0 || rect.Right < (float)FrameWithRoi.Width ||
rect.Top > 0 || rect.Bottom < (float)FrameWithRoi.Height)
{
Graphics g = Graphics.FromImage(FrameWithRoi);
g.DrawImage(Frame, 0, 0);
using (Bitmap region = new Bitmap(FrameWithRoi.Width, FrameWithRoi.Height, FrameWithRoi.PixelFormat))
{
Graphics roi_g = Graphics.FromImage(region);
roi_g.FillRectangle(new SolidBrush(Color.FromArgb(64, 0, 0, 0)), new Rectangle(0, 0, FrameWithRoi.Width, FrameWithRoi.Height));
roi_g.FillRectangle(new SolidBrush(Color.Red), rect);
region.MakeTransparent(Color.Red);
g.DrawImage(region, 0, 0);
}
}
this.framePictureBox.Image = FrameWithRoi;
}
Чесноков
|
|
|
|
|
Two things strike me immediately:
- you are not disposing of the graphics objects you obtain. This might actually be a resource leak.
- you create a new RectangleF every time (though I think this is a value type, so it shouldn't matter).
It's also possible that the bitmap doesn't get garbage collected immediately, even though it is being disposed of at the end of the using. But it should do.
|
|
|
|
|
Yes, disposing graphics objects solved the problem.
But why the drawing is very slow? The larger the image the slower update.
Чесноков
|
|
|
|
|
Graphics objects are expensive, and you should dispose them. Why not use a using statement, just like you did for the region Bitmap?
And why would you need two Graphics instances, and a DrawImage()? Couldn't you just use the Bitmap(Image) constructor, and have its Graphics, just the one?
Luc Pattyn [My Articles] Nil Volentibus Arduum
The quality and detail of your question reflects on the effectiveness of the help you are likely to get. Please use <PRE> tags for code snippets, they improve readability. CP Vanity has been updated to V2.3
|
|
|
|
|
I was looking for a way to darken non ROI region in the image.
You optimizations would be welcomed.
Why the picture box update is very slow if I keep drawing on every mouse move event?
Чесноков
|
|
|
|
|
Mouse-move events are fired continuesly, and the drawing routine looks expensive. You might want to check whether the mouse has moved in the meantime, not redrawing if there's no movement detected.
Bastard Programmer from Hell 
|
|
|
|
|
I recommend overlaying the semitransparent mask over the base image (either in OnPaint on the picture box or by placing another object over the top). Then all you have to redraw is the mask.
Creating graphics objects is slow, which is why you should typically do it in OnPaint. In fact, in this case, you should do that, and your mouse move handler should only call Invalidate.
|
|
|
|
|
One other thing to bear in mind. The managed version of GDI+ is slow in comparison to the unmanaged version. You pay a price for the simplicity, and that price is speed.
|
|
|
|
|
Chesnokov Yuriy wrote: Why the picture box update is very slow if I keep drawing on every mouse move event?
Because the MouseMove events fire at high frequency (allowing you to track well assuming your handler is fast), possibly 50Hz or higher (which is too high for the human eye to notice), because Graphics objects are expensive, and finally because PictureBoxes are stupid (I wouldn't be surprised if the fact you change the image just causes another mouse move event).
You want to run such things continuously? I would experiment with keeping the two Graphics and the Bitmap around as class members, i.e. not create new objects at all, just keep using them over and over.
Luc Pattyn [My Articles] Nil Volentibus Arduum
The quality and detail of your question reflects on the effectiveness of the help you are likely to get. Please use <PRE> tags for code snippets, they improve readability. CP Vanity has been updated to V2.3
|
|
|
|
|
I need to implement rubber band like ROI selection in the image as in paint editor
Чесноков
|
|
|
|
|
What I understand is that you are trying to darken a rectangle within a image, and that rectangle must be refreshed as long as the mouse is moving. Am I right?
If this is what you want to do, the problem is that you are creating too many new Graphics and Bitmap objects in a very short time (and these are expensive objects), and updating the Image property a lot of times as well. The best way to do it is to subscribe to the Paint event of the control and code everything in this event handler, so you will receive a Graphics object as a parameter. In the next example there is only a PictureBox in the Form, called pct:
public partial class Form1 : Form
{
bool drawRect = false;
Rectangle r = new Rectangle();
Brush br = new SolidBrush(Color.FromArgb(50, Color.Red));
public Form1()
{
InitializeComponent();
}
private void pct_Paint(object sender, PaintEventArgs e)
{
if (drawRect)
e.Graphics.FillRectangle(br, r);
}
private void pct_MouseDown(object sender, MouseEventArgs e)
{
drawRect = true;
r.X = e.X;
r.Y = e.Y;
}
private void pct_MouseMove(object sender, MouseEventArgs e)
{
r.Width = e.X - r.X;
r.Height = e.Y - r.Y;
Refresh();
}
private void pct_MouseUp(object sender, MouseEventArgs e)
{
drawRect = false;
Refresh();
}
}
I've made it quick. You would have to modify it in order to draw the rectangle if the mouse moves to a lower X and/or lower Y coordinate.
|
|
|
|
|
No, I need to darken the area outside rectangle.
Consider ROI region. ROI remains clear and the area outside it is darked.
I tried preinitialization of graphics and bitmaps and found that the problem part taking much of the time is:
...
roi_g.FillRectangle(new SolidBrush(Color.FromArgb(64, 0, 0, 0)), new Rectangle(0, 0, FrameWithRoi.Width, FrameWithRoi.Height));
roi_g.FillRectangle(new SolidBrush(Color.Red), rect);
region.MakeTransparent(Color.Red);
g.DrawImage(region, 0, 0);
...
Despite prinitialization it takes about 150ms
Чесноков
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
