Verschlüsslungs Programm Funktioniert nicht richtig C++

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Verschlüsslungs Programm Funktioniert nicht richtig C++

    Ich hab ein problem mit einem Programm das ich geschrieben hab bin ein Blutiger anfänger vllt kann mir wer Helfen(ist mein erster Thread hier;)
    Ich hab ein problem mit einem Programm das ich geschrieben hab bin ein Blutiger anfänger vllt kann mir wer Helfen(ist mein erster Thread hier;)

    C-Quellcode

    1. [syntax="cpp"]
    2. #include <fstream>
    3. #include <iostream>
    4. using namespace std;
    5. int main(int argc, char *argv[]){
    6. //voreinstellung
    7. string text;
    8. fstream f;
    9. f.open("text.txt", ios::out);
    10. f.close();
    11. char auswahl;
    12. char exit1;
    13. char exit2;
    14. char textwahl;
    15. char schluesselcode;
    16. //voreinstellung
    17. mark1:;
    18. cout << "Text Verschluesseln(1)" <<endl;
    19. cout << "Text Entschluesseln(2)" <<endl;
    20. cin >> auswahl;
    21. switch (auswahl)
    22. {
    23. mark2:;
    24. case ('1'):
    25. cout << "Sie haben die 1 fuer Verschluesseln gedrueckt, wollen sie fortfahren?(J/N)" <<endl;
    26. cin >> exit1;
    27. switch (exit1)
    28. {
    29. case('N'):
    30. case('n'):
    31. cout << "Abbrechen" <<endl;
    32. goto mark1;
    33. break;
    34. default:
    35. cout << "Falsche eingabe";
    36. goto mark2;
    37. break;
    38. case('J'):
    39. case('j'):
    40. f << endl;
    41. f.close();
    42. cout << "Fuegen sie in dieses Dokument ihren Text ein(maximal 512 zeichen)" <<endl;
    43. cout << "und geben sie nun einen 4 Stelliges Passwort ein"<<endl;
    44. mark3:;
    45. cin >> schluesselcode;
    46. cin.get();
    47. float schluesselcodeedit;
    48. int tausend =1000;
    49. int eins =1;
    50. char a;
    51. schluesselcode/=tausend%=eins=schluesselcodeedit;
    52. if (schluesselcodeedit==1)
    53. {
    54. fstream f;
    55. char cstring[1024];
    56. f.open(argv[1], ios::in);
    57. while (!f.eof())
    58. {
    59. f.getline(cstring, sizeof(cstring));
    60. char *a;
    61. while((a = strchr(cstring, ' '))!=NULL)*a = '_';
    62. //GORßBUCHSTABEN
    63. char *c;
    64. while((c = strchr(cstring, 'A'))!=NULL)*c = '1,'; //A
    65. char *b;
    66. while((b = strchr(cstring, 'B'))!=NULL)*b = '2,'; //B
    67. }
    68. f.close();
    69. f << cstring;
    70. }
    71. else{
    72. cout << "Das war Nicht Vierstellig" <<endl;
    73. cin.get();
    74. cin.get();
    75. goto mark3;
    76. }
    77. }
    78. }
    79. return.0;
    80. getchar();
    81. }
    82. [/syntax]
    Alles anzeigen

    Ist noch nicht ganz fertig sollte aber eigentlich in einem txt Dokument was vorher vom Programm erstellt wird aus dem eingefügten text die A's durch 1, B durch 2, und die Leerzeichen durch einen Unterstrich ersetzen stattdessen kann die .exe Datei nicht mehr ausgeführt werden nachdem das Programm bei Zeile 66 ist. Auserdem sind beim Compilieren noch ein paar Fehler auf getreten:
    In function `int main(int, char**)':
    65 [Warning] converting to `int' from `float'

    84:63 [Warning] multi-character character constant

    86:63 [Warning] multi-character character constant

    231 [Warning] converting to `int' from `double'

    wäre nett wenn ihr mir helfen könntet
  • Okay, blutiger Anfänger ;)

    Hab den Quelltext nur überflogen.
    Irgend etwas ist bei deiner Quelltextformatierung schief gelaufen. Versuche das in den Griff zu bekommen, dann kann man den Quelltext meist leichter lesen und verstehen.

    Die Zeilennummern des Quelltextes und die der Fehlermeldungen passen nicht zueinander.

    Dann Zeile 9 öffnest du eine Datei um die sofort wieder zu schliessen? Warum nicht eine Aktion mit der Datei?
    Dann sind da noch weitere Ungereimtheiten bei deinem Dateihandling.
    Kann auch sei das ich mit deinem goto - Gefrickel den Überblick verloren habe. Statt des goto gehen da auch normale Schleifen.
    Packe nicht den ganzen Quelltext unter main - lagere den in Funktionen aus. Z.B. einmal Menu und dann Verschlüsselung.

    Dann mischt du auch noch C und Cpp wahllos. Wenn du C++ programmieren möchtest, nimm auch möglichst nur C++ String-Funktionen.

    Hoffe ich habe dich jetzt nicht verschreckt :D

    Nachtrag: welche IDE mit welchem Compiler nutzt du?

    MfG bcc-fan

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von bcc-fan ()

  • Danke für die Antwort das mit den Zeilen ist mir grad auch aufgefallen, ich hatte einige herauskommentierte befehle weggelassen wegen der Übersichtlichkeit. ;)
    Ich benutze Dev-C++ 4.9.9.2
    Das mit dem auslagern der Funktionen werde ich machen ist dann bestimmt einfacher zu lesen :D
    Das ich C und C++ Vermisch habe war mir garnicht bewusst ich hab ein kleines Vorwissen von C++ durch Tutorials und hab die anderen befehle gegoogled
    muss wohl irgendwie falsche Informationen bekommen haben:)
    Und nein du hast mich nicht verschreckt:D

    Nachtrag: Wäre eine andere IDE besser?

    mfg Lennart
  • Die Version des DEV die du hast der 4.9.9.2 ist ja schon ein wenig älter und der Compiler da drunter wahrscheinlich der alte MinGW 3.4.2 oder 3.4.5.

    Hier jemand der versucht das mit aktuellen Mitteln weiter zu entwickeln:
    orwelldevcpp.blogspot.de/
    Oder eine andere Gruppe:
    codeblocks.org/
    Wenn du Code::Blocks 10.05 mit Compiler MinGW 4.4.1 lädst aktualisiere den sofort :!: Beispiel hier:
    tdm-gcc.tdragon.net/

    Es gibt noch weitere IDE Projekte.

    Einfachen Code schreibe ich meist auf dem Editor SciTE. Auch aus dem kann man Compiler ansteuern.
    Wenn ich mehr Komfort will z.Z. Code::Blocks10.05 mit MinGW 4.6.1 oder auch 4.7.0. Nicht alle MinGW 4.7.0 sind für Anfänger einfach zu installieren. Der MinGW aus dem Link oben sollte relativ einfach an laufen zu bekommen sein.

    cstring ( Zeile 58 ) wird schon in den String-Headern definiert - keine gute Idee einen C-String so zu benennen.

    MfG bcc-fan

    Dieser Beitrag wurde bereits 3 mal editiert, zuletzt von bcc-fan ()

  • Hab mal schnell etwas zusammen genagelt:

    Quellcode

    1. // zum testen einfache Datei erstellen ;)
    2. #include <fstream>
    3. #include <iostream>
    4. #include <string>
    5. void codieren(std::string txt)
    6. {
    7. std::cout << "Fuegen sie in dieses Dokument ihren Text ein(maximal 512 zeichen)\n"; // stand halt noch im Editor ;)
    8. std::cout << "und geben sie nun einen 4 Stelliges Passwort ein"<<std::endl;
    9. std::ifstream ifstr(txt); // oeffnet Datei
    10. std::string buffer;
    11. while(std::getline(ifstr, buffer))
    12. std::cout << "Zeichen: " << buffer.length() << std::endl; // keine Ahnung ob diese Passage hier benötigt wird
    13. std::cout << "Dateiinhalt: " << buffer << std::endl;
    14. // hier das mit dem Codieren ergänzen ...
    15. }
    16. int main(int argc, char *argv[])
    17. {
    18. char auswahl;
    19. do
    20. {
    21. std::cout << "c = Text Verschluesseln(1)\n";
    22. std::cout << "d = Text Entschluesseln(2)\n";
    23. std::cout << "q = Programm beenden" <<std::endl;
    24. std::cin >> auswahl;
    25. switch (auswahl)
    26. {
    27. case ('c'):
    28. if(argc)
    29. codieren(argv[1]);
    30. break;
    31. case ('d'):
    32. if(argc)
    33. // decodieren(argv[1]); // [1] oder [2] ?
    34. break;
    35. case('Q'):
    36. case('q'):
    37. auswahl = 'q';
    38. break;
    39. default:
    40. std::cout << "Falsche eingabe";
    41. }
    42. }
    43. while(auswahl != 'q');
    44. }
    Alles anzeigen


    Ist nur damit man den C++ Stil erkennen kann. Ist nicht perfekt. Ich würde noch daran schrauben. Sollte aber schon laufen.
    Wenn du dich über einige "fehlende" Zeilen wunderst, fragen warum die da in C++ nicht stehen 8)

    MfG bcc-fan

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von bcc-fan ()