arduino clone

i had a look at some other arduino clone circuits and i designed my own basic arduino clone the circuit is attached below as i am new to all this i am asking to see if it will work i think it will but i am not sure

It looks ok.
You will need to add a 6-pin header to connect to an FTDI breakout, for uploading sketches.

He does have an FTDI header. Not organized to accept an FTDI Basic as is, but easily re-organized. Needs a 0.1uF cap between the reset on the connector pin and the reset pin. Having the 0.1uF caps across the VCC, AVCC, and AREF pins to gnd also helps to show that decoupling is understood.

Just the de-coupling caps, other than that it looks good.

is this now ok

Change the header order like this:
Gnd
Gnd
Vcc (+5)
Rx [ edit - swapped Rx & Tx - these reflect the uC pins]
Tx
DTR (just rename it, goes to the reset capacitor)

Then put 3.3V on it somewhere if you want.
This way you can plug on an FTDI Basic for downloading sketches.

ok i will do that and can i use a ATmega328 on it

Yes and yes.

thanks

when programming it with a USB to TTL converter does the TXD pin on the converter go to pin 3 on the ATmega328 (TXD pin on the chip)

These pins in this order:

Gnd
Gnd
Vcc (+5)
Rx [ edit - swapped Rx & Tx - these reflect the uC pins]
Tx
DTR

reflect the uC names.
The FTDI will then connect its Rx to the uC Tx, and its Tx will connect to uC Rx.