How to pass multiple parameters to DELETE request C# Rest API

Question

I need to pass 3 parameters to my API DELETE request. Here is my code what I have try.

TaskModel

public class TaskModel
{
    public int DeveloperID { get; set; }
    public int ProjectID { get; set; }
    public string WorkDate { get; set; }
}

This a controller class. called TaskController

    [Route("api/Task")]
    public void Delete(TaskModel value)
    {
        TaskPersistent tp = new TaskPersistent();
        tp.deleteTask(value);
    }

This is TaskPersistent.class

public void deleteTask(TaskModel task)
{
    try
    {
        string sqlString = "DELETE from devproj WHERE (DeveloperID, ProjectID, WorkDate) =  VALUES ('" + task.DeveloperID + "', '" + task.ProjectID + "', '" + task.WorkDate + "')"; // System.NullReferenceException throw
        MySql.Data.MySqlClient.MySqlCommand cmd = new MySql.Data.MySqlClient.MySqlCommand(sqlString, conn);
        cmd.ExecuteNonQuery();
        long x = cmd.LastInsertedId;
    }
    catch (MySqlException x)
    {
        int errr = x.Number;

        Console.WriteLine(errr);
    }
}

I consume this API using ARC rest client like this,http://localhost:2731/api/Task?DeveloperID=1&ProjectID=2&WorkDate="2018-03-14" But when I pass the parameters like this,API thrown exception: 'System.NullReferenceException' in DeleteAPI.dll (I commented error occurred line in my code). What I did wrong here.

Possible duplicate of What is a NullReferenceException, and how do I fix it? — Camilo Terevinto
– Camilo Terevinto, Commented Mar 14, 2018 at 10:38
You need to consume this API by posting your TaskModel as a body for the model binder to work with the way you have your Delete controller action set up. Alternatively, change the Delete method parameters to int developerId, int projectId, DateTime workDate — Vidmantas Blazevicius
– Vidmantas Blazevicius, Commented Mar 14, 2018 at 10:39
@VidmantasBlazevicius Can you give me it as a answer.please. — Gamma
– Gamma, Commented Mar 14, 2018 at 10:40
If your API method is for a delete action, a good practice is to add [HttpDelete] as attribute and send DELETE instead GET. — User.Anonymous
– User.Anonymous, Commented Mar 14, 2018 at 10:45
Yes, between [Route] attribute and method declaration [Route("api/Task")] [HttpDelete] public void Delete(TaskModel value) and so you can call with DELETE verbs instead of GET. — User.Anonymous
– User.Anonymous, Commented Mar 14, 2018 at 10:55

Vidmantas Blazevicius · Accepted Answer · 2018-03-14 10:43:04Z

2

You need to consume this API by posting your TaskModel as a body for the model binder to work with the way you have your Delete controller action set up. Alternatively, change the Delete method parameters to int developerId, int projectId, DateTime workDate.

    [Route("api/Task")]
    public void Delete(int developerId, int projectId, DateTime workDate)
    {
        var taskModel = new TaskModel
        {
            DeveloperId = developerId,
            ProjectID = projectId,
            WorkDate = workDate
        };
        TaskPersistent tp = new TaskPersistent();
        tp.deleteTask(taskModel);
    }

edited Mar 14, 2018 at 10:43

answered Mar 14, 2018 at 10:41

Vidmantas Blazevicius

4,8322 gold badges14 silver badges32 bronze badges

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

5 Comments

Vidmantas Blazevicius Over a year ago

I gave a sample code of how to set up the controller to work with the way you are consuming it currently

Gamma Over a year ago

Sir I try this, But when I try this,this exception occurred now, 'MySql.Data.MySqlClient.MySqlException' in MySql.Data.dll

Vidmantas Blazevicius Over a year ago

@Gamma What is the exception you are getting and what is the runtime value now of sqlString variable? Please be as thorough as you can so we can help you properly.

Gamma Over a year ago

Problem solved sir, I edited my query as User.Anonymous's answer. Thank you very much

Vidmantas Blazevicius Over a year ago

@Gamma no problems, I am happy you have solved your issue

User.Anonymous · Accepted Answer · 2018-03-14 10:59:20Z

string sqlString = "DELETE from devproj WHERE (DeveloperID, ProjectID, WorkDate) =  VALUES ('" + task.DeveloperID + "', '" + task.ProjectID + "', '" + task.WorkDate + "')"; // System.NullReferenceException throw

This query don't work. SQL delete is :

string sqlString = $"DELETE from devproj WHERE DeveloperID = {task.DeveloperID} AND ProjectID = {task.ProjectID} AND WorkDate = {task.WorkDate}";

And don't forget to check for null value in your model before execute query.

Collectives™ on Stack Overflow

How to pass multiple parameters to DELETE request C# Rest API

2 Answers 2

5 Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Linked

Related