|
Depending on the complexity of the calculation and the number of times the getter is called, I would expect option 1 to be the preferred choice. This is because after the first call there is no calculation to be performed and the test for nullability should take fewer cycles than the call to the calculation method and the actual calculation.
In option 2 the _someValThatNeedsSetting variable would not be needed, but each call would require a call to the calc routine and the actual calculation.
It's time for a new signature.
|
|
|
|
|
I expect example 1 (with the test) to always be at least as fast as example 2 (with extra method call), unless the method is very simple (yours is!) and conditions are such that the compiler can inline the code (I think that requires a "no debug" condition, say a release build).
However, I don't like nullables much, so I would opt for a separate boolean flag here. So that would be an example 1+
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
Hi, still getting to know C# and recently stumbled upon a problem I'm not sure about.
I have two threads, each of them updating a WinForm's TextBox. I'm using the usual InvokeRequired:
if (this.OutputConsole.InvokeRequired)
{
this.OutputConsole.Invoke(
new OutputConsoleUpdater(delegate(string s) { this.OutputConsole.AppendText(s + "\r\n"); }),
text
);
}
else
{
this.OutputConsole.AppendText(text + "\r\n");
}
But bad things happened when I tried to add some sort of locking to that  o. I just added a lock around the above mentioned code
lock(ConsoleLock)
{
}
A deadlock occured, the thread that actually needs the Invoke gets the lock ("user thread"), and tries to call Invoke, "delegating" the code to the thread that created the control ("original thread"). The original thread, though, wants to write something to the TextBox too, and is stuck waiting for the lock, currently held by the user thread, which in turn keeps waiting for the original thread to process the delegated call. So far, so good.
The question is: if I remove the lock, why does it still work okay... is the TextBox, in some way, thread-safe? Is it due to the way the Invoke call works (and if so, how does it work in the first place)? I don't really see how I could make it work using locks anyway (as long as I have to use the above mentioned paradigm), do you see how it could be done?
thanks for your time !
|
|
|
|
|
Why do you think a lock is necessary here? Lock is necessary when accessing finite resources such as a file where you need to ensure only one thread updates it a time. With the textbox you only need to use the Invoke method it you are trying to update it from a thread other than the UI thread.
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
Um, coming from C++, I would have thought calling a function from two different threads might cause some re-entrancy issues when working on a common resource (the TextBox's content they both try to update). I would have expected the output to be either interleaved ("AAAA" and "BBBB" resulting in "AABABBBA" for example) in the better case, or some variable corruption in the worse case.
|
|
|
|
|
Yes, the original code is thread-safe. This is how it works:
- thread 1 calls the method containing your snippet, I'll call it SetText();
- it detects invoke is required, launches a windows message to the GUI thread, and waits for it to return;
- the GUI thread now executes the same method SetText(), sees no invoke is required, sets the TextBox property, and returns;
- this causes thread 1, still waiting inside its SetText(), to continue.
Now everything the GUI thread does gets sequenced (well, not absolutely aacurate, but good enough here) through its event queue, so even if another thread (thread 2) were to call SetText, it would basically execute the same steps as listed above, and the common part, executed on the GUI thread, would be sequenced by the event queue.
So any number of SetText() calls will execute in the order they get called, without them interfering amongst themselves.
Adding a lock does not make it any safer; and as you discovered, if it covers the entire content of SetText(), it brings everything to a halt, as the basic concept of the construct is that SetText, when called on a thread other than the GUI thread, causes itself to be executed once more, on the GUI thread, which would be prevented by the very lock you added.
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
Hi, thanks for the answer... good to know about the queue
I think you confused the SetText (aka the snippet) thing though. Unless C# does something behind my back (not that I'd be surprised  but I guess it would require the CLR to detect the caller of the Invoke method and recall it, and on top of that, it would render the delegate parameter useless), SetText doesn't get called again, so the deadlock is not due to the SetText method recursively calling itself in another thread (in fact, the deadlock only happened in certain cases and required a lot of tries to replicate in each of those cases).
In other words, "as the basic concept of the construct is that SetText, when called on a thread other than the GUI thread, causes itself to be executed once more, on the GUI thread" seems wrong to me Just going over it here so that I know I'm not mistaken...
|
|
|
|
|
Sorry, my reply wasn't very accurate; I should have payed more attention to the details of your code. The way I do such things typically is like so:
public void SetText(Control control, string text) {
if (control.InvokeRequired) {
control.Invoke(new ControlStringConsumer(SetText), new object[]{control, text});
} else {
control.Text=text;
}
}
and then my earlier reply would apply. My "canonical form" as I call it here[^], has the advantage of containing the actual operation code only once.
Your code, which was
if (this.OutputConsole.InvokeRequired) {
this.OutputConsole.Invoke(
new OutputConsoleUpdater(delegate(string s) { this.OutputConsole.AppendText(s + "\r\n"); }),
text);
} else {
this.OutputConsole.AppendText(text + "\r\n");
}
could be simplified to
this.OutputConsole.Invoke(
new OutputConsoleUpdater(delegate(string s) { this.OutputConsole.AppendText(s + "\r\n"); }),
text);
assuming we are NOT on the GUI thread. Using Invoke() it also causes the delegate to be executed on the GUI thread, through Windows messages, implying an automatic sequential treatment.
Now I'm not sure about the lock issue you raised; it may depend on where exactly you put it, and even more on what is inside the OutputConsoleUpdater constructor. But anyway, the conclusion will be there is no need for such a lock.
FWIW: you could break the automatic sequencing by using Application.DoEvents() inside some event handler; that is exactly why doing so is a very bad idea most of the time.
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
Perfect, the idiom you use is definitely more elegant  glad I learned something new today.
As for the deadlock, I'm quite sure why it happens (the "pump" that does the Invoke requests doesn't get called as the whole thread is stuck waiting in the first place), no problems there.
thanks a lot for your time! 
|
|
|
|
|
hello.
i am creating a jpeg encoder i need to access a image file.
can u tell me how can i read and write using dll.
getPixel(), setPixel() is very slow.
|
|
|
|
|
Is there a reason you can't use:
Bitmap RawImage = new Bitmap(Filename);
Once it's in the Bitmap, then you can use the pixel buffer for faster access. Google will help with how to do that.
CQ de W5ALT
Walt Fair, Jr., P. E.
Comport Computing
Specializing in Technical Engineering Software
|
|
|
|
|
|
I have the following classes in Java which i want to convert to C# but have errors with the undelined codes. (Can someone help me with the C# equivalent of PUT and GET)
class TagInfo{
public int tagOffset;
public int tagLength;
public TagInfo(){
}
}
private void buildMetaData()
{
Record record = getData();
int offset = IMGREC_TAGLIST_INDEX;
int numTags = 0 ;
short tagID = 0;
try
{
numTags = this.getNumberOfTags() ;
for (int i=0; i < numTags; i++){
TagInfo info = new TagInfo();
info.tagOffset = offset;
info.tagLength = TagRecord.TAG_DATA_INDEX + record.readInt(offset + TagRecord.TAG_DATALEN_INDEX);
tagID = record.readShort(offset + TagRecord.TAG_ID_INDEX);
offset += info.tagLength;
tagInfo.put(new Short(tagID),info);
Console.WriteLine("tagID = " + tagID);
}
}
catch (Exception ex)
{
}
finally
{
}
this.imageOffset = offset;
this.imageSize = record.getLength() - offset;
}
public TagRecord getTagByID(short id)
{
TagRecord record = null;
TagInfo info = (ImageRecord.TagInfo)tagInfo.get(new Short(id));
if (info != null){
byte[] data = this.getData().read(info.tagOffset,info.tagLength);
record = new TagRecord(data);
}
return record;
}
|
|
|
|
|
Those lines appear to be using a variable which does not exist. What is the type of tagInfo and where does it come from? It isn't a local variable or parameter or even a field in the class - so what is it?
|
|
|
|
|
This might not be quite right as you did not post the entier code, but the following should help you. Note the dictionary declaration and add/get code. I have used a generic dictionary to enable strong typing.
class TagInfo{
public int tagOffset;
public int tagLength;
public TagInfo(){
}
}
System.Collections.Generic.Dictionary<int, TagInfo>; TagInfoDictionary = new System.Collections.Generic.Dictionary<int, TagInfo>();
private void buildMetaData()
{
Record record = getData();
int offset = IMGREC_TAGLIST_INDEX;
int numTags = 0 ;
short tagID = 0;
try
{
numTags = this.getNumberOfTags() ;
for (int i=0; i < numTags; i++){
TagInfo info = new TagInfo();
info.tagOffset = offset;
info.tagLength = TagRecord.TAG_DATA_INDEX + record.readInt(offset + TagRecord.TAG_DATALEN_INDEX);
tagID = record.readShort(offset + TagRecord.TAG_ID_INDEX);
offset += info.tagLength;
this.TagInfoDictionary.Add(tagID, info);
Console.WriteLine("tagID = " + tagID);
}
}catch (Exception ex){}
finally{}
this.imageOffset = offset;
this.imageSize = record.getLength() - offset;
}
public TagRecord getTagByID(short id)
{
TagRecord record = null;
TagInfo info = this.TagInfoDictionary[id];
if (info != null){
byte[] data = this.getData().read(info.tagOffset,info.tagLength);
record = new TagRecord(data);
}
return record;
}
|
|
|
|
|
hi
what is the equivalent of this code in c#
this code is in VB6
Chr$(Asc("0")
thanks
|
|
|
|
|
Try Convert.ToChar(6);.
The output is a unicode value.
|
|
|
|
|
Apart from lacking a parenthesis, it does not make much sense; Asc() returns a number holding the ASCII value, which is 48 for a zero; and Chr$() does the inverse, it returns the ASCII character equivalent of a numeric value. So they cancel each other, it simply means "0"
In C# we use different delimiters for string literals and character literals, so you may want a simple '0'
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
i want convert eny caractere in ascii the 0 is a exemple
so if there is some code to calc ascci please send it to me
i need it now
|
|
|
|
|
in .NET an ASCII character is just a character, the first 128 Unicode characters ARE the ASCII characters.
So
'a'
"a"[0]
(char)97
are all the same lower-case a, as they are
a character literal
the way to get the first character of a string
a number cast to a character
If you don't understand the fundamentals, go buy a book and study it. If you have an overall problem, tell us about the problem rather than asking a detail question on what probably isn't even the right detail.
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
There is no equivalent; C# developers are smart enought not to write anything as silly as that. 
|
|
|
|
|
thank you it's like that when sharing information has a beginning researcher in c #
thank you very much
|
|
|
|
|
Sure, however you should allow some slack for VB coders trying to spit out some C#.
How about
char ascee='0';
char uniquode=Encoding.UTF8.GetChars(Encoding.ASCII.GetBytes(new char[]{ascee}))[0];
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
I only read formatted code with indentation, so please use PRE tags for code snippets.
I'm not participating in frackin' Q&A, so if you want my opinion, ask away in a real forum (or on my profile page).
|
|
|
|
|
the out result is the same of first value of ascee no chage ???
|
|
|
|
|
Luc Pattyn wrote: allow some slack for VB coders
They made their sty they can wallow in it.
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
