httpbin.org/ipからグローバルIPを取得するWindowsフォームアプリケーションを作りたい

Question

初心者なので、質問の仕方がわかりづらかったらすみません。

以下のソースコードでコマンドプロンプトからcurlを打ちたいのですが、実行すると何も書かれていないメッセージボックスが出るだけです。。

pingで試したらできたので、おそらくコマンドプロンプトは開けています。

curl.exeはC:\Program Files\curl-7.69.1-win64-mingw\binにあります。

参考サイトは、
http://dobon.net/vb/dotnet/process/standardoutput.html

よろしくお願いします。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace curl_API
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click_1(object sender, EventArgs e)
        {

            //Processオブジェクトを作成
            System.Diagnostics.Process p = new System.Diagnostics.Process();

            
            //OutputDataReceivedイベントハンドラを追加
            p.OutputDataReceived += p_OutputDataReceived;

            //ComSpec(cmd.exe)のパスを取得して、FileNameプロパティに指定
            p.StartInfo.FileName =
                System.Environment.GetEnvironmentVariable("ComSpec");
            //出力をストリームに書き込むようにする
            p.StartInfo.UseShellExecute = false;
            p.StartInfo.RedirectStandardOutput = true;
            p.StartInfo.RedirectStandardInput = false;
            //ウィンドウを表示しないようにする
            p.StartInfo.CreateNoWindow = true;

            string cmd = "curl httpbin.org/ip";
            p.StartInfo.Arguments = @"/c " + cmd;

            //起動
            p.Start();

            //非同期で出力の読み取りを開始
            p.BeginOutputReadLine();

            p.WaitForExit();
            p.Close();

            Console.ReadLine();



        }
        //OutputDataReceivedイベントハンドラ
        //行が出力されるたびに呼び出される
        static void p_OutputDataReceived(object sender,
            System.Diagnostics.DataReceivedEventArgs e)
        {
            //出力された文字列を表示する
            MessageBox.Show(e.Data);
        }
    }
}

Comments

[gentaro]

「できません」じゃなく「何が起きているのか」を書いた方がいいです。

[Gunjirk]

失礼しました！

修正しました！

[hihijiji]

非同期で書いているようですが、同期では取得できているのですか？

[Gunjirk]

このコードも大本がコピペなので、非同期でしか試していません！

[Y.H.]

curl.exeはC:\Program Files\curl-7.69.1-win64-mingw\binにあります。
pathは通ってるんでしょうか？

[Gunjirk]

コマンドプロンプトでコマンドを実行すると、期待値が取得できるので、パスが通っている認識です！

C:\Users\XXX>cd C:\Program Files\curl-7.69.1-win64-mingw\bin

C:\Program Files\curl-7.69.1-win64-mingw\bin>curl httpbin.org/ip
{
"origin": "165.76.168.60"
}

C:\Program Files\curl-7.69.1-win64-mingw\bin>

[Y.H.]

> cd C:\Program Files\curl-7.69.1-win64-mingw\bin
cd "C:\Program Files\curl-7.69.1-win64-mingw\bin"
じゃなくて？
っていうよりCDしてますね。
cd でカレントディレクトリ移動せずに
実行デキないならpath 通ってないです。

[tamoto]

C# で Web サービスから IP を取るのに curl を使う必要は全く無いので、IP を取りたいのか curl を使いたいのか、どちらかはっきりさせたほうが良いと思います。

[Zuishin]

コピペしたなら必ず、くどいようですが、必ず、出典を明記してください。

Answer 1

下記は、コマンドから返された文字列を表示するイベントのようですね。
System.Diagnostics.DataReceivedEventArgs

参考URL：https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.datareceivedeventargs?view=netframework-4.8

意図している動きか分かりませんが、メッセージボックスで表示させたいのであれば、下記のように変更してみては如何でしょうか？

        private static string strcmd = "";　//文字列受け取り用

        public Form1()
        {
            InitializeComponent();
        }

        private void Button1_Click(object sender, EventArgs e)
        {
            //Processオブジェクトを作成
            System.Diagnostics.Process p = new System.Diagnostics.Process();


            //OutputDataReceivedイベントハンドラを追加
            p.OutputDataReceived += p_OutputDataReceived;

            //ComSpec(cmd.exe)のパスを取得して、FileNameプロパティに指定
            p.StartInfo.FileName =
                System.Environment.GetEnvironmentVariable("ComSpec");

            //出力をストリームに書き込むようにする
            p.StartInfo.UseShellExecute = false;
            p.StartInfo.RedirectStandardOutput = true;
            p.StartInfo.RedirectStandardInput = false;
            //ウィンドウを表示しないようにする
            p.StartInfo.CreateNoWindow = true;

            string cmd = "curl httpbin.org/ip";
            p.StartInfo.Arguments = @"/c " + cmd;

            strcmd = @"/c " + cmd + Environment.NewLine; // コマンドを打ったことを記録

            //起動
            p.Start();

            //非同期で出力の読み取りを開始
            p.BeginOutputReadLine();

            p.WaitForExit();
            p.Close();

            MessageBox.Show(strcmd); //表示

            //Console.ReadLine();

        }

        //OutputDataReceivedイベントハンドラ
        //行が出力されるたびに呼び出される
        static void p_OutputDataReceived(object sender,
            System.Diagnostics.DataReceivedEventArgs e)
        {
            //出力された文字列を表示する
            //MessageBox.Show(e.Data);
            strcmd = strcmd + e.Data + Environment.NewLine; //リダイレクトされてくる文字列を格納
        }

Comments

[Gunjirk]

ありがとうございます！

あと、curl httpbin.org/ipをコマンドプロンプト上で打って結果をメッセージボックスで表示させたいです。

[SN_Ares]

下記のコードでコマンドは打たれていると思います。

```C#
string cmd = "curl httpbin.org/ip";
p.StartInfo.Arguments = @"/c " + cmd;
```

コマンドプロンプト上ということは、コマンドプロンプトを起動したいという事でしょうか？？

[Y.H.]

SN_Aresさん
cmd.exe経由でcurl実行のコードですよ。
p.StartInfo.FileNameにcmd.exe(環境変数
ComSpec)を指定

[SN_Ares]

Y.Hさん
えーと、、それはもう経由しているという事で良いですよね？

Gunjirkさん
コードを変更してみました。こういうことで良いですか？？

[Y.H.]

はい。

[Y.H.]

なのでうまくいかない原因は、cmd.exeがcurlを見つけられてないだけたと思います。
「質問への追記・修正、ベストアンサー選択の依頼」に記載している通りで
「C:\Program Files\curl-7.69.1-win64-mingw\bin」にPATHが通ってないだけだと思いますよ。

[SN_Ares]

あ、申し訳ありません。
「質問への追記・修正、ベストアンサー選択の依頼」に記載あったんですね。完全に見落としていました(^^;)（そういう欄もあるんですね・・・）