Ok, first of all i'd like to point out some 'rules' of design.... firstly it doesn't matter if your card is portrait or landscape, so dont worry about that. Second even if your cards arnt 'standard business card size' that doesnt matter either... i know dozens of pro wedding photogs who deliberatly have different shaped cards.... to make you stand out, and to just be different.... there is no rule saying you have to stick with the standard format.
Next, absolutly have a picture on your business card.... hell yes.... pictures speak a thousand words.... i would very much discourage anyone, esecially a photographer to go for a plain typography based card.... that would be very odd indeed. So by all means have pics on cards.
However... im not too keen on the using a whole image as the background.... whenever i design cards, i try to place images on the card seperatly from the text, it just looks neater i think, and also doesnt make the text difficult to read.
I dont think you've done a bad job here for a first effort, but typography is always the hardest thing for an amature to get right, so dont feel bad about it!