Hilfe bei einem C-Programm fuer einen ASCII Art Dekodierer

[ektoplasma (Gast)]

Hallo Liebe Forumuser,

ich habe eine Aufgabe fuer die Uni zu bewaeltigen und stecke fest.

ich soll aehnlich wie in

http://cboard.cprogramming.com/c-programming/56439-ascii-art-decoder-encoder.html

einen Dekodierer auf C Basis erstellen, der mir solch verschluesselten 
Dateien dekodiert.

Das Programm Soll:
- Benutzer nach Dateinamen fragen.
- Zeilen dieser Datei als Datensätze des dargestellten Kodierers 
interpretieren
- Die Daten dekodieren und gibt die verschluesselten Daten dann 
entsprechend ihrer Anzahl und Reihenfolge am Bildschirm aus, damit man 
das urspruengliche Bild sehen kann.

Danach die Ausgabe nicht nur auf dem Bildschirm sondern zusätzlich in 
eine TXT-Datei schreiben.

Die Elemte der Syntax duerfen durch beliebig viel Leerraum getrennt 
sein. Dazu zaehlen:
- Leerzeichen
- Tabulatorzeichen
- Neue Zeile Zeichen

Da dann mit der ctype.h Funktion isspace zur Erkennung von 
Leerraumzeichen arbeiten.

[ektoplasma (Gast)]

Mein Code sieht folgendermaßen aus:

/*

    fp = fopen("abc.txt", "w");  //Datei im write modus geöffnet -> wird automatisch erzeugt.


    fputs( //zaehler * //zeichen, fp);


    //Ausgeben von dem Zeichen * Zaehler auf der Console:

    int p = 0;

    while (p < //zaehler, p++){

        printf("%c", //zeichen);

    }



    fclose(fp);

}*/


int i = 1, b, zn= 0;
char array[];
int main ()
{

   FILE *input, *output;
   char* z; //Anzahl Zeichen
   char* c, *cn = ""; //Zeichen
   char BREITE, name_input[255], name_output[255];

   printf("Nennen sie den Dateinamen : ");
   scanf("%s",name_input);
   input=fopen(name_input,"r");
   if(input == NULL) {
      printf("%s Datei konnte nicht geoeffnet werden.\n",name_input);
      return EXIT_FAILURE;
   }
   /*else {
      printf("Nennen Sie einen Dateinamen zum speichern : ");
      scanf("%s",name_output);
      output=fopen(name_output,"w+");
      if(output==NULL)  {
         printf("Datei konnte nicht gespeichert werden.\n");
         return EXIT_FAILURE;
      }*/

    fgets(array, 1200, input); //array erzeugen.
    //erste zeile
    char* e = strtok(array,"=;"); //initialiserung, welche char* array
    e = strtok(NULL,"=;"); //bereits initialisiert

    int y = 0;
    b=0;
    for(y; y<sizeof(e)-2; y++){   //Berechnung Breite
        b+=((int)e[y]-'0');
        b=b*10;
    }
    b = b/10; //Breite
    int boolean = 1;
    while(fgets(array, 1200, input)){
        z ='\0';
        z = strtok(array, ("' \n\t"));
        //z ="12";
        printf("%s", z); //unsere Zahl vor dem Zeichen
        c = '\0';
        c = strtok(NULL, ("'"));
        printf("%s", c); //unser zeichen nach zahl
        for(y = 0; y < sizeof(z)-2; y++){   //Berechnung Breite -2 wegen \0
            zn+=((int)z[y]-'0');
            zn=zn*10;
        }
        zn = zn/10; //Breite
        printf("zn %d",zn);
        char test[10] = "abc";
        strncat(test,c,3); //Zusammensetzen
             printf("%s",test);
       /* for(zn; zn>0; z--){

        }*/
    }

       /* while(boolean == 1){
        z = strtok(NULL, ("' \n\t"));
        printf("%s",z);
        c = strtok(NULL, ("'"));
        printf("%s", c);
        boolean = 0;
}*/
        /*z = strtok(array, ("' \n\t"));
        while(z != NULL){
            printf("%s", z);
            c = strtok(NULL, ("'"));
            printf("%s", c);
        }*/

     return EXIT_SUCCESS;
   }

[ektoplasma (Gast)]

Tut mir leid fuer das Durcheinander, aber ich bin noch ein Anfaenger.
Es sind noch viele printf's im Programm zur Fehlersuche und ein Test zur 
Stringverkettung.

Meine Ausgabe auf der Konsole ist sehr wirr und aber hier meine erste 
Zeile der Ausgabe.

Nennen sie den Dateinamen : asterix.caa
8 zn 32abc 24 zn 344abc 3 zn 3422abc 14 zn 34234abc 3 zn 342322abc 81 zn 
3423301

Der erste Part stimmt. 8 und " " ein Leerzeichen als Zeichen, jedoch 
weiss ich nicht wieso er dann bei 32 ankommt und der Zahlenwert immer 
laenger wird. Stimmt was mit dem /0 nicht?

[Max H. (hartl192)]

ektoplasma schrieb:
> char array[];
Das wird so nichts, woher soll der Compiler wissen, wie viel Speicher er 
für dein Array reservieren soll?

