C#でTodo.lyにPostでItemを追加したいです。

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 621

cancat

score 247

こんにちは。 
Windows10でWPFのアプリケーションを開発しています。 
Visual Studio 2015 Communityを使っています。 

前提・実現したいこと

Todo.lyにPostでItemを追加したいです。

試したこと

下記のとおり。

発生している問題・エラーメッセージ

Postされないです。。。
エラーも発生しないです。

該当のソースコード

private const string UserName = "cancat";
        private const string Password = "password";
        private const string useragent = "Shin-Mozilla/5.5 (Windows NeXT 5.5; Trident/7.0; rv:11.0) like GeckohKamen";

        private const string BaseUri = "https://todo.ly";
        private const string TokenUri = "api/authentication/token.json";
        private static string UriOfItems = "api/items.json";

        private static HttpClient client;
        private static DateTime? ExpirationTime = null;

        static Todoly() {
            var credentials = Encoding.ASCII.GetBytes($"{UserName}:{Password}");

            client = new HttpClient();
            client.DefaultRequestHeaders.Add("User-Agent", useragent);
            client.DefaultRequestHeaders.Add("Accept-Language", "ja-JP");
            client.BaseAddress = new Uri(BaseUri);
            client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Basic", Convert.ToBase64String(credentials));
        }

        private void Window_Loaded(object sender, RoutedEventArgs e) {
            todoly();
        }

        private async void todoly() {
            var todoly = new Todoly.Todoly();
            //var items = await todoly.GetAllItems();
            todoly.PostItems("");
        }

        public void PostItems(string message) {
            message = "カーテン。";

            var encode = Encoding.GetEncoding("UTF-8");
            var postData = "Content=" + Uri.EscapeDataString(message);
            var postDataBytes = encode.GetBytes(postData);

            // WebRequest作成
            var requestUrl = string.Format(BaseUri, UriOfItems);
            var request = WebRequest.Create(requestUrl);
            request.Method = "POST";
            request.ContentType = "application/x-www-form-urlencoded";
            // POSTデータ長を指定
            request.ContentLength = postDataBytes.Length;
            try {
                // データをPOST送信するためのStreamを取得
                var requestStream = request.GetRequestStream();
                // 送信するデータを書き込む
                requestStream.Write(postDataBytes, 0, postDataBytes.Length);
                requestStream.Close();
            }
            catch (WebException webexception) {
                string error = webexception.Response.ToString();
            }
            catch (Exception exception) {
                string error = exception.Message;
            }
        }
    }
}
//http://todo.ly/apiwiki/?items/todo-ly-rest-api-method-post-items

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

Microsoft Visual Studio Community 2015 
Version 14.0.25424.00 Update 3 
Microsoft .NET Framework 
Version 4.6.01038 

インストールしているバージョン:Community 

Visual C# 2015   00322-20000-00000-AA575 
Microsoft Visual C# 2015 

です。 
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

突っ込みどころが多すぎるのでパタパタと書きますが....

var requestUrl = string.Format(BaseUri, UriOfItems);
ってrequestUrlになにはいってますかね?謎いです。(BaseUriの値を見る限りstring.Formatして何か値が変わることがないの)

private async void todoly() {のasyncにvoid使うのはここではバットプラクティスですし、場合によっては非同期処理がしぬかと。
(async/awaitの仕組みや基礎をさっと語るだけだと語弊しか生まれないので説明はしませんが、いろんな方が記事書いてますので整理したほうが良いかと思います。)

そしてなぜWebRequestを使っちゃったんでしょう...HttpClientがあるので、それを使えばよいかと。BaseUri(https://todo.ly)がセットされているインスタンスがあるのでそれを利用して、PostAsyncすればさらっと接続できると思います。
(これも詳しくは語りませんが、私個人的にはWebRequest is deadです。)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/12/08 15:40

    コメントありがとうございます。
    以前、chatworkに投稿できまして、それをベースにしてみました。
    Webとのやりとりのコード、ほんっとに初心者なのです。
    そのChatworkのときが初めてで、今回が2回目。
    まだまだ五里霧中です。。。
    WebRequestだめなのですね。そうかインスタンス引き継がれないですね。なるほど。

    キャンセル

  • 2016/12/08 15:52

    動作を実現できるかどうかの話だとWebRequestでも全く問題ないです。

    ただ、HttpClient使って認証とか色々設定されているのに、それ使わず別のClient立てる(かつ、古いクラスなので設定したりPostしたりするのすらめんどい)のは微妙って話ですね。

    キャンセル

  • 2016/12/08 18:00

    あらためて。

    using System;
    using System.Text;

    namespace Todoly {
    public partial class Todoly {
    public async void PostItems(string message) {
    message = "カーテン。";

    var encode = Encoding.GetEncoding("UTF-8");
    var postData = "Content=" + Uri.EscapeDataString(message);
    var postDataBytes = encode.GetBytes(postData);

    if (NeedAuth()) {
    var tokenResponse = await client.GetAsync(TokenUri);
    }

    var response = await client.PostAsync(UriOfItems, message);

    var resultString = await response.Content.ReadAsStringAsync();
    }}}
    としてみました。
    message(string)をSystem.Net.Http.HttpContentにできればよさそうです。

    キャンセル

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

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

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