Did anyone notice during the duel which harry and sirius were fighting side by side, that sirius said "good one james" instead of harry?

that was supposed to happen.
in the books Molly Weasley is always saying that she thinks Sirius sometimes thinks Harry is James..like he's soo much like his father, he remembers all the good times with James and thinks he's with him.
so they applied that to the movie.
they kinda threw it in there, and if you've read the books you would understand that.

Didn't it also turn up earlier in the movie, in the scene from Snape's memory? It's not in that scene in the book, but I thought it was a nice touch to put it in there, suggesting that it was something Sirius said a lot. In the DoM scene in the movie, I can't make up my mind whether Sirius actually thought for a moment that it was James (as Molly thought he was doing earlier), or it was his way of saying Harry was like his Dad.

Well, either way Harry took it as a compliment because he was smiling haha.

Yes I did and I think that Harry noticed too.

I don't remember that..But it sounds familiar. I really don't remember

Maybe this is a reference to how Sirius and James treated Snape? I'm sure they said "nice one" at some point for bullying Snape.
The obvious thing is that Sirius views Harry almost like a brother, like James, and in the heat of the battle he feels he has his best friend back, as Sirius and James both loved combat.

I remember this, because I just recently saw the movie But I think it's kind of sad that the last thing Harry ever hears from Sirius is "good one James" .

I think it's sad that just before Sirius is killed Harry hears him say he's just like his father when he exclaims "Nice one, James!"