> char* e = strtok(array,"=;"); //initialiserung, welche char* array
>     e = strtok(NULL,"=;"); //bereits initialisiert
>
>     int y = 0;
>     b=0;
>     for(y; y<sizeof(e)-2; y++){   //Berechnung Breite
sizeof(e) liefert dir hier die Größe in Bytes des Pointers und nicht die 
Länge des Strings, dafür ist strlen da.

Beschäftig dich lieber mal mit den Grundlagen.


ektoplasma schrieb:
> Tut mir leid fuer das Durcheinander
Was schon mal viel zu Lesbarkeit beiträgt sind die Code Tags:

[c]C-Code[/c]

[Steffen M. (ektoplasma)]

Stimmt, das habe ich ganz vergessen.

Also reserviere ich mir mit malloc einfach meinen Speicher?

array = (int *) malloc(size * sizeof(int));

[Max H. (hartl192)]

Steffen M. schrieb:
> Also reserviere ich mir mit malloc einfach meinen Speicher?
> array = (int *) malloc(size * sizeof(int));
Dann würde ich array aber mit

char *array;

deklarieren.
Den Pointer würde ich auch nach (char*) und nicht nach (int*) casten und 
mit der Größe size * sizeof(char) allokieren. Nicht vergessen, den 
Speicher am Ende wieder freizugeben.

Wenn du auch mit statischer Größe zurechtkommst, würde ich auf das 
malloc verzichten.

[Daniel A. (daniel-a)]

Für Stringverarbeitung ist c ziemlich unpraktisch. Ich würde in c eine 
statemachine dafür verwenden.

Hier mal ein getestetes JavaScript beispiel:

javascript:void ((function(){
function draw(x){
  var res = x.match(/([^\n]+)=([^\n]+);\n((.|\n)*)/m);
  if(!res)
    return '';
  var v = res[1];
  var n = res[2];
  var data = res[3].match(/(\d+)\w*'(.)'/g);
  var s = '';
  for(var i=0,j=0;i<data.length;i++){
    var m = data[i].match(/(\d+)\w*'(.)'/);
    if(!m) continue;
    var len = m[1];
    var ch = m[2];
    while(len--){
      s += ch;
      if(!((j+++1)%n))
        s+="\n";
    }
  }
  return s;
}
document.body.style.whiteSpace="pre";
document.body.style.fontFamily="monospace";
document.body.style.fontSize="11px";
document.body.innerText=draw(document.body.textContent||document.body.text||document.body.innerText);
})());

Aber das Bild ist gut gelungen, sehr gut!

[Steffen M. (ektoplasma)]

Ja, leider muessen wir das in C machen. Da ich auch im Stoff hinterher 
hinke, bin ich ziemlich ratlos wie ich das anstellen soll.

[Steffen M. (ektoplasma)]

Max H. schrieb:
> Steffen M. schrieb:
>> Also reserviere ich mir mit malloc einfach meinen Speicher?
>> array = (int *) malloc(size * sizeof(int));
> Dann würde ich array aber mit

char *array;

deklarieren.
> Den Pointer würde ich auch nach (char*) und nicht nach (int*) casten und
> mit der Größe size * sizeof(char) allokieren. Nicht vergessen, den
> Speicher am Ende wieder freizugeben.
>
> Wenn du auch mit statischer Größe zurechtkommst, würde ich auf das
> malloc verzichten.

Soll ich dann einfach mit MAX Größe initialisieren?

[Max H. (hartl192)]

Steffen M. schrieb:
> Soll ich dann einfach mit MAX Größe initialisieren?
Wenn du eine sinnvolle MAX Größe angeben kannst, dann ja.

[foobar (Gast)]

Sowas in der Uni?!?

Naja, für schlecht gestellte Aufgaben gibt's auch schlechte Lösungen:

/* quick'n'dirty .caa interpreter - copyright [e-mail] */
#include <stdio.h>

int main(int argc, char **argv)
{
    int b = 80;
    int x = 0;
    int c;

    while ((c = getchar()) != EOF)
        if (c >= '0' && c <= '9')
        {
            int n = 0;

            do
                n = 10*n + c-'0';
            while ((c = getchar()) >= '0' && c <= '9');

            if (c == ';')
                b = n;
            if (c == '\'')
            {
                c = getchar();
                while (n-- > 0)
                {
                    putchar(c);
                    if (++x >= b)
                        putchar('\n'), x = 0;
                }
            }
        }
    putchar('\n');
    return 0;
}

Benutzung: ./a.out <asterix.caa
bzw: ./a.out <asterix.caa >ausgabe.txt

Dies soll nur ein Beispiel sein, wie man sowas auf die einfachste Weise 
implementieren kann (nicht sollte!).  Auch wenn man den Parser schöner 
macht, sind komplexe String-Funktionen oder dynamische 
Speicherverwaltung nicht nötig.  Btw, Kommentare habe ich mit Absicht 
weggelassen - klamüser selbst raus, wie/warum das funktioniert, wo 
Probleme auftreten und mach es dann besser ;-)

[Daniel A. (daniel-a)]

Hier noch eine Lösung in c von mir im Anhang.

Kompilieren mit:
gcc -Wall -Werror -Wextra -pedantic -std=c99 main.c -o main

Verwendung:
main < asterix.caa

[Robert L. (lrlr)]

man muss nur dumm genug fragen, dann macht einem schon irgend ein Depp 
die Hausaufgaben, oder was ist das hier ...

;-)

[Ben (Gast)]

foobar schrieb:
> Sowas in der Uni?!?

Eher für eine TH, ehemals HAW. Die Aufgabe kommt mir bekannt vor...

[Steffen M. (ektoplasma)]

Ich bin jetzt doch drauf gekommen. Etwas einfacher als die von dir 
Daniel.

Trotzdem vielen Dank an euch alle fuer eure Hilfe.