Try posting the actual code that the compiler is given rather than just the parts. If the problem was that obvious, you would ahve already found it.

THe default SD library is built on top of an old version of the Sdfat library. The SD library limits the number of open files to 1 as far as I can tell. The latest Sdfat library (rather than the old one distributed with sd library) only limits the number of open files basaed on the RAM available. see http://arduino.cc/forum/index.php/topic,51087.0.html for example.

fatlib16 is the author of sdfat. sdfatlib20120719 is the latest version as far as I can tell.

Try using arrays rather than having 5 similar named variables.

eg.
#DEFINE ARRAYSIZE 5
int Finger[ARRAYSIZE]= {0,1,2,3,4};

byte servoValue[ARRAYSIZE];

Then you can loop over the array rather than having multiple lines with just slightly different variable names.

for (byte loopCount =0; loopCount < ARRAYSIZE; loopCount++)
   {
   servoValue[loopCount] = Function(FInger[loopCount]);
   }

It will also help you spot all those unused variables. It is much easier to declare all the variables at the start of apiece of code rather than some at the start and some part way through.

Edit - OK Tom beat me to it but the same idea.

Are you sure the line
MagX = Wire.read()<<8; //X msb
is working as you expect. The byte needs to be cast to an int, is this done before the shift or after? If before the cast then the values you are getting will all be zero. Might be worth putting a debug line in to check the values read for MagX make sense (also MagY and Z)

What sort of Arduino are you running this on? I ask as the Uno only has 2K memory iirc and you use 1514 for a buffer plus whatever the w5100 uses along with the various other arrays and variables you have.

When you only show two pieces of code that don't appear to have the error in it makes it very difficult to guess what he problem is - try posting ALL the code!

Are you sure you're reading the values in the same order you are writing them?

You need to post the whole code, especially if you are wanting to keep values in variables where teh definition of those variables and the location cannot be seen. It may be that you want to define them as static, or make them global, or defined at a differerent context.

If I understand the original question, it relates more as to where to store the correction factors (or whatever you prefer to call them). Assuming they do not change very often, for ease you could store them as an array in progmem http://www.arduino.cc/en/Reference/PROGMEM or in eeprom http://www.arduino.cc/en/Reference/EEPROM

You should have an OR in your while loop not an AND, so continue looping until both are zero.

You may also want to make sure the value does not drop below 0 when you leave the loop.

Sounds like a similar probem to :

http://arduino.cc/forum/index.php/topic,116837.0.html

The Arduino doesn't do rs232.

If you want any help, I suggest you indicate what the problem is, how it occurs, and what if anything you have tried.

Youare writing to add1 in both loops of your code, so the second loop overwrites the contents written by the first loop and the second address will be blank. Try altering your code to

Code:

  for(int i = 0; i < 16; i++)
  {
    Wire.beginTransmission(add1);
    Wire.send(i);
    Wire.send(dataku1[i]);
    Wire.endTransmission();
    delay(100);
  }
 
  Serial.println("Write EEPROM 2"); 
  for(int i = 0; i < 16; i++)
  {
    Wire.beginTransmission(add2);    //Previously add1
    Wire.send(i);
    Wire.send(dataku2[i]);
    Wire.endTransmission();
    delay(100);

2730 * 12 = 32760  which is the highest value you can get below 32767. 32767 being the highest value in a 16bit unsigned integer.

If you want to go further than record 2730 you will need to use a datatype larger than 16bit surely?

Surely you should null terminate the string, otherwise the receiving routine won't know how long the string is and will go through memory until it hits a null.