Arrays Vergleichen

vergleiche doch einfach byteweise:

for(unsigned char i=0; i < 16; i++)
{
    if(data[i] != old_data[i])
    {
         send(data);
         break;
    }
}

Zwei Arrays sind gleich, wenn jede einzelne Zelle gleich ist. Du 
könntest eine Schleife machen, die bei jedem Durchgang jede Zellen mit 
dem gleichen Index vergleicht.
ungefähr so:

1

int i;

2

int sendok;

3

for (i=0;i<data.length;i++){

4

if (data_old[1]!= data[1]){

5

int sendok = 0;

6

i=data.length;}

7

else sendok = 1;}

8

9

if (sendok)

10

send(data);

Oder so ähnlich

Sorry, natürlich so:

1

unsigned char i;

2

unsigned char sendok;

3

for (i=0;i<data.length;i++){

4

if (data_old[i]!= data[i]){

5

sendok = 0;

6

i=data.length;}

7

else sendok = 1;}

8

9

if (sendok)

10

send(data);

if (!memcmp(data,olddata,16))
 {
    send(data);
    memcpy(olddata,data,16);
 }

Gruss Helmi

memcpy und auch memcmp sind Teile der Standart C Bibilothek
meistens sind sie in der "string.h" enthalten

Du uebergibst nicht das Array sondern einen Pointer auf das Array.
Die angabe von 16 in meinem Code heisst das die beiden Arrays 16Byte 
lang sind -> 128 Bit Daten.

Du musst in deinem Programm das Header File string.h inkludieren.

#include <string.h>

Gruss Helmi

Opps ich hatte einen kleinen Fehler in dem Programm
nimm mal das '!' zeichen vor dem memcmp weg.

memcpy b.z.w. memcmp funktioniert auf byte (char) ebene


if (memcmp(data,olddata,16))
 {
    send(data);
    memcpy(olddata,data,16);
 }



gruss helmi

Caste die Varibeln mal mit (char*)

da könnte deine Fehlermeldung herkommen

if (memcmp((char*)data,(char*)olddata,16))
 {
    send(data);
    memcpy((char*)olddata,(char*)data,16);
 }

Gruss Helmi

Naja, mit

((Pin_input)<<i)

findet ja auch kein Portzugriff statt. Es wird vom Preprozessor 
erweitert zu

(((PINC,PINC0))<<i)

Der C-Compiler nimmt dann PINC als Ergebnis dieses Ausdrucks und schiebt 
den entsprechend.

Schreib doch einfach mal:

if ((PINC & (1<<PINC0)) != 0)
    Data_to_send[a] |= 1<<i;

Wenn es aber wirklich Bit 0 von PORTC ist, hast Du Glück und könntest 
einfacher schreiben:

Data_to_send[a] |= (PINC & 1)<<i;

Du hast Dir etwas ausgedacht, was der Compiler übersetzt, aber nicht das 
macht, was Du willst :-)

Du wolltest eine Abfrage, die Dir 1 liefert, wenn Dein Bit auf 1 ist, 
ansonsten auf 0. Das geht so aber nicht. Dein #define müßte lauten:

#define Pin_Input ((PINC & (1<<PINC0)) != 0 ? 1 : 0)

Reichlich unübersichtlich....

Ja, probier's mal. Deine Code läßt sich übrigens perfekt mit 
for-Schleifen realisieren:

unsigned char a,i;
for (a=0; a<16; a++)
{
  Data_to_send[a] = 0; // Annahme: Alles ist 0
  for (i=0; i<8; i++)
  {
    PORTC |= (1<<Input_Clock);
    _delay_us(10);
    PORTC &= ~(1<<Input_Clock);
    Data_to_send[a] >>= 1;  // Platz schaffen für's nächste Bit
    if (PINC & (1<<PINC0))
      Data_to_send[a] |= 0x80; // Ggf. Bit setzen
  }
}

Der Typ von Data_to_send muss ein unsigned char sein (nicht signed), da 
sonst beim Rechtsschieben das Vorzeichen erweitert wird. Das gibt 
schönen Datenmüll.

Dein Eingang darf nun auch wieder auf einem beliebigen Bit vom Port 
liegen.

Eine fiese Version einer for-Schleife wäre noch:

unsigned char a,i;
for (a=0; a<16; a++)
{
  Data_to_send[a] = 0;     // Annahme: Alles ist 0
  for (i = 1; i; i <<= 1)  // Wir schicken ein Bit durch ein Byte,
                           // bis es hinten heraus fällt
  {
    PORTC |= (1<<Input_Clock);
    _delay_us(10);
    PORTC &= ~(1<<Input_Clock);
    if (PINC & (1<<PINC0))
      Data_to_send[a] |= i; // Ggf. Bit setzen
  }
}

So etwas läßt sich gerade für so eine Anwendung wie Deine sehr oft 
gebrauchen.