well the white with black outline is much easier to read, but it still seems like a pretty lackluster design, IMO.
is the assignment to just use one photo as a background and then your information on top, like you have it? or was it "create a business card" ?
I feel like it would be much more professional if you ghosted the photograph and then laid out your information properly on top, rather than handing in a 100% photo with your information scrunched in the corner like an afterthought...
A business card is meant to be handy for your contact information. It seems to me that right now your main focus is that picture, which it shouldn't be. Yes, you are a photographer and that is what your business card says, but if they want to see sample work, then they should head over to a website or portfolio.. this business card shouldn't serve that purpose.