Kuon Utilities On .NetFramework

こんにちは。　kuon/trunk/Utilities/ShellLink.csのコメントにあるサイトの管理人です。

サイトで紹介していたShellLinkの実装について、誤りがあったのでお知らせします。

まず1つめ、HotKeyプロパティの実装についてです。
IShellLinkW.GetHotkeyで取得できるキーコードはushortですが、下位8ビットに
仮想キーコード、上位8ビットに修飾キーが入ります。
一方System.Windows.Forms.Keysは下位16ビットに仮想キーコード、上位16ビットに
修飾キーが入ります。
IShellLinkW.GetHotkeyから取得したushort値をそのままSystem.Windows.Forms.Keysに
キャストしても正しい値にはなりません。

次に2つめ、Load, Saveした時のファイル名を保持するcurrentFileメンバについてです。
カレントファイルはIPersistFileでも保持していて、IPersistFile.GetCurFileメソッドで
パスを取得できます。　そのため、currentFileメンバでも保持するのは二重管理と
なってしまいます。

以上2点です。
実装を見させて頂いた限り、どちらも問題にはならなそうでしたが、念のため
ご報告させて頂きました。

上記2点を修正したパッチを作成しましたので、よろしければお使いください。
なお、ソース中に記載されているURLですが、ページを移動したため古いURLと
なっています。　勝手ながら新しいURLに変更させて頂いています。


Index: ShellLink.cs
===================================================================
--- ShellLink.cs (revision 297)
+++ ShellLink.cs (working copy)
@@ -8,14 +8,11 @@
namespace Kuon.Utilities{
using ComTypes = System.Runtime.InteropServices.ComTypes;

- // based on http://smdn.invisiblefulmoon.net/ikimasshoy/dotnettips/tips043.html
+ // based on http://smdn.invisiblefulmoon.net/programming/tips/createlnk/
public sealed class ShellLink : IDisposable{
// IShellLinkインターフェイス
private IShellLink shellLink;

- // カレントファイル
- private string currentFile;
-
// 各種定数
internal const int MAX_PATH = 260;

@@ -25,7 +22,6 @@

#region "コンストラクション・デストラクション"
public ShellLink(){
- this.currentFile = "";
this.shellLink = null;

try{
@@ -71,7 +67,15 @@
/// </summary>
public string CurrentFile{
get{
- return currentFile;
+ ComTypes.IPersistFile persistFile = GetIPersistFile();
+ if (persistFile == null) {
+ return null;
+ }
+ else {
+ string file;
+ persistFile.GetCurFile(out file);
+ return file;
+ }
}
}

@@ -210,13 +214,29 @@
get{
ushort hotKey = 0;
this.shellLink.GetHotkey(out hotKey);
- return (Keys)hotKey;
+ return TranslateKeyCode(hotKey);
}
set{
- this.shellLink.SetHotkey((ushort)value);
+ this.shellLink.SetHotkey(TranslateKeyCode(value));
}
}

+ private static ushort TranslateKeyCode(Keys key)
+ {
+ int virtKey = ((int)(key & Keys.KeyCode) & 0x00ff);
+ int modifier = (((int)(key & Keys.Modifiers) >> 8) & 0xff00);
+
+ return (ushort)(virtKey | modifier);
+ }
+
+ private static Keys TranslateKeyCode(ushort key)
+ {
+ Keys virtKey = (Keys)(key & 0x00ff);
+ Keys modifier = (Keys)((key & 0xff00) << 8);
+
+ return virtKey | modifier;
+ }
+
#endregion

#region "保存と読み込み"
@@ -235,7 +255,7 @@
/// <exception cref="COMException">IPersistFileインターフェイスを取得できませんでした。</exception>
public void Save()
{
- this.Save(currentFile);
+ this.Save(this.CurrentFile);
}

/// <summary>
@@ -244,13 +264,11 @@
/// <param name="linkFile">ショートカットを保存するファイル</param>
/// <exception cref="COMException">IPersistFileインターフェイスを取得できませんでした。</exception>
public void Save(string linkFile){
+ if (linkFile == null) throw new ArgumentNullException("linkFile");
// IPersistFileインターフェイスを取得して保存
ComTypes.IPersistFile persistFile = GetIPersistFile();
if(persistFile == null) throw new COMException("IPersistFileインターフェイスを取得できませんでした。");
persistFile.Save(linkFile, true);
-
- // カレントファイルを保存
- this.currentFile = linkFile;
}

/// <summary>
@@ -316,9 +334,6 @@
}

this.shellLink.Resolve(hWnd, flags);
-
- // カレントファイルを指定
- currentFile = linkFile;
}
#endregion
}