Nick Roberts

My feedback

  1. 241 votes
    Vote
    Sign in
    (thinking…)
    Sign in with: facebook google
    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  · 
  2. 1 vote
    Vote
    Sign in
    (thinking…)
    Sign in with: facebook google
    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  · 
  3. 143 votes
    Vote
    Sign in
    (thinking…)
    Sign in with: facebook google
    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  · 
  4. 330 votes
    Vote
    Sign in
    (thinking…)
    Sign in with: facebook google
    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