Advising the people around us is a good thing to do. However, there are certain guidelines our deen mentions when advising someone. Especially when a person has done something wrong, these guidelines should be followed to ensure the person you are advising takes it in the best way possible.
Shaykh Saad Tasleem sheds some more light on the question
When you see someone doing something wrong, how do you go about advising them?
One of the first things to keep in mind is that advice must be given personally and privately. What we mean by that is personally meaning you don't make a status about them on Facebook or put them on glass on Twitter or anything like that. You don't convey the message through people, you don't talk to other people about it. It has to be done personally meaning you yourself, talk to them, and has to be done privately. Meaning you don't sit amongst people and say, Hey, you did this thing which is wrong, and I disagree and so on and so forth. You take them aside, and you give them advice in private, and that what that serves to do is that lets their guard down and their ego is not a factor
in that case. And that advice would be more effective. And the last parameter Allah knows best