I need help with arduino shield ethernet

 String dataString = "";

Don't go there.

Data written to a file on an SD card is already buffered. Storing all the data in a resource wasting String does not help performance.

I see nothing in that code that does anything with any web page. It does not generate a page. It does not request a page.