Nick Roberts

My feedback

  1. 241 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 » IDE and Editor  ·  Flag idea as inappropriate…  ·  Admin →
      Nick Roberts supported 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 » User Interface  ·  Flag idea as inappropriate…  ·  Admin →
        Nick Roberts shared this idea  · 
      • 143 votes
        Vote
        Sign in
        Check!
        (thinking…)
        Reset
        or sign in with
        • facebook
        • google
          Password icon
          Signed in as (Sign out)
          You have left! (?) (thinking…)
          8 comments  ·  Visual Studio IDE » .NET  ·  Flag idea as inappropriate…  ·  Admin →
          Nick Roberts supported this idea  · 
        • 330 votes
          Vote
          Sign in
          Check!
          (thinking…)
          Reset
          or sign in with
          • facebook
          • google
            Password icon
            Signed in as (Sign out)
            You have left! (?) (thinking…)
            under review  ·  28 comments  ·  Visual Studio IDE » Languages - Visual Basic  ·  Flag idea as inappropriate…  ·  Admin →
            Nick Roberts commented  · 

            Just to note that I largely accept Pavinll's comments.

            Nick Roberts commented  · 

            Hmmm. In your example (perhaps this is not fair) I would rather write:

            Structure Point
            Public X As Integer
            Public Y As Integer
            Public Sub New(X As Integer, Y As Integer)
            Me.X = X
            Me.Y = Y
            End Sub
            End Structure
            Structure Size2D
            Public Width As Integer
            Public Height As Integer
            Public Sub New(Width As Integer, Height As Integer)
            Me.Width = Width
            Me.Height = Height
            End Sub
            End Structure
            Public Function CanDoSomething(TopLeft As Point, Size As Size2D, Value As Integer, Name As String, Count As Integer) As Boolean
            ...
            End Function

            Or better still:

            Public Function CanDoSomething(TopLeft As Point, -- top left corner of box
            Size As Size2D, -- width and height of box
            Value As Integer, -- median value expected
            Name As String, -- name of division to search
            Count As Integer) -- number of sub-boxes within box
            As Boolean

            In other words, I don't think this suggestion is likely to be genuinely useful very often.

            Occasionally it might be. E.g. a function that computes SHA takes eight parameters the same type (if I remember correctly).

          Feedback and Knowledge Base