Dylan Nicholson

My feedback

  1. 5,999 votes
    Vote
    Sign in
    Check!
    (thinking…)
    Reset
    or sign in with
    • facebook
    • google
      Password icon
      Signed in as (Sign out)
      You have left! (?) (thinking…)
      124 comments  ·  Visual Studio IDE » Languages - C#  ·  Flag idea as inappropriate…  ·  Admin →
      Dylan Nicholson supported this idea  · 
    • 221 votes
      Vote
      Sign in
      Check!
      (thinking…)
      Reset
      or sign in with
      • facebook
      • google
        Password icon
        Signed in as (Sign out)
        You have left! (?) (thinking…)
        13 comments  ·  Visual Studio IDE » Languages - C#  ·  Flag idea as inappropriate…  ·  Admin →
        Dylan Nicholson commented  · 

        readonly products = GetProductList();
        getOrders().ForEach(o => o.UpdateProductsInformation(products));

        There's no guarantee the actual items in the products list won't get changed - just that 'products' is still referring to the same list, so the use of 'readonly' here would potentially be misleading. The guarantee that products still points to the same list is given by the C# convention of pass by value.

        Having said that I do like the idea of local readonly "named values" ('variables' would be seem to be an obvious misnomer - I gather they're called 'bindings' or bound identifiers in F#).

      • 6 votes
        Vote
        Sign in
        Check!
        (thinking…)
        Reset
        or sign in with
        • facebook
        • google
          Password icon
          Signed in as (Sign out)
          You have left! (?) (thinking…)
          0 comments  ·  Visual Studio IDE » Debugging and Diagnostics  ·  Flag idea as inappropriate…  ·  Admin →
          Dylan Nicholson shared this idea  · 
        • 1 vote
          Vote
          Sign in
          Check!
          (thinking…)
          Reset
          or sign in with
          • facebook
          • google
            Password icon
            Signed in as (Sign out)
            You have left! (?) (thinking…)
            0 comments  ·  Visual Studio IDE » Languages - C#  ·  Flag idea as inappropriate…  ·  Admin →
            Dylan Nicholson shared this idea  · 

          Feedback and Knowledge Base