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

View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001334SAS.ПланетаРефакторинг / Refactoringpublic13-06-2012 19:4210-10-2012 11:44
Reporterzed Assigned ToGarl  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
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.

Activities

Garl

13-06-2012 20:13

manager   ~0007387

самое интересное что при повторной компиляции варнинг пропадает :)
и как его ловить?

Garl

13-06-2012 20:20

manager   ~0007388

может 1200 строк сильно много?
завтра попробую разбить на 2 отдельных геокодера:
отдельно URL + отдельно координаты

zed

14-06-2012 16:17

manager   ~0007391

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

>может 1200 строк сильно много?
Нет. Что-то не в порядке именно в той функции.

Garl

14-06-2012 16:35

manager   ~0007392

переделываю чуток, ещё в процессе.

Garl

14-06-2012 20:21

manager   ~0007393

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

ImagingFormats строка 1639

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

zed

14-06-2012 20:45

manager   ~0007394

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

>и вылезли ещё варнинги
Это совершенно независимый варнинг от стороннего компонента. Там ещё и от GR32 есть варнинги. Повылазили после недавнего обновления компонентов.

Garl

14-06-2012 20:49

manager   ~0007395

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

Garl

14-06-2012 21:17

manager   ~0007396

всему виной строка
Vlink := ReplaceStr(Vstr,'''','');
без неё варнинга нету :)

zed

14-06-2012 21:18

manager   ~0007397

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

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
08-08-2025 13:25 zed Category Рефакторинг => Рефакторинг / Refactoring