質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

c++でのGetCellDataについて

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,828

score 18

前提・実現したいこと

Com初心者です。
現在cellからデータを取り出すコードを書いている途中で、
参考にと渡されたコードがあるのですが、
どういった処理を行っているのかいまいちわかっていないことと、
Variantを使用していますが、VariantInit()やVariantClear()がありません。
これって解放されていることになってるのでしょうか。

該当のソースコード

.h
void GetCellData(const char* sc,const char* ec,CStringArray &dt,bool formula = false);
void GetCellData(int cols,int rows,int cole,int rowe,CStringArray &dt,bool formula = false);
CExcelCtrl();
virtual ~CExcelCtrl();

_Application    m_excel;

.cpp
CExcelCtrl::CExcelCtrl()
{
    CoInitialize(NULL);
}

CExcelCtrl::~CExcelCtrl()
{
    CoUninitialize();
}

void CExcelCtrl::GetCellData(int cols,int rows,int cole,int rowe,CStringArray &dt,bool formula)
{
    CString sc,ec;
    sc.Format("%s%d",Num2Col(cols),rows);
    ec.Format("%s%d",Num2Col(cole),rowe);
    GetCellData(sc,ec,dt,formula);
}

void CExcelCtrl::GetCellData(const char* sc,const char* ec,CStringArray &dt,bool formula)
{
    try{
        _Worksheet ws = m_excel.GetActiveSheet();
        Range range = ws.GetRange(COleVariant(sc),COleVariant(ec));

        COleVariant data;
        if(formula)    data = range.GetFormula();
        else        data = range.GetValue(vtMissing);

        COleSafeArray sa;
        sa.Attach(data);

        long rowmax,colmax;
        sa.GetUBound(1,&rowmax);
        sa.GetUBound(2,&colmax);

        CString str;
        VARIANT val;
        long idx[2];

        dt.RemoveAll();
        for(long row = 1;row <= rowmax;row++){
            idx[0] = row;
            for(long col = 1;col <= colmax;col++){
                idx[1] = col;
                sa.GetElement(idx,&val);
                switch(val.vt){
                case    VT_R8:
                    str.Format("%1.2f", val.dblVal);
                    break;
                case    VT_BSTR:
                    str.Format("%s",(CString)val.bstrVal);
                    ::SysFreeString(val.bstrVal);
                    break;
                case    VT_EMPTY:
                    str.Empty();
                    break;
                }
                dt.Add(str);
            }
        }
    }catch(COleDispatchException *e){
        AfxMessageBox(e->m_strDescription,MB_ICONEXCLAMATION);
        dt.RemoveAll();
    }
}

補足情報(FW/ツールのバージョンなど)

windows10/64bit
visual studio 2017

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

Variantを使用していますが、VariantInit()やVariantClear()がありません。
これって解放されていることになってるのでしょうか。

この辺はC++とマイクロソフトのMFC(Microsoft Foundation Class)と言うものをある程度理解しないと分かりづらいのですが、C言語で生でCOMを扱う際のVARIANT型などをC++のクラスでラップしたCOleVariantクラス
や COleSafeArray クラスがあり、これらクラスのメソッドであるClear等に隠されています。
ご提示のコードは一見そのClearメソッドすら見えないのでアレレ?となってしまいますが、これらクラスのデストラクター内でも呼ばれていたりして、クラスのインスタンスが破棄されるときに適切に呼び出されます。

ご提示のコードの場合はautoでCOleSafeArray saを宣言したのち、sa.Attach(data)で実体が当てはめられ、そのインスタンス変数のスコープの終わりでデストラクトされる際にClearが呼ばれます。MFCはVisual Studio のバージョンにもよりますがソースコードが提供されています。Visual Studio を適切にインストールした後でデバッグ時にトレースするとそのコードに行き当たりますので、実際に追ってみるとよいです。C言語でのCOMプログラミングの参考にもなります。例えばCOleSafeArrayクラスですと、デバッグでのトレース時に下図のように行き当たります。

イメージ説明
COMは(もともと難解な上に)最近ではCやC++を使ったサンプルや情報が少ないので大変かもしれませんが、がんばってください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/12/25 16:19

    ご回答ありがとうございます。
    なるほど別のところに用意されていて、書いて無くても大丈夫だったのですか。
    ここら辺が理解できなかったので、助かりました。

    キャンセル

+1

vt含めた各メンバは sa.GetElement(で適切に代入されているはずなのでVariantInit()VariantClear()は不要と思われます。
なお、配列操作部分のコードは基本的にColesafearray::getelementのサンプルコードと同一のようです。
一点気になるのはVT_BSTRの場合に::SysFreeStringが必要かどうですが、ヘルプのコードには記載されていませんね。
VARIANT型を知ってみるVARIANT型を使ってみるによるとBSTR型の場合は必要そうに思えますが、元のCOleSafeArray内部で解放しているので不要なのかもしれません。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/12/26 14:22 編集

    atata0319さん、コメント&情報ありがとうございます。
    あらら、サンプルの不具合なんですね…
    ちなみにですが、SafeArrayGetElement利用のAPIサンプルの公開URLをもしご存知でしたら教えていただけないでしょうか?
    SysFreeStringが必要である根拠なり(サンプルコード)が知りたいのですが、ネット上では
    http://forums.codeguru.com/showthread.php?157122-SafeArrayGetElement-problem
    しかヒットしませんでした。

    キャンセル

  • 2018/12/26 15:07

    https://docs.microsoft.com/en-us/windows/desktop/api/oleauto/nf-oleauto-safearraygetelement
    です。Remarksには値はコピーされることが記載されています。
    一番下のサンプルでは受け取った値はすべてVariantClearで解放していますが、これがBSTRを直接使用する場合、SysFreeStringに変更することがあると言うことの根拠ですね。

    キャンセル

  • 2018/12/26 15:10

    > ~すべてVariantClearで解放していますが~
    なるほど。VariantClearとSysFreeStringの関係に気づけませんでした。
    ありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る