I assume the shake you eat in the morning is a store-bought protein shake.
That's big mistake number one. Nothing good will come from that. The only good thing is that it's easy.
Mistake number 2: All the animal protein in your salad. I'm sure it tastes very good but it's not doing you any good.
Your dinner could be okay but I doubt it because a lot depends on how much of each item you eat.
Most overweight people don't really like fresh fruit, a wide variety of fresh non-starchy vegetables, legumes/lentils, non-roasted, non-salted nuts and seeds (in reasonable portions).
The secret to attaining a healthy weight is to like the right foods. For example, you would be better off to include beans instead of animal protein in your noontime salad. Do you like beans? Then you could include a reasonable amount of chopped nuts - the kind I mentioned above.
For dinner you should eat (cooked) vegetables, vegetables, vegetables as much as you can possibly fit in your stomach. Then at the end of the meal, when you're already full, you can eat a small portion of meat - maybe 2 ounces. If you feel that the meat is not enough protein, include a serving of beans or lentils with your vegetables.
You need at least 25 to 30 grams of fiber per day. If you're not getting that much it's very likely that your diet will fail at some point.
The last thing is: never take advice from other overweight people because it's likely that they don't like vegetables and will steer you in the wrong direction.