correct way to do this with asp.net-mvc? list of comment on an item

Question

you have some info on an item and you got a list of comment on that item

how would you manage add/delete/edit with the route?

for now I have this:

/item/details/7/editcomment/1
/item/details/7/delcomment/1
/item/details/7/addcomment

is that the correct way?

eu-ge-ne · Accepted Answer · 2009-05-20 19:12:52Z

4

/comment/edit/{commentid}
/comment/delete/{commentid}
/comment/add/{itemid}

edited May 20, 2009 at 19:12

answered May 20, 2009 at 19:04

eu-ge-ne

28.2k6 gold badges73 silver badges62 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Mike Robinson Over a year ago

Although I'd remove the itemid altogether - it's redundant

Fredou Over a year ago

I think I agree with you, having a big route create an ugly controller

user434917 · Accepted Answer · 2009-05-20 18:34:46Z

1

I suggest you remove the "details":

/item/7/editcomment/1
/item/7/delcomment/1
/item/7/addcomment

answered May 20, 2009 at 18:34

user434917

Comments

User · Accepted Answer · 2009-05-20 19:25:10Z

0

I'd like to point out the for security reasons you'd rather not use this approach for deletion of items. Always use POST to perform any modification actions. Otherwise someone can try to abuse the system by inserting an image with a deletion link.

Of course if you just redirect users to confirmation page and they have to click submit button to initiate operation that would be okay.

1 Comment

Fredou Over a year ago

I'm using a confirmation page like it was suggested in the dinner book

Collectives™ on Stack Overflow

correct way to do this with asp.net-mvc? list of comment on an item

3 Answers 3

2 Comments

Comments

1 Comment

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

1 Comment

Related