Changeset 253

Timestamp:

06/30/09 22:52:28 (3 years ago)

Author:

antonbatenev.ya.ru

Message:

Исправление поведения при ошибке парсинга ответа синхронизации (см.  http://www.rsdn.ru/forum/janus/3449147.1.aspx)

Files:

• tags/1.0b/form_main.cpp (modified) (1 diff)
• tags/1.0b/version.h (modified) (1 diff)
• tags/1.0b/webservice.cpp (modified) (3 diffs)
• tags/1.0b/webservice.h (modified) (1 diff)
• trunk/form_main.cpp (modified) (1 diff)
• trunk/version.h (modified) (1 diff)
• trunk/webservice.cpp (modified) (3 diffs)
• trunk/webservice.h (modified) (1 diff)

tags/1.0b/form_main.cpp

@@ -338 +338 @@
                         AUserInfoList list;
 
-                        AWebservice::getUserList_WebserviceParse(answer, list, row_version.User, NULL);
+                        QString result = AWebservice::getUserList_WebserviceParse(answer, list, row_version.User, NULL);
+
+                        if (result.length() > 0)
+                        {
+                                QMessageBox::critical(this, QString::fromUtf8("Ошибка!"), result);
+                                return;
+                        }
 
                         // сохранение ответа

tags/1.0b/version.h

@@ -20 +20 @@
  * \brief Дата билда (заменяется автоматически при каждом билде в version.h, что и приводит к смене номера ревизии)
  */
-#define AVALON_DATE "Пнд Апр 27 05:20:09 MSD 2009"
+#define AVALON_DATE "Втр Июн 30 22:37:55 MSD 2009"
 
 #endif

tags/1.0b/webservice.cpp

@@ -245 +245 @@
 //----------------------------------------------------------------------------------------------
 
-void AWebservice::getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress)
+QString AWebservice::getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress)
 {
         if (progress != NULL)
@@ -280 +280 @@
         row_version = getNextBlock(&data, "<lastRowVersion>", "</lastRowVersion>", seed);
 
+        // проверка ошибок получения версий строк
+        // подавлять ошибки сбросом в значение по умолчанию ("AAAAAAAAAAA=") оказалось неправильно
+        // подробнее см. http://www.rsdn.ru/forum/janus/3449147.1.aspx
         if (row_version.length() == 0)
-                row_version = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastRowVersion>");
+
+        return "";
 }
 //----------------------------------------------------------------------------------------------
@@ -473 +478 @@
         row_version.Moderate = getNextBlock(&data, "<lastModerateRowVersion>", "</lastModerateRowVersion>", seed);
 
+        // проверка ошибок получения версий строк
+        // подавлять ошибки сбросом в значение по умолчанию ("AAAAAAAAAAA=") оказалось неправильно
+        // подробнее см. http://www.rsdn.ru/forum/janus/3449147.1.aspx
         if (row_version.Rating.length() == 0)
-                row_version.Rating = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastRatingRowVersion>");
         if (row_version.Message.length() == 0)
-                row_version.Message = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastForumRowVersion>");
         if (row_version.Moderate.length() == 0)
-                row_version.Moderate = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastModerateRowVersion>");
 
         return "";

tags/1.0b/webservice.h

@@ -53 +53 @@
                  * \param row_version Версия строк (снимка)
                  * \param progress Прогресс
+                 * \return Пустая строка, или сообщение об ошибке
                  */
-                static void getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress = NULL);
+                static QString getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress = NULL);
 
                 /*!

trunk/form_main.cpp

@@ -363 +363 @@
                         AUserInfoList list;
 
-                        AWebservice::getUserList_WebserviceParse(answer, list, row_version.User, NULL);
+                        QString result = AWebservice::getUserList_WebserviceParse(answer, list, row_version.User, NULL);
+
+                        if (result.length() > 0)
+                        {
+                                QMessageBox::critical(this, QString::fromUtf8("Ошибка!"), result);
+                                return;
+                        }
 
                         // сохранение ответа

trunk/version.h

@@ -20 +20 @@
  * \brief Дата билда (заменяется автоматически при каждом билде в version.h, что и приводит к смене номера ревизии)
  */
-#define AVALON_DATE "Пнд Июн 29 10:39:07 MSD 2009"
+#define AVALON_DATE "Втр Июн 30 22:47:08 MSD 2009"
 
 #endif

trunk/webservice.cpp

@@ -269 +269 @@
 //----------------------------------------------------------------------------------------------
 
-void AWebservice::getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress)
+QString AWebservice::getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress)
 {
         if (progress != NULL)
@@ -304 +304 @@
         row_version = getNextBlock(&data, "<lastRowVersion>", "</lastRowVersion>", seed);
 
+        // проверка ошибок получения версий строк
+        // подавлять ошибки сбросом в значение по умолчанию ("AAAAAAAAAAA=") оказалось неправильно
+        // подробнее см. http://www.rsdn.ru/forum/janus/3449147.1.aspx
         if (row_version.length() == 0)
-                row_version = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastRowVersion>");
+
+        return "";
 }
 //----------------------------------------------------------------------------------------------
@@ -501 +506 @@
         row_version.Moderate = getNextBlock(&data, "<lastModerateRowVersion>", "</lastModerateRowVersion>", seed);
 
+        // проверка ошибок получения версий строк
+        // подавлять ошибки сбросом в значение по умолчанию ("AAAAAAAAAAA=") оказалось неправильно
+        // подробнее см. http://www.rsdn.ru/forum/janus/3449147.1.aspx
         if (row_version.Rating.length() == 0)
-                row_version.Rating = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastRatingRowVersion>");
         if (row_version.Message.length() == 0)
-                row_version.Message = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastForumRowVersion>");
         if (row_version.Moderate.length() == 0)
-                row_version.Moderate = "AAAAAAAAAAA=";
+                return QString::fromUtf8("В ответе вебсервиса не найдено поле <lastModerateRowVersion>");
 
         return "";

trunk/webservice.h

@@ -54 +54 @@
                  * \param progress Прогресс
                  */
-                static void getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress = NULL);
+                static QString getUserList_WebserviceParse (const QString& data, AUserInfoList& list, QString& row_version, IProgress* progress = NULL);
 
                 /*!