Member 4773987 Ответов: 2

Как вернуть всю строку из API

Ниже приведен код для получения данных через API, как вернуть все строки, а не только последнюю строку :

Что я уже пробовал:

У меня есть приведенный ниже API, который возвращает только последние записи в таблице, как позволить ему возвращать несколько записей в зависимости от операторов select: 
[HttpGet]
[ActionName("GetReservationByID")]
public Reservation Get(int id)
{
string source = System.Configuration.ConfigurationManager.ConnectionStrings["DefaultConnection"].ConnectionString;
SqlConnection conn = new SqlConnection(source);
SqlCommand cmd = new SqlCommand();
SqlDataReader reader;
string sql = "Select s.Id,a.Room_No ,s.Room_Description,s.Room_Capacity, b.Description, s.Location,s.Start_Date,s.Start_Time,s.End_Time,s.Meeting_Title,s.Reservation_Reason, s.Status,s.Attendance,s.Remarks,s.Mail_To,s.Mail_CC from Reservation s , Rooms a, Room_Type b Where s.Room_No = a.ID And s.Room_Type = b.ID AND s.Id >" + id + "";
cmd.CommandText = sql;
cmd.CommandType = CommandType.Text;
cmd.Connection = conn;
Reservation emp = null;
conn.Open();
reader = cmd.ExecuteReader();
while (reader.Read())
{
//read data
emp = new Reservation();
emp.Id = Convert.ToInt32(reader.GetValue(0));
emp.Room_No = reader.GetValue(1).ToString();
emp.Room_Description = reader.GetValue(2) as string;//reader.GetValue(3).ToString();
emp.Room_Capacity = (reader.GetValue(3) as int?) ?? 0; //Convert.ToInt32(reader.GetValue(4));
emp.Room_Type = reader.GetValue(4) as string;//reader.GetValue(5).ToString();
emp.Location = reader.GetValue(5) as string; //reader.GetValue(6).ToString();
emp.Start_Date = reader.GetValue(6) as string; //reader.GetValue(7).ToString();
emp.Start_Time = reader.GetValue(7) as string; //reader.GetValue(8).ToString();
emp.End_Time = reader.GetValue(8) as string; //reader.GetValue(9).ToString();
emp.Meeting_Title = reader.GetValue(9) as string; //reader.GetValue(10).ToString();
emp.Reservation_Reason = reader.GetValue(10) as string; //reader.GetValue(11).ToString();
emp.Status = reader.GetValue(11) as string; //reader.GetValue(12).ToString();
emp.Attendance = (reader.GetValue(12) as int?) ?? 0;//Convert.ToInt32(reader.GetValue(13));
emp.Remarks = reader.GetValue(13) as string;//reader.GetValue(14).ToString();
emp.Mail_To = reader.GetValue(14) as string;//reader.GetValue(15).ToString();
emp.Mail_CC = reader.GetValue(15) as string;//reader.GetValue(16).ToString();
}
conn.Close();
return emp;
}

RickZeeland

Похоже, ваш запрос смотрит на id: и s.Id & gt; id, может ли это быть проблемой ?

Afzaal Ahmad Zeeshan

Хотя я предоставил код для этого решения, но дело в том, что вам нужно пересмотреть свой подход к построению веб-API, элемент, найденный в "id", всегда должен быть единственным скалярным значением и не должен быть списком или контейнером.

См. Решение 1.

2 Ответов

Рейтинг:
14

Afzaal Ahmad Zeeshan

Взгляните на свой код, 

while (reader.Read())
{
//read data
emp = new Reservation();
emp.Id = Convert.ToInt32(reader.GetValue(0));
emp.Room_No = reader.GetValue(1).ToString();
emp.Room_Description = reader.GetValue(2) as string;//reader.GetValue(3).ToString();
emp.Room_Capacity = (reader.GetValue(3) as int?) ?? 0; //Convert.ToInt32(reader.GetValue(4));
emp.Room_Type = reader.GetValue(4) as string;//reader.GetValue(5).ToString();
emp.Location = reader.GetValue(5) as string; //reader.GetValue(6).ToString();
emp.Start_Date = reader.GetValue(6) as string; //reader.GetValue(7).ToString();
emp.Start_Time = reader.GetValue(7) as string; //reader.GetValue(8).ToString();
emp.End_Time = reader.GetValue(8) as string; //reader.GetValue(9).ToString();
emp.Meeting_Title = reader.GetValue(9) as string; //reader.GetValue(10).ToString();
emp.Reservation_Reason = reader.GetValue(10) as string; //reader.GetValue(11).ToString();
emp.Status = reader.GetValue(11) as string; //reader.GetValue(12).ToString();
emp.Attendance = (reader.GetValue(12) as int?) ?? 0;//Convert.ToInt32(reader.GetValue(13));
emp.Remarks = reader.GetValue(13) as string;//reader.GetValue(14).ToString();
emp.Mail_To = reader.GetValue(14) as string;//reader.GetValue(15).ToString();
emp.Mail_CC = reader.GetValue(15) as string;//reader.GetValue(16).ToString();
}
conn.Close();
return emp;

Вы возвращаете объект" emp", и в каждой итерации записи вы создаете новый его экземпляр и устанавливаете его значение на текущую запись, которая к концу итерации является последней записью. Вот почему ваш API возвращает только последний.

Вместо этого я бы рекомендовал вам добавить общий тип списка и сохранить объекты emp в этом списке. Что-то вроде этого, 
// Somewhere before list
List<Reservation> emps = new List<Reservation>();

while (reader.Read()) {
   emp = new Reservation();

   // Other code in the loop
   // Last line
   emps.Add(emp);
}

// Return emps, not emp
return emps;

Чтобы вернуть "emps", вам также нужно будет изменить подпись, 
public List<Reservation> Get(int id) {

После всего этого вы должны изучить взлом SQL-инъекции, с помощью которого ваш код может быть легко атакован. Никогда не объединяйте SQL-запросы, всегда старайтесь параметризовать их, чтобы сохранить себя.

SQL-инъекция-Википедия[^]

Рейтинг:
0

Member 4773987

Большое спасибо, решено, но что касается SQL-инъекции, то мне нужно получить данные из SQL DB, и это единственный известный мне способ получить данные. Очень ценю вашу помощь.

Спасибо
Икрами

Richard Deeming

1) Если вы хотите ответить на решение, используйте команду "есть вопрос или комментарий?" кнопка под этим решением. НЕ опубликуйте свой комментарий как новое решение.

2) вы избегаете SQL-инъекции, используя правильно параметризованный запрос:

const string query = "SELECT s.Id, a.Room_No, s.Room_Description, s.Room_Capacity, b.Description, s.Location, s.Start_Date, s.Start_Time, s.End_Time, s.Meeting_Title, s.Reservation_Reason, s.Status, s.Attendance, s.Remarks, s.Mail_To, s.Mail_CC FROM Reservation s INNER JOIN Rooms a ON a.ID = s.Room_No INNER JOIN Room_Type ON b.ID = s.Room_Type Where s.Id > @id";

using (SqlConnection conn = new SqlConnection(source))
using (SqlCommand cmd = new SqlCommand(query, conn))
{
    cmd.Parameters.AddWithValue("@id", id);
    
    conn.Open();
    using (SqlDataReader reader = cmd.ExecuteReader(CommandBehavior.CloseConnection))
    {
        while (reader.Read())
        {
            ...
        }
    }
}

Richard Deeming

Все, что вы хотели знать о SQL-инъекции (но боялись спросить) | Трой Хант[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? | Обмен Стеками Информационной Безопасности[^]
Шпаргалка по параметризации запросов / OWASP[^]