|
Don't make it async then.
Every "transaction" should return a "return code" to the "caller"; the ui in this case.
Make a "synchronous" call with say a 5 sec. timeout. A timeout is a "no server response".
Don't leave the user hanging; you need to tell them success or failure.
In any event, if it's not a duplicate, etc. it's a "system error" and needs to be escalated.
"Graceful degeneration". If the user CANNOT proceed, these is nothing "graceful" you can do except inform the user.
It was only in wine that he laid down no limit for himself, but he did not allow himself to be confused by it.
― Confucian Analects: Rules of Confucius about his food
|
|
|
|
|
ContinueWith doesn't really belong in an async method.
private async void AddUser()
{
UserEntity user = new UserEntity
{
FirstName = "Jack",
LastName = "Stone",
UserName = "jstone",
Password = ""
};
await Task.Run(() =>
{
IBizObj bo = new BizObj(connString);
bo.AddUser(user);
});
UserList.Add(user);
} Also, you should try to avoid async void methods wherever possible.
Avoid async void methods | You’ve Been Haacked[^]
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
So the method doesn't need to return anything, and I'd like the UI to remain resonsive until the server trip is finished. What's the right approach here?
If it's not broken, fix it until it is.
Everything makes sense in someone's mind.
Ya can't fix stupid.
|
|
|
|
|
The code I posted will push the IBizObj onto a background thread. When it completes, the method will result on the UI thread to run the UserList.Add code. There's no need to use ContinueWith here.
Ideally, your method should return a Task . If you want it to be a "fire and forget" call, assign the returned task to a discard variable:
_ = AddUser(); That way, any unhandled exceptions will raise the TaskScheduler.UnobservedTaskException[^] event.
With an async void method, any unhandled exceptions could bring down the entire process.
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
I have this setup of a list containing dates and strings:
var d0 = new DateTime(2019, 10, 20);
var d1 = new DateTime(2018, 10, 20);
var d2 = new DateTime(2019, 10, 21);
var d3 = new DateTime(2019, 10, 20);
var d4 = new DateTime(2017, 10, 21);
var d5 = new DateTime(2013, 10, 22);
var d6 = new DateTime(2015, 10, 21);
var d7 = new DateTime(2010, 10, 23);
var d8 = new DateTime(1995, 10, 20);
var d9 = new DateTime(2019, 10, 22);
var list = new List<(DateTime Date, string Name)>();
list.Add((d0, "D0"));
list.Add((d1, "D1"));
list.Add((d2, "D2"));
list.Add((d3, "D3"));
list.Add((d4, "D4"));
list.Add((d5, "D5"));
list.Add((d6, "D6"));
list.Add((d7, "D7"));
list.Add((d8, "D8"));
list.Add((d9, "D9")); I want to use LINQ to transform the list in the following order:
(2019-10-20, "D0")
(2018-10-20, "D1")
(2019-10-20, "D3")
(1995-10-20, "D8")
(2019-10-21, "D2")
(2017-10-21, "D4")
(2015-10-21, "D6")
(2013-10-22, "D5")
(2019-10-22, "D9")
(2010-10-23, "D7") Notice the month and day decides the order, not the year.
I have tried to mess around with LINQ's GroupBy() method, but cannot understand how to group dates where months and days are equal.
Also there is a sorting rule:
Notice the "D0"-item comes before the "D1"-item, even if the "D1"-item has year set to 2018 and the "D0"-item has year set to 2019.
|
|
|
|
|
You're not grouping the list; you're sorting it.
With LINQ:
var sortedSequence = list
.OrderBy(p => p.Date.Month)
.ThenBy(p => p.Date.Day)
.ThenBy(p => p.Name); With a custom comparer:
public class MonthDayNameComparer : IComparer<(DateTime date, string name)>
{
public int Compare((DateTime date, string name) left, (DateTime date, string name) right)
{
int result = left.date.Month.CompareTo(right.date.Month);
if (result == 0) result = left.date.Day.CompareTo(right.date.Day);
if (result == 0) result = StringComparer.Ordinal.Compare(left.name, right.name);
return result;
}
}
...
list.Sort(new MonthDayNameComparer());
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Thank you!
The LINQ example worked just fine!
|
|
|
|
|
Hello @all,
if have a question regarding a ToolTip i created:
private void showTooltip(Control position, string Title, string Message)
{
ToolTip t = new ToolTip();
try
{
t.UseFading = true;
t.UseAnimation = true;
t.IsBalloon = true;
t.ShowAlways = true;
t.AutoPopDelay = 5000;
t.InitialDelay = 500;
t.ReshowDelay = 500;
t.ToolTipTitle = Title;
t.RemoveAll();
t.SetToolTip(position, Message);
}
catch (Exception ex)
{
MessageBox.Show(ex.Message.ToString());
}
}
The Issue is that i have to dispose Tooltip. But if i do the Tooltip will not be shown.
May i ask you for help?
Thank you very much.
|
|
|
|
|
You cannot use, display, or even access a Disposed control - Dispose is there to tell the system that the object is done with, finished, caput: once you call it on an object you will get an error (if you are lucky) when you try to use it again.
Once you call Dispose, you will need to recreate a new instance of the ToolTip if you want to show the same information again.
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
It sort of depends on how you're calling "showTooltip".
You can return a reference to "ToolTip" from your "show" routine and dispose of that (at the right time).
It was only in wine that he laid down no limit for himself, but he did not allow himself to be confused by it.
― Confucian Analects: Rules of Confucius about his food
|
|
|
|
|
Hi,
you seem to be creating several tooltips, one for each control that needs a tip.
That is not what WinForms is suggesting.
The more natural way to use ToolTip class is by creating one ToolTip for a Form, then use that ToolTip for each and every Control on that Form that needs a tip. When you are done with the Form, that is the right moment to dispose of the ToolTip. Look at the example code in ToolTip.SetToolTip(Control, String) Method (System.Windows.Forms) | Microsoft Docs[^].
|
|
|
|
|
I have an interface and a type implementing it:
public interface IValueWithUnit {...}
[Serializable]
public struct DoubleValueWithUnit : IValueWithUnit { ... } At a different place, I have a string with the name of the type, and must find out if the type derives from above interface:
string name = "DoubleValueWithUnit";
Type t;
if (TypeHelper.TryFindType(name, out t))
{
m_IsValueWithUnit = t.IsSubclassOf(typeof(IValueWithUnit));
m_IsValueWithUnit = t.IsSubclassOf(typeof(DoubleValueWithUnit));
m_IsValueWithUnit = t.IsInstanceOfType(typeof(IValueWithUnit));
m_IsValueWithUnit = t.IsInstanceOfType(typeof(DoubleValueWithUnit));
m_IsValueWithUnit = t.IsAssignableFrom(typeof(IValueWithUnit));
m_IsValueWithUnit = t.IsAssignableFrom(typeof(DoubleValueWithUnit));
m_IsValueWithUnit = t.GetInterface(nameof(IValueWithUnit)) != null; The TryFindType function is from c# - Type.GetType("namespace.a.b.ClassName") returns null - Stack Overflow[^].
You see the many "false" comments? DoubleValueWithUnit is neither a subclass nor an instance of DoubleValueWithUnit or IValueWithUnit , and is also not assignable from IValueWithUnit , but at least it is assignable from DoubleValueWithUnit .
I am confused now: my brain threw a TypeOverflowException .
Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!
modified 15-Nov-19 6:47am.
|
|
|
|
|
What is the relation between the name variable, and the m_Setting.Type property?
Here's what I can understand from the results you get:
DoubleValueWithUnit is not a subclass of IValueWithUnit . An interface implementation is not considered as inheritance.DoubleValueWithUnit is not a subclass of DoubleValueWithUnit . It is the very same class.DoubleValueWithUnit is not an instance of IValueWithUnit . It is the type itself, not an instance of the type.DoubleValueWithUnit is not an instance of DoubleValueWithUnit . Same comment as above.DoubleValueWithUnit is not assignable from IValueWithUnit . It implements the interface, but that does not mean that every implementation of the interface should allow to cast to actual type. There could be some properties/methods of the class which are not part of the interface.DoubleValueWithUnit is assignable from DoubleValueWithUnit . This one seems logical.DoubleValueWithUnit implements IValueWithUnit . Thus IValueWithUnit is an element of the list of interfaces implemented by DoubleValueWithUnit .
"Five fruits and vegetables a day? What a joke!
Personally, after the third watermelon, I'm full."
|
|
|
|
|
phil.o wrote: What is the relation between the name variable, and the m_Setting.Type property? Oh, I forgot to replace that; question updated.
Thanks for your explanation!
Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!
|
|
|
|
|
|
Thanks. Good to know that.
Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!
|
|
|
|
|
I'm have a listBox of categories that are added through user input via a label and a checkedListBox. I'm trying to get a way to add panels dynamically based on the number of categories in the listBox. I need to use hide/show for each panel since I want the information to switch between each panel.
I have this for adding the new panel dynamically based on each new category in the listBox:
private void button2_Click(object sender, EventArgs e)
{
for(int i = 0; i < checkedListBox1.CheckedItems.Count; i++)
{
if (!listBox1.Items.Contains(checkedListBox1.CheckedItems[i]))
{
listBox1.Items.Add(checkedListBox1.CheckedItems[i]);
newPanel.Name = "panel " + (i+2).ToString();
newPanel.Location = new Point(405, 10);
newPanel.Parent = this;
newPanel.Size = new Size(341, 333);
newPanel.Visible = false;
newPanel.Controls.Add(nameLabel);
newPanel.Controls.Add(testButton);
listPanel.Add(newPanel);
newPanel.Show();
}
}
}
And then for whichever category is clicked on the sub-category it should open the respective panel using this code:
private void listBox1_MouseDoubleClick(object sender, MouseEventArgs e)
{
int index = this.listBox1.IndexFromPoint(e.Location);
if(index != System.Windows.Forms.ListBox.NoMatches)
{
if(index < listPanel.Count - 1)
{
listPanel[index++].Visible = true;
MessageBox.Show("" + listPanel.Count.ToString());
listPanel[--index].Show();
}
}
}
I just either get no respective panel opening or an out-of-range index at listPanel[--index].Show();
Thanks for your help
|
|
|
|
|
Use the debugger and find out what value index holds - I'd suspect that IndexFromPoint is returning ListBox.NoMatches , which is -1.
Since that by definition is less than ListBox.Count, that would explain your problem.
Then all you have to do is find out why - and that's probably down to the Location not being where you expect it to.
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
I will definitely try that, thanks.
|
|
|
|
|
jacko1123 wrote:
listPanel[index++].Visible = true;
listPanel[--index].Show(); I would suggest you remove those inline increment and decrement operators, since the net result is the same as:
listPanel[index].Visible = true;
listPanel[index].Show(); Since you never use the incremented value of index , those operators serve no purpose other than obfuscating your code.
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Right, okay thanks, I'm not exactly very experienced yet so much in programming still learning so all the advice is appreciated thanks, just found a program that I could make that could help me in something in everyday life but it's been giving me a big headache so thought I'd come on a couple of forums for some help. Thanks
|
|
|
|
|
I have more comments to add:
1. assuming nameLabel and testButtons are controls, you are adding them to every Panel you create. However a Control can have only one Parent, so the last addition will win, your nameLabel and testButtons will only belong to and appear on the last Panel you created! If you want them to show on whatever Panel you make visible, you either have to create them for each Panel, or move them around from one Panel to the next that becomes visible. The easy alternative is to put them on your Form itself, not on a switching Panel.
2. your newPanel.Show(); will apply to every Panel you create; it also undoes your newPanel.Visible = false; You should decide how you make one and only one Panel visible: either have them all but one invisible (then don't use Show), or make them all visible and modify their z-order, see e.g. c# - How to set Z-order of a Control using WinForms - Stack Overflow[^] . (I would go for the former). Same applies to your click handler code.
1+2. you need a method that makes a panel visible, by
a) making the previous one (if any) invisible (Visible=false)
b) making the selected one visible, i.e. visible=true, and testButtons.Parent=this;
Your click handler would then call this method, as probably would your Load handler.
3. I would be inclined to create all panels at once (unless there are hundreds of them of course).
4. I see no need to specify a name for all these panels. If you later need to refer to a specific Panel, have a look at the Tag property, where you can store any number, string, Control reference, or whatever fits your needs.
|
|
|
|
|
i want to import database.mdf file in visual studio 2010 project, i done many solutions but its not work, the problem is access denied if copy from MSSQL13.MSSQLSERVER\MSSQL\DATA and paste into my project giving error Access is denied, i already done in SMS RIght Click on databese=>properties=>options=>changing Restrict Access in Single_User than ok, second to datch database, after go go to MSSQL13.MSSQLSERVER\MSSQL\DATA right click on mdf file and giving permission, i done yet, but still thats not work.
|
|
|
|
|
You shouldn't be trying to use a MsSQL database file directly - MsSQL is serving that file to multiple users and you do not have the user permissions on the folder that holds it for good reasons!
You need to work out exactly what you are trying to do: are you trying to use MsSQL in your app - in which case you shouldn't try to touch the file directly; are you trying to use a MDF file in yoru app for a single user - in which case you need to get MsSQL to "release" the file first; are you assuming that you just copy the DB files and MsSQL will "pick it up automatically" on a different computer - in which case you are in for a shock.
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
i do copy paste and add through existing item both not work if i add existing item, its says path\database.mdf can not be opened because it is version 852, this server support version 655 and earlier.
|
|
|
|
|