Что люди любят делать не так в IUnknown.QueryInterface

Это перевод The ways people mess up IUnknown::QueryInterface. Автор: Реймон Чен.

Когда вы имеете дело с вопросами совместимости приложений, каких только вещей, работающих только благодаря случайности, вы ни обнаруживаете! Сегодня я поговорю о некоторых "творческих" способах напутать с методом IUnknown.QueryInterface.

Сейчас вы должно быть подумали: "Этот интерфейс так важен для всего COM, как же можно сделать с ним что-то не так?". Ну-ну.

Забываем отвечать на IUnknown.

Иногда вы так взволнованы необходимостью отвечать на все свои прекрасные интерфейсы, что вы совершенно забываете о необходимости ответить на сам интерфейс IUnknown. Мы видели объекты, у которых вызов

var
  psf: IShellFolder;
  punk: IUnknown;
...
  psf := some object;
  psf.QueryInterface(IID_IUnknown, punk);

завершался неудачей с кодом E_NOINTERFACE!

Забываем отвечать на свой собственный интерфейс.

Есть методы, которые возвращают объект с конкретным интерфейсом. А если вы спросите у объекта его же собственный интерфейс (единственную причину для его существования), то он вам в ответ: "чего? О_о"

var
  psf: IShellFolder;
  peidl, peidl2: IEnumIDList;
...
  psf := some object;
  psf.EnumObjects(..., peidl);
  peidl.QueryInterface(IID_IEnumIDList, peidl2);

Некоторые объекты возвращают здесь E_NOINTERFACE из QueryInterface, хотя вы спрашиваете объект о нём самом!
Похоже они хотят сказать: "Извините, но я не существую".

Забываем отвечать на базовый интерфейс.

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

var
  psv: IShellFolder;
  pow: IOleView;
...
  psv := some object;
  psv.QueryInterface(IID_IOleView, pow);

А то некоторые объекты забывают об этом и возвращают E_NOINTERFACE из QueryInterface.

Требование секретного тук-тук.

Вообще-то, следующие два фрагмента кода эквивалентны:

var
  psf: IShellFolder;
  punk: IUnknown;
...
  CoCreateInstance(CLSID_xyz, ..., IID_IShellFolder, psf);
  psf.QueryInterface(IID_IUnknown, punk);

  CoCreateInstance(CLSID_xyz, ..., IID_IUnknown, punk);
  punk.QueryInterface(IID_IShellFolder, psf);

А в действительности, некоторые реализации лажают при втором вызове CoCreateInstance. Единственным способом создать объект успешно будет создать его через интерфейс IShellFolder.

Забываем правильно сказать "Нет".

Одним из правил при ответе "Нет" является требование установки выходного интерфейса в nil перед выходом. Некоторые забывают это делать

var
  pmbl: IMumble;
...
  punk.QueryInterface(IID_IMumble, pmbl);

Если вызов метода QueryInterface успешен, тогда pmbl должен быть не nil при выходе. А если вызов завершился неудачей, то pmbl обязан быть nil.

А ведь оболочка (shell) обязана быть совместимой со всеми этими багнутыми реализациями, потому что если она не будет совместима, то пользователи расстроятся, а пресса получит свою сенсацию. Некоторые из тех, кто не выполняет эти соглашения - программы С Большим Именем. Если они не будут работать в следующей версии Windows, люди будут говорить: "не обновляйтесь на Windows XYZ, она не совместима с <программа-с-большим-именем>!". А параноидально настроенные люди будут кричать: "Microsoft намеренно не даёт работать <программе-с-большим-именем>! Доказательство нечестной бизнес-тактики!".