Re: Azure Active Directory question - C# Discussion Boards

Richard Deeming4-Aug-20 0:01

4-Aug-20 0:01

pkfox wrote:
It must be the UI thread in Winforms

Correct. Your async method tries to return to the current "execution context" after the await. In a WinForms app, when the async method was called from the UI thread, it tries to return to the UI thread. But you've blocked the UI thread by trying to execute the async method synchronously:

this.GetAccessToken().GetAwaiter().GetResult(); // Blocks the UI thread

Therefore, the async method can never complete.

If you're not accessing the UI from your GetAccessToken method, you can add .ConfigureAwait(false) to your ExecuteAsync call, so that the rest of the method can complete on a thread-pool thread:

this.Result = await this.App.AcquireTokenForClient(this.ResourceIds).ExecuteAsync().ConfigureAwait(false);

Otherwise, you'll need to find a way to avoid blocking the UI thread. You'd need to make the calling method async. However, you want to avoid async void methods[^]. A simple solution is to move the code into a task-returning async method, and assign the returned task to a "discard" variable to avoid the compiler warning.

Old code, don't use:

private void ButtonClick(object sender, EventArgs e)
{
    // Some code...
    this.GetAccessToken().GetAwaiter().GetResult();
    // Some more code...
}

Bad fix, don't use:

private async void ButtonClick(object sender, EventArgs e)
{
    // Some code...
    await this.GetAccessToken();
    // Some more code
}

Better fix:

private void ButtonClick(object sender, EventArgs e)
{
    _ = SomeMethodAsync();
}

private async Task SomeMethodAsync()
{
    // Some code...
    await this.GetAccessToken();
    // Some more code
}

ConfigureAwait FAQ | .NET Blog[^]

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

pkfox4-Aug-20 3:36

4-Aug-20 3:36

Thanks Richard should I leave the code as is in GetAccessToken() ? I got around it by using a BackGroundWorker but I think maybe your way is better

"We can't stop here - this is bat country" - Hunter S Thompson - RIP

Richard Deeming4-Aug-20 3:40

4-Aug-20 3:40

If you're moving the code to a task-returning async method, and getting rid of the .GetAwaiter().GetResult() call in favour of awaiting the operation, then the GetAccessToken code should be fine.

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

pkfox4-Aug-20 5:38

4-Aug-20 5:38

How will I know when the job has finished ?

"We can't stop here - this is bat country" - Hunter S Thompson - RIP

Richard Deeming4-Aug-20 5:48

4-Aug-20 5:48

That depends what you're trying to do.

If you just want to run some code immediately after GetAccessToken has finished, you can put it in the new async method after the await this.GetAccessToken(); line.

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

pkfox4-Aug-20 6:52

4-Aug-20 6:52

Except GetAccessToken() is in a separate class from the form

"We can't stop here - this is bat country" - Hunter S Thompson - RIP

Richard Deeming4-Aug-20 23:14

4-Aug-20 23:14

That doesn't matter. You have a task-returning async method which calls it, and put any code you need to run after it's returned after the await.

If you need to run code in the form after the async method in another class returns, you also use a task-returning async method, and put the code after the await.

class SomeClass
{
    private async Task GetAccessToken() { ... }
    
    public async Task DoSomething()
    {
        // Code to run in this class before getting the access token here...
        
        await GetAccessToken();
        
        // Code to run in this class after getting the access token here...
    }
}

class YourForm
{
    private SomeClass _someClass;
    
    private void FormLoad(object sender, EventArgs e)
    {
        _someClass = new SomeClass();
        _ = DoSomethingInTheForm();
    }
    
    private async Task DoSomethingInTheForm()
    {
        // Code to run in the form before getting the access token here...
        
        await _someClass.DoSomething();
        
        // Code to run in the form after getting the access token here...
    }
}

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

pkfox5-Aug-20 3:12

5-Aug-20 3:12

Thanks Richard that's pretty much what I've ended up with - brilliant article in the link you posted

"We can't stop here - this is bat country" - Hunter S Thompson - RIP

pkfox4-Aug-20 23:12

How to change the aspect ratio of an Windows media player by using C#

4-Aug-20 23:12

Hi Richard , thanks for the link lots of good stuff in there - I understand it a lot more now

"We can't stop here - this is bat country" - Hunter S Thompson - RIP

Member 1488657531-Jul-20 0:25

Member 14886575

31-Jul-20 0:25

Watching a Video on windows media player will produce black bars on the left and right.

this.MPlayer.stretchToFit = true;
AxWindowsMediaPlayer mPlayer = this.MPlayer;
mPlayer.URL = "Location of a file";

Below attached file output of the code.
 
Please suggest me how to change the aspect ratio of an Windows media player.

Re: How to change the aspect ratio of an Windows media player by using C#

Gerry Schmitz31-Jul-20 7:32

31-Jul-20 7:32

The "player" itself doesn't resize ... that's what you're expecting. You'd go wonky watching a slideshow.

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

Name Generator

Brandnoob30-Jul-20 8:04

Brandnoob

30-Jul-20 8:04

I am using the VB text editor and want to do some basics in C#. All I want to do is create a list of verbs and have them randomly pair to another list of adjectives to perform a primitive gamertag generator. I am not having a specific problem other than I need an example of this simple function to help me learn and surprisingly I can't find anything maybe because it's too simple...If this is not a good place to learn can someone point me in the right direction because YouTube did not have what I wanted.

