HIGHSCORE, BITTE JEDER DER AHNUNG HAT KURZ MAL REIN SCHAUEN, BRAUCHE DAS MEGA SCHNELL

[Konfusius]

dein code ist soo haarsträubend umständlich, daß die einzige möglichkeit eines korrekturvorschlags das posten eines neuen codes ist:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct highpl
{
  int  punkt;
  char name[4]; /* soll der name nur 3 zeichen lang sein, */
                /* dann brauchts auch nicht mehr platz */
}; 

void
highscore_lesen (struct highpl *highpl)
{
  FILE *pfile;
  int i;

  pfile=fopen("C:\\system.zrx","r"); 

  for(i=0; i<8; i++)
    fscanf(pfile,"%i %s",&highpl[i].punkt,highpl[i].name); 

  fclose(pfile); 
}

void
highscore_schreiben (struct highpl *highpl)
{
  FILE *pfile;
  int i;

  pfile=fopen("C:\\system.zrx","w"); 

  for(i=0; i<8; i++)
    fprintf(pfile,"%i %s\n",highpl[i].punkt,highpl[i].name); 

  fclose(pfile); 
}

void
highscore_einfuegen (struct highpl *highpl, int punkt, char *name)
{
  int pos,i;

  /* bestimme position, an der der neue highscore eingefügt werden soll */
  for(pos=0; pos<8; pos++)
    if(punkt>=highpl[pos].punkt)
      break;

  if(pos==8) return; /* highscore kleiner als letzter platz => keine änderung */

  /* schiebe highscores nach unten */
  for(i=7; i>pos; i--)
  {
    highpl[i].punkt=highpl[i-1].punkt;
    strcpy(highpl[i].name,highpl[i-1].name);
  }

  /* füge neuen highscore ein */
  highpl[pos].punkt=punkt;
  strcpy(highpl[pos].name,name);
}

void
highscore_eingeben (int punkte) 
{ 
  struct highpl highpl[8];
  char name[256];

  highscore_lesen(highpl);

  if(punkte>=highpl[7].punkt) 
  { 
    printf("NEUER HIGHSCORE!\n\n"); 
    printf("Ihre Punktzahl ist: %i\n",punkte); 

    printf("Ihr Name (max. 3 Zeichen): "); 
    scanf("%s",name);
    name[3]='\0'; /* auf 3 zeichen kürzen */
    printf("\n\n"); 

    highscore_einfuegen(highpl,punkte,name);
    highscore_schreiben(highpl);
  } 
} 

void
highscore_ausgeben (void)
{ 
  struct highpl highpl[8]; 
  int i;

  highscore_lesen(highpl);

  for(i=0; i<8; i++)
    printf(" %i. %i Pkt %s\n",i+1,highpl[i].punkt,highpl[i].name); 
}

im wesentlichen ist das das selbe wie dein code nur mit arrays, schleifen un unterprogrammen.

[Master-of-Scare]

Thx a lot,

ist es eigentlich auch möglich das die zeichen und punkte irgendwie verschlüsselt werden, so das man die in der datei net einfach so umschreiben kann, zb das man den asci hoch zählt? und wenn ja, wie mach ich daS?

[Trolljaeger]

Da seht ihr, was man davon hat, wenn man Noobs das "goto" beibringt

[Master-of-Scare]

Trolljaeger schrieb:

Da seht ihr, was man davon hat, wenn man Noobs das "goto" beibringt

naja, groß töne spucken kannst ja, aber auch was anderes? zeig lieber mal das du es besser kannst, ich lerne erst seit paar wochen c...

[PAD]

1. -Verschlüsseln oder besser verschleiern. Alle Werte for dem Schreiben mit 0xff xoren
2. - Anstelle mit 0xff xor mit einer beliebigen festen Zeichenfolge xoren

3.-im struct ein Feld anhängen in dem ein CRC32 über die Daten drin ist, wenn einer pfuscht muß er das erstens wissen das CRC gesichert zweitens wie er gebildet wird.

4. - Nächster Schritt zusätzlich den gesamten Datenbestand mit CRC32 absichern.

5. - High Score und zweier Komplement abspeichern. Wer fälschen will muss die
wissen und beides ändern

6. - Den Inhalt der struct als array of byte auffassen und die Daten mit einem festen muster vertauschen

7.- Orginal reihenfolge 123456789
vertauschung 192837465
Das vertausche Schreiben.

8. - mit pgp-methodik verschlüsseln und ablegen

9. - eine beliebige perverse kombination aus den dingen da oben.

reicht das für den Anfang

[nman]

Doppelposts sind verdammt unhöflich.

[Master-of-Scare]

sorry @ mod, aber wusste net wo ich das genau reinsetzen muss um schnelle antwort zu erhalten

[Master-of-Scare]

super, thx, noch kurze frage, wie mach ich das

[Daniel E.]

Das kannst Du bei google recherchieren und geht deutlich schneller als hier auf die Antwort zu warten.

