Не все ветви кода возвращают значение

@Alb_Sap · Регистрация: 08.02.2013

Студворк — интернет-сервис помощи студентам

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 [WebMethod]
        public String GetDataAvg(int [] detectors, DateTime [,] dates, out String readings)
        {
            SqlConnection MyConnection = new SqlConnection("Server=VyazServ;database=Microfor;uid=microfor;pwd=sonicview");
            SqlDataAdapter myData = new SqlDataAdapter("GetAvgNext", MyConnection);
            myData.SelectCommand.CommandType = CommandType.StoredProcedure;
            for (int k = 0; k < detectors.GetLength(0); ++k)
            {
                for (int i = 0; i < dates.GetLength(0); ++i)
                    for (int j = 0; j < dates.GetLength(1); ++j)
                    {
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@detector_id", SqlDbType.Int, 0, "detector_id"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@begin_date", SqlDbType.DateTime, 0, "begin_date"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@end_date", SqlDbType.DateTime, 0, "end_date"));
                        myData.SelectCommand.Parameters[0].Value = detectors[k];
                        myData.SelectCommand.Parameters[1].Value = dates[i, 0];
                        myData.SelectCommand.Parameters[2].Value = dates[i, 1];
 
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        readings = ds.GetXml() +"," + k.ToString() + ";";
                        return readings;
                    }
             }
         }

помогите разобраться ,в чем ошибка? гляньте свежим взглядом?)

@Abrakadabra_13 · 08.02.2013, 10:31

КАк я понимаю, программа может не зайти в эти циклы, тогда значение не вернется. поэтому ретурн возвращай после циклов.

@Spectral-Owl · 08.02.2013, 11:43

Сообщение от Alb_Sap

return readings;

первое:
у тебя этот метод вернёт 1 значение для k=0,i=0,j=0, а дальше смотреть даже не будет.
это конечно при условии что detectors.GetLength(0),dates.GetLength(0 ),dates.GetLength(1) больше нуля.
второе:
Я, конечно, знаю толк в извращениях, но возвращать методом то, что в него было передано с параметром out - даже для меня немножко жутко.

по этому делай так как сказал Abrakadabra_13: внутри циклов формируй значение, после циклов возвращай.

@mrBlade · 08.02.2013, 11:48

Можно делать так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 [WebMethod]
        public String GetDataAvg(int [] detectors, DateTime [,] dates, out String readings)
        {
            readings = ""; // или readings = null
            SqlConnection MyConnection = new SqlConnection("Server=VyazServ;database=Microfor;uid=microfor;pwd=sonicview");
            SqlDataAdapter myData = new SqlDataAdapter("GetAvgNext", MyConnection);
            myData.SelectCommand.CommandType = CommandType.StoredProcedure;
            for (int k = 0; k < detectors.GetLength(0); ++k)
            {
                for (int i = 0; i < dates.GetLength(0); ++i)
                    for (int j = 0; j < dates.GetLength(1); ++j)
                    {
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@detector_id", SqlDbType.Int, 0, "detector_id"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@begin_date", SqlDbType.DateTime, 0, "begin_date"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@end_date", SqlDbType.DateTime, 0, "end_date"));
                        myData.SelectCommand.Parameters[0].Value = detectors[k];
                        myData.SelectCommand.Parameters[1].Value = dates[i, 0];
                        myData.SelectCommand.Parameters[2].Value = dates[i, 1];
 
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        readings = ds.GetXml() +"," + k.ToString() + ";";
                    }
             }
             return readings;
         }

Но не ясно, зачем readings передаётся по ссылке (через out) и он же возвращается.

@Alb_Sap · 08.02.2013, 12:32 **[ТС]**

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
  [WebMethod]
        public String GetDataAvg(int [] detectors, DateTime [,] dates)
        {
            SqlConnection MyConnection = new SqlConnection("Server=VyazServ;database=Microfor;uid=microfor;pwd=sonicview");
            SqlDataAdapter myData = new SqlDataAdapter("GetAvgNext", MyConnection);
            myData.SelectCommand.CommandType = CommandType.StoredProcedure;
            
            string result;
            string readings;
            for (int k = 0; k < detectors.GetLength(0); ++k)
            {
                result = "";
                for (int i = 0; i < dates.GetLength(0); ++i)
                    for (int j = 0; j < dates.GetLength(1); ++j)
                        readings = "";
                    {
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@detector_id", SqlDbType.Int, 0, "detector_id"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@begin_date", SqlDbType.DateTime, 0, "begin_date"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@end_date", SqlDbType.DateTime, 0, "end_date"));
                        myData.SelectCommand.Parameters[0].Value = detectors[k];
                        myData.SelectCommand.Parameters[1].Value = dates[i, 0];
                        myData.SelectCommand.Parameters[2].Value = dates[i, 1];
 
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        readings = ds.GetXml() +"," + k.ToString() + ";" + readings;
                     }
                result = readings + result;
             }
            return result;
         }

