Arduino + 7 Segment Display [Feedback Needed]

Thanks for the reply.

Even I was confused whether it has a common cathode or common anode. The datasheet says it is common anode, but I am not very sure. Is there a way by which I can check whether it is common anode or cathode?

Regarding the current limiting resistors, I will remove one of them.