Ein einfaches a XOR schluessel bekommst Du mit 'a ^ schluessel'. Das reicht eigentlich für den Anfang, so top secret wird deine Highscore kaum sein.

[mantiz]

hm, also, ich will ja jetzt nix sagen, gerade weil ich auch noch sehr neu auf dem Gebiet C/C++ bin und auch ziemlich neu hier im Forum, aber für mich sieht das ziemlich so aus, dass Du nen komplett fertigen Code haben willst, den Du dann am Ende wahrscheinlich noch nicht einmal verstehst. Du bist noch am Anfang, ist ja auch OK, aber frag' doch dann nicht nach einer kompletten Lösung.

Master-of-Scare schrieb:

... das man entweder übernehmen oder daraus lernen (Anm.: oder teilweise klauen?) kann ...

... Falls ihr was habt könnt ihr auch gerne mailen: ...

... zeig lieber mal das du es besser kannst, ich lerne erst seit paar wochen c...

Wie gesagt, will mich nicht zu weit aus dem Fenster lehnen, aber für mich liest sich das so ziemlich nach einer Forderung (einem Wunsch) nach einer Komplettlösung und nicht nach einer Lösung eines Problems, wo man durch Tips (passiert ja immer wieder, dass man nicht alles weiß (mir auch sehr, sehr häufig)) das Problem selbst in den Griff bekommt.

[Master User]

Vileicht mit Huffman irgentwie verschluseln?. Damit du eine lange string mit 1 und 0 bekommst das keiner lessen kan.

[Master-of-Scare]

mantiz schrieb:

hm, also, ich will ja jetzt nix sagen, gerade weil ich auch noch sehr neu auf dem Gebiet C/C++ bin und auch ziemlich neu hier im Forum, aber für mich sieht das ziemlich so aus, dass Du nen komplett fertigen Code haben willst, den Du dann am Ende wahrscheinlich noch nicht einmal verstehst. Du bist noch am Anfang, ist ja auch OK, aber frag' doch dann nicht nach einer kompletten Lösung.

Master-of-Scare schrieb:

... das man entweder übernehmen oder daraus lernen (Anm.: oder teilweise klauen?) kann ...

... Falls ihr was habt könnt ihr auch gerne mailen: ...

... zeig lieber mal das du es besser kannst, ich lerne erst seit paar wochen c...

Wie gesagt, will mich nicht zu weit aus dem Fenster lehnen, aber für mich liest sich das so ziemlich nach einer Forderung (einem Wunsch) nach einer Komplettlösung und nicht nach einer Lösung eines Problems, wo man durch Tips (passiert ja immer wieder, dass man nicht alles weiß (mir auch sehr, sehr häufig)) das Problem selbst in den Griff bekommt.

Nope, so war das absolut net gemeint, hatte nur das verdammte problem das ich egal wie ich das durchgedacht hatte nie ne schleife rein bekommen hab. die lösung von konfusius hab ich erst mal studiert, hab daran jedoch noch einiges dann geändert da ich das mit einigen anderen ideen noch verbunden habe, mit kopieren war das so gemeint, falls ich das dann absolut net hin bekomm...

[Korbinian]

Master User schrieb:

Vileicht mit Huffman irgentwie verschluseln?. Damit du eine lange string mit 1 und 0 bekommst das keiner lessen kan.

jemand der noch nicht mal arrays einsetzen kann, sollte auf dem teppich bleiben, einfaches xor ist angesagt glaub ich. sonst wird die highscore komplizierter als das programm

[Konfusius]

anstatt die highscoredaten gleich mit fprintf() in die datei zu schreiben, kannst du sie erst mit sprintf() in einen char-puffer schreiben. dort verschlüsselst du sie dann und schreibst das ergebnis in die datei.

zum verschlüsseln würde ich auch xor nehmen. mache einfach buf[i]=buf[i]^0xAA für jedes zeichen im puffer. anstelle von 0xAA kannst du auch eine andere zahl von 0 bis 255 nehmen. du mußt aber aufpassen die zahl so zu wählen, daß kein reguläres zeichen dabei null wird. sonst wird das als stringende interpretiert und das wird dir beim anschließenden schreiben mit fputs() probleme machen. am besten nimm zahlen >127.

das entschlüsseln geht übrigens genauso. xor ist nämlich selbstinvers. wenn du eine zahl zwei mal mit der selben zahl xor'st, dann kommt wieder das selbe raus wie am anfang.

am besten schreibst du dir eine verschlüsselungs-funktion, die als argument den zu verschüsselnden string bekommt. die kannst du dann überall da aufrufen, wo du sie brauchst.

@alle
ich gebs ja zu. ich bin zu noob-freundlich. das fertigcode-posten ist vielleicht doch nicht die beste idee. nem alten c-programmierer wie mir fällts halt leichter, sich in code auszudrücken als in natürlicher sprache. ich gelobe hiermit, mich zu bessern.

