Salvation was cack as the t800 had about 4 occasions where John Connor was in its hands only to prefer to lob him around instead of pulling his head out of his arse, it's whole mission is to take him out so it just came off as stupid that it would prefer to throw him around. The t600 earlier in the film had grabbed him twice, yet again just decided to throw him around. The other issue with that movie was the extreme lack of any actual terminators, i think at most there were about 2 of the human styled robots on screen at once and that was only when the t800 destroyed the malfunctioning t600. For a future to be ruled by robots there sure as hell wasn't many of them around. The cg was pretty good, especially on the robot t800 when it had its skin blasted off, overall though it felt like a very empty movie with not a whole lot going on.
The whole skynet plan in Salvation was to get Marcus to lure Connor into the skynet base, yet only a few hours after Marcus is activated a T600 tries to kill him, with the worlds most inaccurate minigun it has to be said. And when Connor finally gets to a skynet robot creation base he has to contend with.....a single t800 and that's it. Didn't help matters you seen a lot of endoskeletons basically on racks, yet for some reason in skynets grand plan only 1 was activated, and this comes after the skynet ai admitted they had failed to kill Connor numerous times. derp
A movie set in the future war is what people want though, instead of this convoluted "remake t2" thing that seems to be what t3 and genisys were based on. Funny thing is the movie makers are aware of people wanting this which is apparently one of the reasons genisys had a chunk being set in the future war before going back to a variation of T2. Apparently setting the entire movie in the future war as depicted in the flashbacks is something they're not comfortable doing.