prevent creation of Lists and use IEnumerable where possible

PocketSharp.Examples/PocketSharp.WP8/ViewModels/MainViewModel.cs

@@ -5,6 +5,7 @@ using System.Collections.ObjectModel;
 using System.ComponentModel;
 using System.Diagnostics;
 using System.Threading.Tasks;
+using System.Linq;
 
 namespace PocketSharp.WP8.ViewModels
 {
@@ -74,7 +75,7 @@ namespace PocketSharp.WP8.ViewModels
 
       try
       {
-        items = await client.Get();
+        items = (await client.Get()).ToList();
 
         items.ForEach(item =>
         {

PocketSharp.Examples/PocketSharp.Wpf/MainWindow.xaml.cs

@@ -2,6 +2,7 @@ using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Diagnostics;
 using System.Windows;
+using System.Linq;
 
 namespace PocketSharp.Wpf
 {
@@ -40,7 +41,7 @@ namespace PocketSharp.Wpf
 
       try
       {
-        items = await client.Get();
+        items = (await client.Get()).ToList();
 
         items.ForEach(item =>
         {

PocketSharp.Tests/AddTests.cs

@@ -3,6 +3,7 @@ using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using Xunit;
+using System.Linq;
 
 namespace PocketSharp.Tests
 {
@@ -47,7 +48,7 @@ namespace PocketSharp.Tests
         tweetID: "380051788172632065"
       );
 
-      List<PocketItem> items = await client.Get();
+      List<PocketItem> items = (await client.Get()).ToList();
       PocketItem itemDesired = null;
 
       items.ForEach(itm =>
@@ -60,7 +61,7 @@ namespace PocketSharp.Tests
 
       Assert.NotNull(itemDesired);
       Assert.Equal(itemDesired.ID, item.ID);
-      Assert.Equal(itemDesired.Tags.Count, 3);
+      Assert.Equal(itemDesired.Tags.Count(), 3);
 
       itemsToDelete.Add(item.ID);
     }

PocketSharp.Tests/GetTests.cs

@@ -5,6 +5,7 @@ using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using Xunit;
+using System.Linq;
 
 namespace PocketSharp.Tests
 {
@@ -16,7 +17,7 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task AreItemsRetrieved()
     {
-      List<PocketItem> items = await client.Get();
+      List<PocketItem> items = (await client.Get()).ToList();
       Assert.True(items.Count > 0);
     }
 
@@ -24,7 +25,7 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task IsItemRetrievedById()
     {
-      List<PocketItem> items = await client.Get();
+      List<PocketItem> items = (await client.Get()).ToList();
       PocketItem item = items[0];
       PocketItem itemDuplicate = await client.Get(item.ID);
 
@@ -35,7 +36,7 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task IsItemJsonPopulated()
     {
-      List<PocketItem> items = await client.Get();
+      List<PocketItem> items = (await client.Get()).ToList();
       string schemaJson = @"{
                 'description': 'PocketItem',
                 'type': 'object'
@@ -53,16 +54,16 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task AreFilteredItemsRetrieved()
     {
-      List<PocketItem> items = await client.Get(RetrieveFilter.Favorite);
+      IEnumerable<PocketItem> items = await client.Get(RetrieveFilter.Favorite);
 
-      Assert.True(items.Count > 0);
+      Assert.True(items.Count() > 0);
     }
 
 
     [Fact]
     public async Task RetrieveWithMultipleFilters()
     {
-      List<PocketItem> items = await client.Get(
+      IEnumerable<PocketItem> items = await client.Get(
         state: State.unread,
         tag: "pocket",
         sort: Sort.title,
@@ -70,29 +71,29 @@ namespace PocketSharp.Tests
         count: 2
       );
 
-      Assert.InRange<int>(items.Count, 0, 2);
+      Assert.InRange<int>(items.Count(), 0, 2);
     }
 
 
     [Fact]
     public async Task ItemContainsUri()
     {
-      List<PocketItem> items = await client.Get(count: 1);
+      IEnumerable<PocketItem> items = await client.Get(count: 1);
 
-      Assert.True(items.Count == 1);
-      Assert.True(items[0].Uri.ToString().StartsWith("http"));
+      Assert.True(items.Count() == 1);
+      Assert.True(items.First().Uri.ToString().StartsWith("http"));
     }
 
 
     [Fact]
     public async Task InvalidRetrievalReturnsNoResults()
     {
-      List<PocketItem> items = await client.Get(
+      IEnumerable<PocketItem> items = await client.Get(
         favorite: true,
         search: "xoiu987a#;"
       );
 
-      Assert.False(items.Count > 0);
+      Assert.False(items.Count() > 0);
 
       PocketItem item = await client.Get("99999999");
 
@@ -100,48 +101,48 @@ namespace PocketSharp.Tests
 
       items = await client.Get(RetrieveFilter.Video);
 
-      Assert.False(items.Count > 0);
+      Assert.False(items.Count() > 0);
     }
 
 
     [Fact]
     public async Task SearchReturnsResult()
     {
-      List<PocketItem> items = await client.Search("pocket");
+      IEnumerable<PocketItem> items = await client.Search("pocket");
 
-      Assert.True(items.Count > 0);
-      Assert.True(items[0].FullTitle.ToLower().Contains("pocket"));
+      Assert.True(items.Count() > 0);
+      Assert.True(items.First().FullTitle.ToLower().Contains("pocket"));
     }
 
 
     [Fact]
     public async Task InvalidSearchReturnsNoResult()
     {
-      List<PocketItem> items = await client.Search("adsüasd-opiu2;.398dfyx");
+      IEnumerable<PocketItem> items = await client.Search("adsüasd-opiu2;.398dfyx");
 
-      Assert.False(items.Count > 0);
+      Assert.False(items.Count() > 0);
     }
 
 
     [Fact]
     public async Task RetrieveTagsReturnsResult()
     {
-      List<PocketTag> items = await client.GetTags();
+      IEnumerable<PocketTag> items = await client.GetTags();
 
-      Assert.True(items.Count > 0);
+      Assert.True(items.Count() > 0);
     }
 
 
     [Fact]
     public async Task SearchByTagsReturnsResult()
     {
-      List<PocketItem> items = await client.SearchByTag("pocket");
+      IEnumerable<PocketItem> items = await client.SearchByTag("pocket");
 
-      Assert.True(items.Count == 1);
+      Assert.True(items.Count() == 1);
 
       bool found = false;
 
-      items[0].Tags.ForEach(tag =>
+      items.First().Tags.ToList().ForEach(tag =>
       {
         if (tag.Name.Contains("pocket"))
         {
@@ -156,9 +157,9 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task InvalidSearchByTagsReturnsNoResult()
     {
-      List<PocketItem> items = await client.SearchByTag("adsüasd-opiu2;.398dfyx");
+      IEnumerable<PocketItem> items = await client.SearchByTag("adsüasd-opiu2;.398dfyx");
 
-      Assert.False(items.Count > 0);
+      Assert.False(items.Count() > 0);
     }
 
 
@@ -183,10 +184,10 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task IsNewPocketItemListGeneratorWorking()
     {
-      List<PocketItem> items = await client.Get(count: 1, tag: "pocket");
-      PocketItem item = items[0];
+      IEnumerable<PocketItem> items = await client.Get(count: 1, tag: "pocket");
+      PocketItem item = items.First();
 
-      Assert.True(item.Tags.Find(i => i.Name == "pocket") != null);
+      Assert.True(item.Tags.ToList().Find(i => i.Name == "pocket") != null);
     }
 
 
@@ -195,20 +196,20 @@ namespace PocketSharp.Tests
     {
       DateTime since = DateTime.UtcNow;
 
-      List<PocketItem> items = await client.Get(state: State.all);
-      PocketItem itemToModify = items[0];
+      IEnumerable<PocketItem> items = await client.Get(state: State.all);
+      PocketItem itemToModify = items.First();
 
-      Assert.True(items.Count >= 3);
+      Assert.True(items.Count() >= 3);
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items == null || items.Count == 0);
+      Assert.True(items == null || items.Count() == 0);
 
       await client.AddTags(itemToModify, new string[] { "pocketsharp" });
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items.Count > 0);
+      Assert.True(items.Count() > 0);
 
       await client.RemoveTags(itemToModify, new string[] { "pocketsharp" });
     }
@@ -219,20 +220,20 @@ namespace PocketSharp.Tests
     {
       DateTime since = DateTime.UtcNow;
 
-      List<PocketItem> items = await client.Get(state: State.all);
-      PocketItem itemToModify = items[0];
+      IEnumerable<PocketItem> items = await client.Get(state: State.all);
+      PocketItem itemToModify = items.First();
 
-      Assert.True(items.Count >= 3);
+      Assert.True(items.Count() >= 3);
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items == null || items.Count == 0);
+      Assert.True(items == null || items.Count() == 0);
 
       await client.Favorite(itemToModify);
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items.Count > 0);
+      Assert.True(items.Count() > 0);
 
       await client.Unfavorite(itemToModify);
 
@@ -242,7 +243,7 @@ namespace PocketSharp.Tests
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items.Count > 0);
+      Assert.True(items.Count() > 0);
 
       await client.Unarchive(itemToModify);
     }
@@ -255,7 +256,7 @@ namespace PocketSharp.Tests
 
       PocketItem item = await client.Add(new Uri("http://frontendplay.com"));
 
-      List<PocketItem> items = await client.Get(state: State.all, since: since);
+      IEnumerable<PocketItem> items = await client.Get(state: State.all, since: since);
 
       since = DateTime.UtcNow;
 
@@ -263,7 +264,7 @@ namespace PocketSharp.Tests
 
       items = await client.Get(state: State.all, since: since);
 
-      Assert.True(items.Count == 1 && items[0].IsDeleted);
+      Assert.True(items.Count() == 1 && items.First().IsDeleted);
     }
 
     [Fact]
@@ -271,11 +272,11 @@ namespace PocketSharp.Tests
     {
       PocketItem item = await client.Add(new Uri("http://de.ign.com/m/feature/21608/die-20-besten-kurzfilme-des-jahres-2013?bust=1"));
 
-      List<PocketItem> items = await client.Get(state: State.all);
+      IEnumerable<PocketItem> items = await client.Get(state: State.all);
 
       Assert.NotNull(item.Uri);
-      Assert.NotNull(items[0].Uri);
-      Assert.Equal(item.Uri, items[0].Uri);
+      Assert.NotNull(items.First().Uri);
+      Assert.Equal(item.Uri, items.First().Uri);
 
       itemsToDelete.Add(item.ID);
     }

PocketSharp.Tests/MiscTests.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Threading.Tasks;
 using Xunit;
+using System.Linq;
 
 namespace PocketSharp.Tests
 {
@@ -19,8 +20,8 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task CheckPreRequestAction()
     {
-      List<PocketItem> items = await client.Get(count: 1);
-      PocketItem item = items[0];
+      IEnumerable<PocketItem> items = await client.Get(count: 1);
+      PocketItem item = items.First();
 
       await client.Favorite(item);
       await client.Unfavorite(item);

PocketSharp.Tests/ModifyTagsTests.cs

@@ -21,7 +21,7 @@ namespace PocketSharp.Tests
 
       item = await GetItemById(item.ID);
 
-      Assert.True(item.Tags.Count >= 2);
+      Assert.True(item.Tags.Count() >= 2);
 
       Assert.NotNull(item.Tags.Single<PocketTag>(tag => tag.Name == "test_tag"));
       Assert.NotNull(item.Tags.Single<PocketTag>(tag => tag.Name == "test_tag2"));
@@ -44,7 +44,7 @@ namespace PocketSharp.Tests
 
       item = await GetItemById(item.ID);
 
-      Assert.True(item.Tags.Count >= 2);
+      Assert.True(item.Tags.Count() >= 2);
 
       Assert.True(await client.RemoveTags(item));
 
@@ -63,7 +63,7 @@ namespace PocketSharp.Tests
 
       item = await GetItemById(item.ID);
 
-      Assert.Equal(item.Tags.Count, 2);
+      Assert.Equal(item.Tags.Count(), 2);
 
       Assert.NotNull(item.Tags.SingleOrDefault<PocketTag>(tag => tag.Name == "test_tag"));
       Assert.NotNull(item.Tags.SingleOrDefault<PocketTag>(tag => tag.Name == "test_tag2"));
@@ -85,7 +85,7 @@ namespace PocketSharp.Tests
 
     private async Task<PocketItem> GetItemById(string id, bool archive = false)
     {
-      List<PocketItem> items = await client.Get(state: archive ? State.archive : State.unread);
+      List<PocketItem> items = (await client.Get(state: archive ? State.archive : State.unread)).ToList();
       PocketItem itemDesired = null;
 
       items.ForEach(itm =>

PocketSharp.Tests/ModifyTests.cs

@@ -137,7 +137,7 @@ namespace PocketSharp.Tests
 
     private async Task<PocketItem> GetItemById(string id, bool archive = false)
     {
-      List<PocketItem> items = await client.Get(state: archive ? State.archive : State.unread);
+      List<PocketItem> items = (await client.Get(state: archive ? State.archive : State.unread)).ToList();
       PocketItem itemDesired = null;
 
       items.ForEach(itm =>

PocketSharp.Tests/StressTests.cs

@@ -30,50 +30,50 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task Are100ItemsRetrievedProperly()
     {
-      List<PocketItem> items = await client.Get(count: 100, state: State.all);
-      Assert.True(items.Count == 100);
+      IEnumerable<PocketItem> items = await client.Get(count: 100, state: State.all);
+      Assert.True(items.Count() == 100);
     }
 
     [Fact]
     public async Task Are1000ItemsRetrievedProperly()
     {
-      List<PocketItem> items = await client.Get(count: 1000, state: State.all);
-      Assert.True(items.Count == 1000);
+      IEnumerable<PocketItem> items = await client.Get(count: 1000, state: State.all);
+      Assert.True(items.Count() == 1000);
     }
 
     [Fact]
     public async Task Are2500ItemsRetrievedProperly()
     {
-      List<PocketItem> items = await client.Get(count: 2500, state: State.all);
-      Assert.True(items.Count == 2500);
+      IEnumerable<PocketItem> items = await client.Get(count: 2500, state: State.all);
+      Assert.True(items.Count() == 2500);
     }
 
     [Fact]
     public async Task Are5000ItemsRetrievedProperly()
     {
-      List<PocketItem> items = await client.Get(count: 5000, state: State.all);
-      Assert.True(items.Count == 5000);
+      IEnumerable<PocketItem> items = await client.Get(count: 5000, state: State.all);
+      Assert.True(items.Count() == 5000);
     }
 
     [Fact]
     public async Task IsSearchSuccessfullyOnBigList()
     {
-      List<PocketItem> items = await client.Get(search: "google");
+      IEnumerable<PocketItem> items = await client.Get(search: "google");
 
-      Assert.True(items.Count > 0);
-      Assert.True(items[0].Title.ToLower().Contains("google") || items[0].Uri.ToString().ToLower().Contains("google"));
+      Assert.True(items.Count() > 0);
+      Assert.True(items.First().Title.ToLower().Contains("google") || items.First().Uri.ToString().ToLower().Contains("google"));
     }
 
     [Fact]
     public async Task IsTagSearchSuccessfullyOnBigList()
     {
-      List<PocketItem> items = await client.SearchByTag(tags[0]);
+      IEnumerable<PocketItem> items = await client.SearchByTag(tags[0]);
 
-      Assert.True(items.Count > 1);
+      Assert.True(items.Count() > 1);
 
       bool found = false;
 
-      items[0].Tags.ForEach(tag =>
+      items.First().Tags.ToList().ForEach(tag =>
       {
         if (tag.Name.Contains(tags[0]))
         {
@@ -87,7 +87,7 @@ namespace PocketSharp.Tests
     [Fact]
     public async Task RetrieveTagsReturnsResultOnBigList()
     {
-      List<PocketTag> items = await client.GetTags();
+      List<PocketTag> items = (await client.GetTags()).ToList();
 
       items.ForEach(tag =>
       {

PocketSharp/Components/Get.cs

@@ -29,7 +29,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    public async Task<List<PocketItem>> Get(
+    public async Task<IEnumerable<PocketItem>> Get(
       State? state = null,
       bool? favorite = null,
       string tag = null,
@@ -58,9 +58,7 @@ namespace PocketSharp
         Offset = offset
       };
 
-      Retrieve response = await Request<Retrieve>("get", cancellationToken, parameters.Convert());
-
-      return response.Items;
+      return (await Request<Retrieve>("get", cancellationToken, parameters.Convert())).Items;
     }
 
 
@@ -74,12 +72,10 @@ namespace PocketSharp
     /// <exception cref="PocketException"></exception>
     public async Task<PocketItem> Get(string itemID, CancellationToken cancellationToken = default(CancellationToken))
     {
-      List<PocketItem> items = await Get(
+      return (await Get(
         cancellationToken: cancellationToken,
         state: State.all
-      );
-
-      return items.SingleOrDefault<PocketItem>(item => item.ID == itemID);
+      )).SingleOrDefault<PocketItem>(item => item.ID == itemID);
     }
 
 
@@ -90,7 +86,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    public async Task<List<PocketItem>> Get(RetrieveFilter filter, CancellationToken cancellationToken = default(CancellationToken))
+    public async Task<IEnumerable<PocketItem>> Get(RetrieveFilter filter, CancellationToken cancellationToken = default(CancellationToken))
     {
       RetrieveParameters parameters = new RetrieveParameters();
 
@@ -121,9 +117,7 @@ namespace PocketSharp
 
       parameters.DetailType = DetailType.complete;
 
-      Retrieve response = await Request<Retrieve>("get", cancellationToken, parameters.Convert());
-
-      return response.Items;
+      return (await Request<Retrieve>("get", cancellationToken, parameters.Convert())).Items;
     }
 
 
@@ -134,18 +128,19 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    public async Task<List<PocketTag>> GetTags(CancellationToken cancellationToken = default(CancellationToken))
+    public async Task<IEnumerable<PocketTag>> GetTags(CancellationToken cancellationToken = default(CancellationToken))
     {
-      List<PocketItem> items = await Get(
+      IEnumerable<PocketItem> items = await Get(
         cancellationToken: cancellationToken,
         state: State.all
       );
 
-      return items.Where(item => item.Tags != null)
-                  .SelectMany(item => item.Tags)
-                  .GroupBy(item => item.Name)
-                  .Select(item => item.First())
-                  .ToList<PocketTag>();
+      return items
+        .Where(item => item.Tags != null)
+        .SelectMany(item => item.Tags)
+        .GroupBy(item => item.Name)
+        .Select(item => item.First())
+        .ToList<PocketTag>();
     }
 
 
@@ -156,7 +151,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    public async Task<List<PocketItem>> SearchByTag(string tag, CancellationToken cancellationToken = default(CancellationToken))
+    public async Task<IEnumerable<PocketItem>> SearchByTag(string tag, CancellationToken cancellationToken = default(CancellationToken))
     {
       return await Get(
         state: State.all,
@@ -175,7 +170,7 @@ namespace PocketSharp
     /// <returns></returns>
     /// <exception cref="System.ArgumentOutOfRangeException">Search string length has to be a minimum of 2 chars</exception>
     /// <exception cref="PocketException"></exception>
-    public async Task<List<PocketItem>> Search(string searchString, string tag = null, CancellationToken cancellationToken = default(CancellationToken))
+    public async Task<IEnumerable<PocketItem>> Search(string searchString, string tag = null, CancellationToken cancellationToken = default(CancellationToken))
     {
       if (String.IsNullOrEmpty(searchString) || searchString.Length < 2)
       {

PocketSharp/Components/Modify.cs

@@ -18,7 +18,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    public async Task<bool> SendActions(List<PocketAction> actions, CancellationToken cancellationToken = default(CancellationToken))
+    public async Task<bool> SendActions(IEnumerable<PocketAction> actions, CancellationToken cancellationToken = default(CancellationToken))
     {
       return await Send(actions, cancellationToken);
     }

PocketSharp/IPocketClient.cs

@@ -127,7 +127,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    Task<List<PocketItem>> Get(
+    Task<IEnumerable<PocketItem>> Get(
       State? state = null,
       bool? favorite = null,
       string tag = null,
@@ -158,7 +158,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    Task<List<PocketItem>> Get(RetrieveFilter filter, CancellationToken cancellationToken = default(CancellationToken));
+    Task<IEnumerable<PocketItem>> Get(RetrieveFilter filter, CancellationToken cancellationToken = default(CancellationToken));
 
     /// <summary>
     /// Retrieves all available tags.
@@ -167,7 +167,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    Task<List<PocketTag>> GetTags(CancellationToken cancellationToken = default(CancellationToken));
+    Task<IEnumerable<PocketTag>> GetTags(CancellationToken cancellationToken = default(CancellationToken));
 
     /// <summary>
     /// Retrieves items by tag
@@ -176,7 +176,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    Task<List<PocketItem>> SearchByTag(string tag, CancellationToken cancellationToken = default(CancellationToken));
+    Task<IEnumerable<PocketItem>> SearchByTag(string tag, CancellationToken cancellationToken = default(CancellationToken));
 
     /// <summary>
     /// Retrieves items which match the specified search string in title and URI
@@ -187,7 +187,7 @@ namespace PocketSharp
     /// <returns></returns>
     /// <exception cref="System.ArgumentOutOfRangeException">Search string length has to be a minimum of 2 chars</exception>
     /// <exception cref="PocketException"></exception>
-    Task<List<PocketItem>> Search(string searchString, string tag = null, CancellationToken cancellationToken = default(CancellationToken));
+    Task<IEnumerable<PocketItem>> Search(string searchString, string tag = null, CancellationToken cancellationToken = default(CancellationToken));
     #endregion
 
     #region modify methods
@@ -199,7 +199,7 @@ namespace PocketSharp
     /// <param name="cancellationToken">The cancellation token.</param>
     /// <returns></returns>
     /// <exception cref="PocketException"></exception>
-    Task<bool> SendActions(List<PocketAction> actions, CancellationToken cancellationToken = default(CancellationToken));
+    Task<bool> SendActions(IEnumerable<PocketAction> actions, CancellationToken cancellationToken = default(CancellationToken));
 
     /// <summary>
     /// Archives the specified item.

PocketSharp/Models/Parameters/ModifyParameters.cs

@@ -16,6 +16,6 @@ namespace PocketSharp.Models
     /// The actions.
     /// </value>
     [DataMember(Name = "actions")]
-    public List<Dictionary<string, string>> Actions { get; set; }
+    public IEnumerable<Dictionary<string, string>> Actions { get; set; }
   }
 }

PocketSharp/Models/Parameters/RegisterParameters.cs

@@ -1,39 +0,0 @@
-using System.Runtime.Serialization;
-
-namespace PocketSharp.Models
-{
-  /// <summary>
-  /// All parameters which can be passed to register a user
-  /// </summary>
-  [DataContract]
-  internal class RegisterParameters : Parameters
-  {
-
-    /// <summary>
-    /// Gets or sets the username.
-    /// </summary>
-    /// <value>
-    /// The username.
-    /// </value>
-    [DataMember(Name = "username")]
-    public string Username { get; set; }
-
-    /// <summary>
-    /// Gets or sets the E-Mail.
-    /// </summary>
-    /// <value>
-    /// The E-Mail.
-    /// </value>
-    [DataMember(Name = "email")]
-    public string Email { get; set; }
-
-    /// <summary>
-    /// Gets or sets the password.
-    /// </summary>
-    /// <value>
-    /// The password.
-    /// </value>
-    [DataMember(Name = "password")]
-    public string Password { get; set; }
-  }
-}

PocketSharp/Models/PocketItem.cs

@@ -322,7 +322,7 @@ namespace PocketSharp.Models
     /// </value>
     [JsonProperty("tags")]
     [JsonConverter(typeof(ObjectToArrayConverter<PocketTag>))]
-    public List<PocketTag> Tags { get; set; }
+    public IEnumerable<PocketTag> Tags { get; set; }
 
     /// <summary>
     /// Gets or sets the images.
@@ -332,7 +332,7 @@ namespace PocketSharp.Models
     /// </value>
     [JsonProperty("images")]
     [JsonConverter(typeof(ObjectToArrayConverter<PocketImage>))]
-    public List<PocketImage> Images { get; set; }
+    public IEnumerable<PocketImage> Images { get; set; }
 
     /// <summary>
     /// Gets or sets the videos.
@@ -342,7 +342,7 @@ namespace PocketSharp.Models
     /// </value>
     [JsonProperty("videos")]
     [JsonConverter(typeof(ObjectToArrayConverter<PocketVideo>))]
-    public List<PocketVideo> Videos { get; set; }
+    public IEnumerable<PocketVideo> Videos { get; set; }
 
     /// <summary>
     /// Gets or sets the authors.
@@ -352,7 +352,7 @@ namespace PocketSharp.Models
     /// </value>
     [JsonProperty("authors")]
     [JsonConverter(typeof(ObjectToArrayConverter<PocketAuthor>))]
-    public List<PocketAuthor> Authors { get; set; }
+    public IEnumerable<PocketAuthor> Authors { get; set; }
 
     /// <summary>
     /// Gets the lead image.
@@ -363,7 +363,7 @@ namespace PocketSharp.Models
     [JsonIgnore]
     public PocketImage LeadImage
     {
-      get { return Images != null && Images.Count > 0 ? Images[0] : null; }
+      get { return Images != null && Images.Count() > 0 ? Images.First() : null; }
     }
 
     /// <summary>

PocketSharp/Models/Response/Retrieve.cs

@@ -36,6 +36,6 @@ namespace PocketSharp.Models
     /// </value>
     [JsonProperty("list")]
     [JsonConverter(typeof(ObjectToArrayConverter<PocketItem>))]
-    public List<PocketItem> Items { get; set; }
+    public IEnumerable<PocketItem> Items { get; set; }
   }
 }

PocketSharp/PocketClient.cs

@@ -8,6 +8,7 @@ using System.Net.Http;
 using System.Net.Http.Headers;
 using System.Threading;
 using System.Threading.Tasks;
+using System.Linq;
 
 namespace PocketSharp
 {
@@ -245,22 +246,13 @@ namespace PocketSharp
     /// </summary>
     /// <param name="actionParameters">The action parameters.</param>
     /// <returns></returns>
-    internal async Task<bool> Send(List<PocketAction> actionParameters, CancellationToken cancellationToken)
+    internal async Task<bool> Send(IEnumerable<PocketAction> actionParameters, CancellationToken cancellationToken)
     {
-      List<Dictionary<string, object>> actionParamList = new List<Dictionary<string, object>>();
-
-      foreach (var action in actionParameters)
-      {
-        actionParamList.Add(action.Convert());
-      }
-
       Dictionary<string, string> parameters = new Dictionary<string, string>() {{
-        "actions", JsonConvert.SerializeObject(actionParamList)                                                                    
+        "actions", JsonConvert.SerializeObject(actionParameters.Select(action => action.Convert()))                                                                    
       }};
 
-      Modify response = await Request<Modify>("send", cancellationToken, parameters);
-
-      return response.Status;
+      return (await Request<Modify>("send", cancellationToken, parameters)).Status;
     }
 
 

PocketSharp/PocketSharp.csproj

@@ -63,7 +63,6 @@
   <ItemGroup>
     <Compile Include="Components\Statistics.cs" />
     <Compile Include="IPocketClient.cs" />
-    <Compile Include="Models\Parameters\RegisterParameters.cs" />
     <Compile Include="Models\PocketBoolean.cs" />
     <Compile Include="Models\PocketLimits.cs" />
     <Compile Include="Models\PocketStatistics.cs" />

PocketSharp/Utilities/JsonExtensions.cs

@@ -123,17 +123,17 @@ namespace PocketSharp
 
 
 
-  public class ObjectToArrayConverter<T> : CustomCreationConverter<List<T>> where T : new()
+  public class ObjectToArrayConverter<T> : CustomCreationConverter<IEnumerable<T>> where T : new()
   {
     public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
     {
       JObject jObject;
-      List<T> result = new List<T>();
-      T target;
+      List<T> results = new List<T>();
+
       // object is an array
       if (reader.TokenType == JsonToken.StartArray)
       {
-        return serializer.Deserialize<List<T>>(reader);
+        return serializer.Deserialize<IEnumerable<T>>(reader);
       }
       else if (reader.TokenType == JsonToken.Null)
       {
@@ -152,11 +152,12 @@ namespace PocketSharp
       // Populate the object properties
       foreach (KeyValuePair<string, JToken> item in jObject)
       {
-        target = serializer.Deserialize<T>(item.Value.CreateReader());
-        result.Add(target);
+        results.Add(
+          serializer.Deserialize<T>(item.Value.CreateReader())
+        );
       }
 
-      return result;
+      return results;
     }
 
     public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
@@ -164,11 +165,10 @@ namespace PocketSharp
       throw new NotImplementedException();
     }
 
-    public override List<T> Create(Type objectType)
+    public override IEnumerable<T> Create(Type objectType)
     {
       return new List<T>();
     }
-
   }