C Stringuntersuchung Problem

[Acer (Gast)]

Hallo zusammen,
ich habe ein Problem mit meinem C Programm.
Der String "proge" soll mittels Unterprogramm `read` untersucht werden.
z1 t1 z2 ... dienen der Deklaration der darauffolgenden Zahl z.B.
z1 entspricht 20
t1 entspricht 30

Das Unterprogramm findet die Werte für z1,t1,z2 heraus.
Allerdings nicht mehr für t2.

Kann mir jemand helfen?
Danke im voraus.

Gruß Acer





//.............//
int read(char tz);      //untergrogramm T bzw. Z Werte auslesen
int i=0, j=1;
char 
proge[100]="z120;t130;z210;t2100;z3500;t31000;z4800;t4900;z51200;t5400;\ 
0";
//-------------------------------------------------------//
int main(void)
{
  UART0_Init(9600);    //Initialisierung der seriellen Schnittstelle




  int z1 = read('z');
  int t1 = read('t');
  j++;
  int z2 = read('z');
  int t2 = read('t');


    printf("%d %d %d %d \r\n", z1,t1,z2,t2);


  return 0 ;
}
//-------------------------------------------------------------------//
int read(char tz)          //Unterprogramm Einlesen der Zeit und 
Tempwerte aus proge
{
  int ret;
  char z1[8];
  int posz1=0;
    if ((proge[i]=tz) && (proge[i+1]=j))
    {

      i=i+2;
    for (;proge[i] != ';';)
      {
      z1[posz1] = proge[i];
      posz1++;
      i++;
      }
    if (proge[i] == ';')
      {
      z1[i] = '\0';
      ret = atoi(z1);
      i++;
      }
    }
    return (ret);
}

[Peter II (Gast)]

proge[i]=tz

sicher das du nicht

proge[i] == tz

meinst?

[Rene B. (themason)]

@Otto

Hat man dir auch so einen Vorschlag gemacht als du mit C angefangen hast 
und das ein oder andere Problem gehabt hast ?!

[us73 (Gast)]

Vielleicht solltest Du Vergleiche mit "==" durchführen statt mit "="....

Siehe: "proge[i]=tz"

[Rene B. (themason)]

@Acer

Ein problem ist wie schon angesprochen der Vergleich (== und nicht =) 
und zum zweiten willst du mit "proge[i+1] == j" auf eine Zahl 
vergleichen was aber nicht klappt da '1' ASCII-mäßig eben nicht 1 
sondern 0x31 ist. Du müsstest also sowas wie "proge[i+1] == (j + 0x30)" 
verwenden.

[Peter II (Gast)]

so der nächste fehler

proge[i+1]=j oder besser proge[i+1]==j

macht überhaupt keinen sinn, weil in proge[i+1] ein zeichen und in j 
eine zahl steht.

und '1' ist nunmal nicht 1

wenn überhaupt dann so:

proge[i+1] == j + '0'

[Yalu X. (yalu) (Moderator)]

Solche einfachen Parsereien kann man auch gut mit sscanf machen, etwa
so:

#include <stdio.h>

char proge[] = "z120;t130;z210;t2100;z3500;t31000;z4800;t4900;z51200;t5400;";
char *progeptr;

int read(char tz, int j)
{
  char tz1;
  int n, j1, ret, nread;

  n = sscanf(progeptr, "%c%1d%d;%n", &tz1, &j1, &ret, &nread);
  progeptr += nread;
  if(tz1 != tz || j1 != j || n != 3) 
  {
    // Fehler im String
    ret = -1;
  }
  return ret;
}

int main(void)
{
  progeptr = proge;

  int z1 = read('z', 1);
  int t1 = read('t', 1);
  int z2 = read('z', 2);
  int t2 = read('t', 2);

  printf("%d %d %d %d\r\n", z1, t1, z2, t2);

  return 0 ;
}

Ausgabe:

20 30 10 100