SASGIS

Веб-картография и навигация
All Projects | SAS.Планета | Домен, сайт, форум, багтрекер | Доработка карты (ZMP) | Переводы и локализации | Прочее

My View | View Issues | Change Log | Roadmap | Search
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001334SAS.ПланетаРефакторингpublic13-06-2012 19:4210-10-2012 11:44
Reporterzed 
Assigned ToGarl 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformWindowsOSXPOS VersionSP3
Product Version.Nightly 
Target Version120808Fixed in Version120808 
Summary0001334: Варнинг в TGeoCoderByURL.ParseResultToPlacemarksList
DescriptionВ той функции сам чёрт ногу сломит, так ещё и варнинг вылез и глаза мозолит.

[DCC Warning] u_GeoCoderByURL.pas(927): W1035 Return value of function 'TGeoCoderByURL.ParseResultToPlacemarksList' might be undefined
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0007387)
Garl (manager)
13-06-2012 20:13

 самое интересное что при повторной компиляции варнинг пропадает :)
и как его ловить?
(0007388)
Garl (manager)
13-06-2012 20:20

 может 1200 строк сильно много?
завтра попробую разбить на 2 отдельных геокодера:
отдельно URL + отдельно координаты
(0007391)
zed (manager)
14-06-2012 16:17

 >самое интересное что при повторной компиляции варнинг пропадает
У меня ничего никуда не пропадает.

>может 1200 строк сильно много?
Нет. Что-то не в порядке именно в той функции.
(0007392)
Garl (manager)
14-06-2012 16:35

 переделываю чуток, ещё в процессе.
(0007393)
Garl (manager)
14-06-2012 20:21

 сейчас почистил \.dcu , и вылезли ещё варнинги

ImagingFormats строка 1639

[DCC Warning] ImagingFormats.pas(1639): W1035 Return value of function 'BuildMappingTable' might be undefined
такая же петрушка с Result := nil;
(0007394)
zed (manager)
14-06-2012 20:45

 >сейчас почистил \.dcu
Аналогичный эффект будет, если ты будешь билдить проект (Build Project), а не компилить (Compile Project).

>и вылезли ещё варнинги
Это совершенно независимый варнинг от стороннего компонента. Там ещё и от GR32 есть варнинги. Повылазили после недавнего обновления компонентов.
(0007395)
Garl (manager)
14-06-2012 20:49

 if <1> then begin
end else
if <2> then begin
end else
if <3> then begin
end else
if <4> then begin
end else
begin
end

непонятно почему но при коментраии некоторых условий варнинг пропадает!
в частности
// if (PosEx('maps.yandex.ru/?oid=', Vlink, 1) > 0 )then begin
// Vlink := ReplaceStr(Vstr,'''','');
// sname := 'yandex';
// i := PosEx('{ll:', Vlink, 1);
// if i=0 then i := PosEx(',ll:', Vlink, 1);
// j := PosEx(',', Vlink, i+1);
// slon := Copy(Vlink, i + 4, j - (i + 4));
// i := j;
// j := PosEx(',', Vlink, i+1);
// slat := Copy(Vlink, i + 1, j - (i + 1));
// sdesc := '[ '+slon+' , '+slat+' ]';
// end else
(0007396)
Garl (manager)
14-06-2012 21:17

 всему виной строка
Vlink := ReplaceStr(Vstr,'''','');
без неё варнинга нету :)
(0007397)
zed (manager)
14-06-2012 21:18

 Попробуй вычленить каждый глобальный if else в отдельную функцию. А ещё лучше сделать одну функцию парсера (на основе регэксов), которой на вход подавать регулярное выражение (шаблон), по которому и проверять полученный результат до первого совпадения.

- Users who viewed this issue
User List Anonymous (1915x)
Total Views 1915
Last View 19-04-2024 06:20

- Issue History
Date Modified Username Field Change
13-06-2012 19:42 zed New Issue
13-06-2012 20:13 Garl Note Added: 0007387
13-06-2012 20:20 Garl Note Added: 0007388
13-06-2012 20:20 Garl Assigned To => Garl
13-06-2012 20:20 Garl Status new => assigned
14-06-2012 05:36 vdemidov Target Version => 120808
14-06-2012 16:17 zed Note Added: 0007391
14-06-2012 16:35 Garl Note Added: 0007392
14-06-2012 20:21 Garl Note Added: 0007393
14-06-2012 20:45 zed Note Added: 0007394
14-06-2012 20:49 Garl Note Added: 0007395
14-06-2012 21:17 Garl Note Added: 0007396
14-06-2012 21:18 zed Note Added: 0007397
17-06-2012 10:40 Garl Status assigned => resolved
17-06-2012 10:40 Garl Resolution open => fixed
05-07-2012 05:49 vdemidov Fixed in Version => 120808
10-10-2012 11:44 Tolik Status resolved => closed

My View | View Issues | Change Log | Roadmap | Search
  

Copyright © 2007 - 2024 SAS.Planet Team