更新Media类的写法。我觉得应该把调用方法所需参数作为类的属性，这样才是真正的面向对象。

[zeusro]

/// <summary>
///  上传/下载媒体基类
/// </summary>
public class Media
{
    /// <summary>
    /// 调用接口凭证 
    /// </summary>
    public string access_token { get; set; }

    /// <summary>
    /// 媒体文件类型，分别有图片（image）、语音（voice）、视频（video）和缩略图（thumb） 
    /// </summary>
    public BaseInfoTYPE type { get; set; }

    /// <summary>
    /// 完整路径+文件名
    /// </summary>
    public string fileName { get; set; }

    /// <summary>
    /// 下载时用到
    /// </summary>
    public string mediaId { get; set; }


    /// <summary>
    /// 上传媒体文件
    /// </summary>
    /// <param name="accessToken"></param>
    /// <param name="type"></param>
    /// <param name="file"></param>
    /// <returns></returns>
    public  UploadResultJson Upload()
    {
        var url = string.Format("http://file.api.weixin.qq.com/cgi-bin/media/upload?access_token={0}&type={1}", access_token, type.ToString());
        var fileDictionary = new Dictionary<string, string>();
        fileDictionary["media"] = fileName;
        return HttpUtility.Post.PostFileGetJson<UploadResultJson>(url, null, fileDictionary, null);
    }


    /// <summary>
    /// 下载多媒体文件,至少要实例化access_token，type，fileName，mediaId三个属性
    /// </summary>
    /// <param name="accessToken"></param>
    /// <param name="mediaId"></param>
    /// <param name="fileName">完整路径E:\testpic_{0}.jpg</param>
    /// <returns></returns>
    public bool DownloadMedia()
    {
        using (MemoryStream ms = new MemoryStream())
        {
            var url = string.Format("http://file.api.weixin.qq.com/cgi-bin/media/get?access_token={0}&media_id={1}",
             access_token, mediaId);
            Get.Download(url, ms);
            if (ms.Length < 1)
            {
                return false;
            }
            using (FileStream fs = new FileStream(fileName, FileMode.Create))
            {
                ms.Position = 0;
                byte[] buffer = new byte[1024];
                int bytesRead = 0;
                while ((bytesRead = ms.Read(buffer, 0, buffer.Length)) != 0)
                {
                    fs.Write(buffer, 0, bytesRead);
                }
                fs.Flush();
            }

        }
        return File.Exists(fileName);
    }



}

[JeffreySu]

感谢提交代码！看了一下，个人不太认同这种写法，比如当用户忘记传入accessToken或者file等其他参数的时候，这个方法会发生异常，也就是说使用者必须非常记住每一个方法需要用到什么参数，然后传入才能成功，不可取。而且粒度太大，不利于测试。

[zeusro]

本来调用接口就要明白参数是什么，你方法参数里面用一堆字段，那微信接口变更是不是再度加个参数跟在后面？参数应该应该尽可能是对象，或者照我理解，干脆就弄成对象的属性。再不然构造函数注入参数。这样一切不就很明白了吗？Waiting for your answer！

[zeusro]

    [TestMethod]
    public void GetTest()
    {
        var accessToken = AccessTokenContainer.GetToken(_appId);

        UploadTest();//上传

        using (MemoryStream ms = new MemoryStream())
        {
            Media.Get(accessToken, mediaId, ms);
            Assert.IsTrue(ms.Length > 0);

            //保存到文件
            var fileName = string.Format(@"E:\testpic_{0}.jpg", DateTime.Now.Ticks);
            using (FileStream fs = new FileStream(fileName, FileMode.Create))
            {
                ms.Position = 0;
                byte[] buffer = new byte[1024];
                int bytesRead = 0;
                while ((bytesRead = ms.Read(buffer, 0, buffer.Length)) != 0)
                {
                    fs.Write(buffer, 0, bytesRead);
                }
                fs.Flush();
            }

            Assert.IsTrue(File.Exists(fileName));
        }
    }

看看你这段测试，业务逻辑都包进测试里面了。我表示从来没见过这样的测试写法。我所了解的测试从来都不是这样拖泥带水的，从来都只有3步：
1new 对象，装填字段：
2调用方法；
3断言方法返回值。
本来MVC模式，所有业务逻辑都在领域对象上，控制器只是一个纽带。你方法写不齐是不是又要控制器给你补全，然后你真的确定你能对模型，视图，控制器分开测试吗？

[JeffreySu]

你好，分别回答一下：
1、参数有参数的用途，传参和面向对象是不矛盾的，如果为了面向对象而面向对象，反而会使系统变得更加复杂，例如你提出的做法，弊端我已经解释过了，坦白地说这种设计是非常糟糕的。

2、这一段测试代码针对Media.Get的测试，其实到Assert.IsTrue(ms.Length > 0);就已经结束了，下面只是给大家演示一下如何保存ms里面的东西（之前有人提到过这个问题），并且确定这种方式是有效的（ Assert.IsTrue(File.Exists(fileName))）。实际操作灵活使用就行了。

非常感谢！