kostik450 Nov 22 2011 at 15:07

Об одной проблеме запуска Архивации и Восстановления при использовании OpenSource кода с сайта CodeGuru

3 min

9.6K

Lumber room

+16

Comments 15

silvansky Nov 22 2011 at 15:17

«goto err_exit;»
Вы убили мозг моего внутреннего программиста!
Пожалуйста, во избежание травмирования мозга других хабраюзеров, поменяйте ЭТО на обычный «break;».

silvansky Nov 22 2011 at 15:20

STDMETHODIMP CShellExt::InvokeCommand(LPCMINVOKECOMMANDINFO lpcmi)
{
//
//немного кода skipped
//
    HRESULT hr = NOERROR;

    if (!HIWORD(lpcmi->lpVerb)) 
    {
         idCmd = LOWORD(lpcmi->lpVerb);

         // process it
         switch (idCmd) 
         {
                default:
                    hr = E_INVALIDARG;
                    break;
                case 0: // operation 1
                case 1: // operation 2
//
//много кода skipped
//
                break;
         }    // switch on command
    }
   return hr;
}

Хотя бы так…

kostik450 Nov 22 2011 at 15:40

С этим кодом тоже проблемы. Нужно не забыть обработать ветку if — else

STDMETHODIMP CShellExt::InvokeCommand(LPCMINVOKECOMMANDINFO lpcmi)
{
//
//немного кода skipped
//
    HRESULT hr = NOERROR;

    if (!HIWORD(lpcmi->lpVerb)) 
    {
         idCmd = LOWORD(lpcmi->lpVerb);

         // process it
         switch (idCmd) 
         {
                default:
                    hr = E_INVALIDARG;
                    break;
                case 0: // operation 1
                case 1: // operation 2
//
//много кода skipped
//
                break;
         }    // switch on command
    }
    else
    {
        hr = E_INVALIDARG;
    }
   return hr;
}

А вообще да, goto это зло.

Просто я лично привык к goto с незапамятных времен школьного BASIC. Там без GOTO обойтись тяжело. Еще помню, была просто волшебная команда RENUMBER.

silvansky Nov 22 2011 at 15:42

Да, согласен, писал второпях, желая пояснить суть своего негодования.

ЗЫ: сам начинал с васика, там были GOTO и GOSUB незаменимы.

silvansky Nov 24 2011 at 11:41

А статью так и не обновили…
И молодёжь, не читающая комменты, будет думать, что goto это хорошо…

kostik450 Nov 24 2011 at 12:58

Ниже уже ответили, что goto это хорошо для обработки ошибок.
Да я и сам так всегда делаю, очень удобно. Вот пример кода:

int TestFunction(void)
{
    int result;
    char *Buffer=NULL;

    Buffer = (char *)ALLOC(MAX_BUFFER_LENGTH);
    if (Buffer == NULL)
    {
        result = ERR_MEM;
        goto err_exit;
    }

    result = DoSomeJob(Buffer);
    if (result) goto err_exit;

    result = DoAnotherJob(Buffer);
    if (result) goto err_exit;

    result = DoThirdJob(Buffer);

err_exit:
    if (Buffer) FREE(Buffer);
    return result;
}

Здесь всегда можно быть уверенным на 100% что не забудем сделать FREE.
А теперь попробуйте эту функцию реализовать без goto — скорее всего будет нечитаемо.

silvansky Nov 24 2011 at 13:00

А чем try/catch/finally не угодил?
Собственно, для этих целей исключения и были придуманы.

Maratyszcza Nov 23 2011 at 07:33

Использование goto для перехода на обработчик ошибок — стандартная практика в C коде. Хотя код автора и на C++, написан он в стиле C.

sineborod Nov 22 2011 at 20:02

Вот что бывает, когда вместо того, чтобы проанализировать чужой код, понять суть и сделать по-своему и правильно, тупо отключают мозг и копируют код.

pietrovich Nov 22 2011 at 23:15

Спасибо! Грандиозное спасибо! Вот оно! Вот из-за чего (CompareIt!) у меня «внезапно» отрубался «Запуск от имени Администратора» на ярлыках. Просто между установкой и обнаружением проблемы обычно проходило достаточно времени чтобы забыть о том что ставилось. Да и, признаться, никогда бы на нее не подумал. К тому же происходило это всего два раза (да, винду переставляю редко), поэтому статистику для анализа собрать пока не успел. Так и жил без «Запустить от имени Администратора», выкручивался посредством «Расположение файла» и «Запуск от Админа» уже непосредственно на экзешнике.
Теперь бы еще придумать чем заменить CompareIt! или как релизовать его глючащие «compare file #1» и «compare file #2» в обход, неглючным методом.

pietrovich Nov 22 2011 at 23:25

Обновил CompareIt!, проблема пропала. Как-то не следил за обновлениями, меня он полностью устраивал, вот и получил граблю :")

kostik450 Nov 23 2011 at 15:40

Да, правильно. «Запуск от имени Администратора» на ярлыках перестает работать — это тоже связано с некорректной обработкой ContextMenu. Нужно бы отдельную статью посвятить и этой проблеме тоже.

Можете ли еще подсказать, какая версия CompareIt! у Вас стояла, в которой наблюдалась ошибка?

pietrovich Nov 23 2011 at 16:00

Посмотрел. Была установлена 4.1.0.2170 (ей года три, наверное). После обновления до 4.2.0.2221 проблема устраниась. Теоретически должна помочь замена wincmpExt.dll(wincmpExt64.dll) на свежие версии (новые на два года моложе).

Gorthauer87 Nov 22 2011 at 23:31

LPCMINVOKECOMMANDINFO lpcmi
Неужели кто-то ещё так именует типы переменных? Это же АДЪ И ИЗРАИЛЬ! Наверняка же есть обновленная версия API, где нет такого душераздирающего ужаса!

nullbie Nov 23 2011 at 00:13

Типичное именование для WinAPI и COM на C++, смысл именовать иначе когда вокруг весь код и все API написаны придерживаясь такой конвенции — только путать будет.