print to webpage in C# code

Question

I'm trying to check whether user is adding dupicate IDs in one cohort and if so, I need to print to the webpage a warning: "ID already exists in the cohort" and prevent the user from entering the existing ID again. Now that I did the checking in the if statement but I do not know how to print out to the webpage in the ELSE statement.

Here is the code in the C# file that I'm trying to modify in the Controller:

foreach (string studentID in cc.students)
{
    SqlDataReader rdr = null;
    cmd = new SqlCommand(@"select * from CustomCohortStudent where PUID = @StudentID and cohortID = @cohortID", sqlConn);
    rdr = cmd.ExecuteReader();
    if (rdr == null)
    {
        cmd = new SqlCommand(@"insert into CustomCohortStudents(cohortID, PUID) values (@id,@StudentID)", sqlConn);
    }
    else
    {
        // code to print warning to the webpage 
    }

And I don't know how should I edit the cshtml file in the Views accordingly.. Here is what I had in my index.cshtml file:

<h2>Cohorts</h2>

<a href="CustomCohort/Create">Create</a><br /><br />

<table class="table">
    <tr>
        <!--
        <th>
            @Html.DisplayNameFor(model => model.id)
        </th>
        -->
        <th>
            @Html.DisplayNameFor(model => model.name)
        </th>
        <th>
            <!--Actions-->
        </th>
    </tr>

    @foreach (var item in Model)
    {
        <tr>
            <!--
            <td>
                @item.id
            </td>
            -->
            <td>
                @item.name
            </td>
            <td>
                <a href="CustomCohort/Edit/@item.id">Edit</a> | 
                <a href="CustomCohort/Delete/@item.id">Delete</a>
            </td>
        </tr>
    }
</table>

So my question is: What should I add in the ELSE statement in the Controller and what to modify in the Views?

Gerino · Accepted Answer · 2017-04-07 14:49:48Z

1

If the warning is non-fatal, i.e. the flow should continue like with everything going smoothly, then you need to return more than one information from your code that does the insert: did it succeed and what, if any, warnings are.

That means that you'll be probably better off with some kind of Result class:

Now, there are many ways to do this, this is a quickly written one that hopefully will give you an idea about what I'm talking about. Note, that you can skip the Result class and go straight for returning List<string>, but it hides your actual intention from whoever is going to read this code in the future (you included). Returning a class with a descriptive name and properties means it's super-easy to understand the idea behind the code with just skimming over it.

The Result class is maximally simplified, if you want you can use it to transfer other information (how many succeeded, or which one failed etc., to for example display that information in your View).

public class Result{
    public List<string> Warnings {get;} = new List<string>();
}

Then in your method that does the inserts:

public Result MyInsertMethod(...){
    var result = new Result();
    foreach (string studentID in cc.students)
    {
        SqlDataReader rdr = null;
        cmd = new SqlCommand(@"select * from CustomCohortStudent where PUID = @StudentID and cohortID = @cohortID", sqlConn);
        rdr = cmd.ExecuteReader();
        if (rdr == null)
        {
            cmd = new SqlCommand(@"insert into CustomCohortStudents(cohortID, PUID) values (@id,@StudentID)", sqlConn);
        }
        else
        {
            // code to print warning to the webpage 
            result.Warnings.Add("Something was not awesome");
        }
    }
    return result;
}

In Controller:

ViewBag.Result = MyInsertMethod(...);

In the view:

<h2>Cohorts</h2>

if(ViewBag.Result != null){
    foreach(var warn in ((Result)ViewBag.Result).Warnings){
        <span class="warning">@warn</span>
    }
}

<a href="CustomCohort/Create">Create</a><br /><br />

answered Apr 7, 2017 at 14:49

Gerino

1,9631 gold badge16 silver badges22 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Sophie Over a year ago

Thank you! If my method is : public ActionResult Create(CustomCohort cc). Does it work the same as Result?

Sophie Over a year ago

And where should I put this line: "ViewBag.Result = MyInsertMethod() " in the controller? Since the "MyInsertMethod()" is already in the controller...Thank you!!!

Gerino Over a year ago

The ViewBag.Whatever goes into the action (Create(CustomCohort cc)) in your example. The actual inserts would be best in their own separate method, as having everything in the Controller Action, both SQL queries, data processing and setting up the View makes it confusing ;)

Gerino Over a year ago

You can however do it, then just instead of doing return result; do ViewBag.Result = result;.

Collectives™ on Stack Overflow

print to webpage in C# code

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related