aber jeden noob anzuschnauzen, er solle erst mal informatik studieren bevor er die unverschämtheit hat sich in diesem forum zu melden, ist auch nicht der weisheit letzter schluß.

Master-of-Scare schrieb:

naja, groß töne spucken kannst ja, aber auch was anderes? zeig lieber mal das du es besser kannst

daran, daß er es besser könnte als du, besteht kein zweifel. und daß hier nicht jeder fertigen code posted, liegt daran, daß noobs ja auch was lernen sollen.

[Master-of-Scare]

ich werde den code ja auch net so übernehmen, aber auf die art und weiße habe ich auch schon HTML gelernt, ich brauche Quellcode um das zeug zu lernen, die highscore muss ich eh noch um einiges dann umschreiben da ich ganz ehrlich nicht die arbeit von einem anderen möchte... nur ich muss sehen wie das funktioniert um es dann selber zu machen... THX A LOT

PS mir ist auch klar das er es besser kann, aber dafür das ich c erst seit etwa 8 oder 9 wochen etwas genauer behandel ist auch klar das ich net das vernetzte denken bisher habe, jedoch will ich das schnell lernen. Und ich bleibe auch net bei c, will noch java, java script, php, flash und sql sowie cgi im verlauf dieses jahres lernen, was nicht so viel ist...

[PAD]

@Konfusius
Ich glaube es gibt eine günstiger Lösung. Mach eine union mit 2 elementen, das eine ist die Struct das andere eine array of byte mit der gleichen Länge wie die struct.
Die Daten in die Struct kopieren und als Array of Byte abholen. ich spreche hier von einem Array of Byte und nicht von einem String alle alle Zeichen auch die \0 an jeder Stelle möglich sind. Die Länge ist fest und bekannt.
Auf dies Array of Byte kann ich jedes beliebige schleierverfahren (siehe oben) anwenden und es dann anschließend mit write binaer in eine File schreiben.

Lesen geht dann genau andersrum.
read aus dem file
entschleiern
in union kopieren
als struct abholen

Das array of byte wird genauso angelegt wie ein string am besten mit

unsigned char Testarry[24];

Alles weiter kann man hier erfragen oder in jedem C-Buch nachlesen. :p

[Konfusius]

PAD schrieb:

Ich glaube es gibt eine günstiger Lösung.

ich finde deinen ansatz auch gut. er ließe sich aber noch weiter vereinfachen. anstatt eine union zu verwenden kann man den zeiger auf die struktur einfach auf (char*) casten.

bliebe nur noch, eine funktion *verschluesseln(char daten, int anz) zu schreiben. dabei muß aber unbedingt darauf geachtet werden, daß keine bytes !=0 auf 0 abgebildet werden und umgekehrt. sonst würde die ende-markierung der strings in der struktur zertört werden und sie könnte nicht mehr mit fprintf() gespeichert werden. das ließe sich beim xor-algorithmus erreichen, indem man nur bytes xor't die sowohl !=0 als auch !=schlüssel sind.

ich kanns nicht lassen und muß jetzt doch wieder mal code posten

const int schluessel=0xAA;

void
verschluesseln (char *daten, int anz)
{
  while(anz--)
    if(daten[anz]!=0 and daten[anz]!=schluessel)
      daten[anz]=daten[anz]^schlüssel;
}

der algorithmus ist selbstinvers, dh das entschlüsseln erfolgt durch einen weiteren aufruf von verschluesseln().

Master-of-Scare schrieb:

mir ist auch klar das er es besser kann

eigentlich war mein post auch ein bißchen selbstkritisch gemeint. ich wollte damit sagen, daß ich mich nicht für einen besseren programmierer halte, nur weil ich mehr code poste als andere. mir macht das coden einfach nur spaß. und wenn andere weniger code posten, dann deshalb, weil sie wollen, daß die anfänger hier im forum was lernen. ich denke aber auch, daß ein stück code einem anfänger häufig mehr sagt als seitenlanges fachchinesisch

[PAD]

@Konfusius
Das ist der Grund warum ich hier nicht mit einem null Terminierten C-String
sondern mit einem Array of Byte wie in Pascal da gehört nämlich noch die Länge dazu.

verschluesseln (char *daten, int anz)
{
for (i=0;i<anz;i++)
daten[i]=daten[i]^schlüssel;
}

MfG
PAD

[Konfusius]

@PAD
hmmm... das haut aber nicht hin. denn mit der ganzen struktur verschlüsselst du ja auch die längenangabe!

[Master-of-Scare]

ok, das:

const int schluessel=0xAA; 

void 
verschluesseln (char *daten, int anz) 
{ 
  while(anz--) 
    if(daten[anz]!=0 and daten[anz]!=schluessel) 
      daten[anz]=daten[anz]^schlüssel; 
}

klingt echt super und ist unkompiziert, eine kurze frage dazu, was ist in dem fall wo du mir zb die eine highscore geschrieben hast daten und was anz?