как то так вероятно, код не оптимальный конечно

Добавлено через 5 минут
спасибо за замечания, очень полезно.

Добавлено через 9 минут
виноват, вот так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
            [WebMethod]
        public String GetDataAvg(int [] detectors, DateTime [,] dates)
        {
            SqlConnection MyConnection = new SqlConnection("Server=VyazServ;database=Microfor;uid=microfor;pwd=sonicview");
            SqlDataAdapter myData = new SqlDataAdapter("GetAvgNext", MyConnection);
            myData.SelectCommand.CommandType = CommandType.StoredProcedure;
            string result;
            string readings;
            result = "";
            for (int k = 0; k < detectors.GetLength(0); ++k)
            {
                readings = "";
                for (int i = 0; i < dates.GetLength(0); ++i)
                    for (int j = 0; j < dates.GetLength(1); ++j)
                      {
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@detector_id", SqlDbType.Int, 0, "detector_id"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@begin_date", SqlDbType.DateTime, 0, "begin_date"));
                        myData.SelectCommand.Parameters.Add(new SqlParameter("@end_date", SqlDbType.DateTime, 0, "end_date"));
                        myData.SelectCommand.Parameters[0].Value = detectors[k];
                        myData.SelectCommand.Parameters[1].Value = dates[i, 0];
                        myData.SelectCommand.Parameters[2].Value = dates[i, 1];
 
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        readings = ds.GetXml() +"," + k.ToString() + ";" + readings;
                     }
                result = readings + result;
             }
            return result;
         }

@mrBlade · 08.02.2013, 12:33

Сообщение от Alb_Sap

код не оптимальный конечно

И будет расходовать много памяти, особенно при длинных циклах, т.к. не очищаются параметры у myData. Вообще лучше взять за правило: после выполнения запроса через SqlCommand или SqlDataAdapter нужно всегда очищать параметры (если они, конечно, есть). В Вашем случае нужно добавить

C#
1
2
3
4
5
6
                        // ...
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        myData.Parameters.Clear();
                        readings = ds.GetXml() +"," + k.ToString() + ";" + readings;
                        //...

Это избавит от потенциальных проблем. А вообще лучше вынести добавление параметров вне цикла, а в цикле только присваивать им значения. Так будет значительно лучше:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
  [WebMethod]
        public String GetDataAvg(int [] detectors, DateTime [,] dates)
        {
            SqlConnection MyConnection = new SqlConnection("Server=VyazServ;database=Microfor;uid=microfor;pwd=sonicview");
            SqlDataAdapter myData = new SqlDataAdapter("GetAvgNext", MyConnection);
            myData.SelectCommand.CommandType = CommandType.StoredProcedure;
            myData.SelectCommand.Parameters.Add(new SqlParameter("@detector_id", SqlDbType.Int, 0, "detector_id"));
            myData.SelectCommand.Parameters.Add(new SqlParameter("@begin_date", SqlDbType.DateTime, 0, "begin_date"));
            myData.SelectCommand.Parameters.Add(new SqlParameter("@end_date", SqlDbType.DateTime, 0, "end_date"));
            
            string result;
            string readings;
            for (int k = 0; k < detectors.GetLength(0); ++k)
            {
                result = "";
                for (int i = 0; i < dates.GetLength(0); ++i)
                    for (int j = 0; j < dates.GetLength(1); ++j)
                        readings = "";
                    {
                        myData.SelectCommand.Parameters[0].Value = detectors[k];
                        myData.SelectCommand.Parameters[1].Value = dates[i, 0];
                        myData.SelectCommand.Parameters[2].Value = dates[i, 1];
 
                        DataSet ds = new DataSet();
                        myData.Fill(ds);
                        readings = ds.GetXml() +"," + k.ToString() + ";" + readings;
                     }
                result = readings + result;
             }
            myData.Parameters.Clear();
            return result;
         }

Новые блоги и статьи Все статьи Все блоги /
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .

@Abrakadabra_13 7 / 7 / 1 Регистрация: 23.01.2013 Сообщений: 43
	08.02.2013, 10:31
	КАк я понимаю, программа может не зайти в эти циклы, тогда значение не вернется. поэтому ретурн возвращай после циклов. 1