Re: Name Generator

Gerry Schmitz30-Jul-20 8:10

ExecuteScalar to cshtml view

30-Jul-20 8:10

You break the project down into tasks. With each task, the problem becomes more transparent.

Your first task would be to compile your word lists. Then, using C#, read them into memory. That's enough to generate more questions once you start.

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

Re: Name Generator

OriginalGriff30-Jul-20 8:55

OriginalGriff

30-Jul-20 8:55

Load your verbs into a collection - an array or list, whichever you want.
Do the same with the adjectives
Then create another collection which takes two indexes into the collections: that stores the "used" tags so you don't repeat. A Dictionary<int, int> should do it. Load it with the gamertags that you've actually issued.
Now, to generate a new tag, pick two indexes at random. Check to see if they are already used together. If they aren't, add them, and you have your tag.
If not, generate two more and try again.

Think of it as if you were doing it with a paper and pencil, and you'll see what I mean.

There are more efficient ways (ensuring you can't generate duplicates instead of checking for them) but they are rather more complicated, particularly for a beginner.

"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!

Member 1490312930-Jul-20 5:30

30-Jul-20 5:30

I am new to .net core and have a question about the functional use of a method that retrieves a single value from a query. Using an MVC pattern, do I need to build a model (entity) for the result?

<pre>public string Line1Today()
        {
            string line1LbsToday = "";
            using (MySqlConnection conn = GetConnection())
            {
                try
                {
                    conn.Open();
                    MySqlCommand cmd = new MySqlCommand("select SUM(value) from tbl1", conn);

                    line1LbsToday = cmd.ExecuteScalar().ToString();
                }
                catch (System.Exception ex)
                {
                    return ex.ToString();
                }
                finally
                {
                    conn.Close();
                }

            }
            return line1LbsToday;

        }

I would like the return value to be sent to a view with some HTML formatting that I have set up. I realize this is quite a trivial question...but I am stuck...i can return whole table results to views, but I am looking for one single scalar value and can't seem to figure out the right approach. Any help would be greatly appreciated.

Gerry Schmitz30-Jul-20 8:02

30-Jul-20 8:02

It's business / data access code; a scalar vs a "table result" with which you are familiar. Think of the scalar as a single-column table result, if that helps.

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

Member 1490312930-Jul-20 11:42

30-Jul-20 11:42

The thing I am not quite wrapping my head around, is typically I have a model framed out to return values in a table...I define the properties etc...those properties I can then walk through with a for loop and @Html.DisplayTextFor() in a table in the view....

With this, I am just looking to return the results of the query. I can't call the function directly from a view..I think my brain is broken at this point...again I know this is a ridiculously simple ask, but for the life of me, I can't google it, can't make it work with existing knowledge, so I am hopefully asking someone for a bit of a kick in the right direction. Do I need a model that defines the result as a property?

Dave Kreskowiak30-Jul-20 12:47

30-Jul-20 12:47

At the top of the Razor code file (*.cshtml), you specify the model, or the type, that the page requires to do it's job, if any data is needed for that page.

When you define a view model to pass to the page, you're just defining a custom type.

So, if all you're doing is passing in an integer to the page, or a string, at the top of the .cshtml page, you just telling to expect that type:

@model int

@model string

Then in your Razor code, you just use Model anywhere you need that value, just like using any other view model class.

To pass that value from your Controller, you just pass whatever that data is, no different than you pass any other object (class instance).

return View(100);

return View("Some string");

return View(myStringVariable)

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

Member 1490312930-Jul-20 12:55

30-Jul-20 12:55

Dave, thanks so much for the detailed response! So if my controller has several methods, all returning somethings to view, how should I approach that? Also, I am using a ViewModel because I have multiple models feeding the page. I also have context classes that I use to feed the data as well...

Dave Kreskowiak30-Jul-20 13:17

30-Jul-20 13:17

A Razor page can be returned by any number of methods in a controller, so long as every one of those methods understands that that page can only accept one particular type.

If you've got multiple methods that are sending data to a view page, each method is going to need it's own Razor page, each defining what type it's looking for to match the data passed to it.

You're not going to get away with a single Razor page handling all kinds of data types.

Now, if you're trying to get multiple controller methods to all send data to the same page being returned, only one method can send data to the Razor page (view).

To get multiple methods to generate data, the method that returns the View is going to have to call the other methods and get the data returned by each. It can then wrap all those pieces of data in a single class you'll have to write, and assemble all that data into a single instance of this container class, passing it to the View.

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

Member 1490312930-Jul-20 13:26

30-Jul-20 13:26

Ok, that makes sense. I come from more of a data science background using more functional programming/scripts, and am trying to implement a semi-functional dashboard of sorts with some signalr components. So if I want to return several boxes that display string results, that is doable, but if I also want some graphical output as well, I am barking up the wrong tree?

Also, I don't suppose the container class return View() would look like return View(context.GetValue1, context.GetValue2, etc...)

Dave Kreskowiak30-Jul-20 17:10

30-Jul-20 17:10

No, you can only send one model to the view. So you have to create a class that holds all of the data you need to send to the view, even if that data comes from separate methods. You then call each of those methods and populate the instance of the container class.

You THEN send that container to the View.

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

Gerry Schmitz31-Jul-20 7:40

31-Jul-20 7:40

If your model is object (sender), object (argument), can you recast them in the view?

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

Dave Kreskowiak31-Jul-20 7:54