Wie Klasse optimieren? Update:Klasse komplett, paar Fehler n

  • Wie Klasse optimieren? Update:Klasse komplett, paar Fehler n

    Hallo,

    ich habe eine Klasse geschrieben welche eine Textdatei ausliest und die Strings zeilenweise in einen Stringvector legt. Auf diesen Stringvector sollen nun verschiedene Funktionen zugreifen können:

    void viewline():

    Soll bestimmte Zeilen anzeigen lassen, arbeitet ok.


    void removeline()

    Soll bestimmte Zeilen löschen, durch das mehrmalige Durchlaufen der for-Schleife "rutschen" die Elemente nach und so ist das Ergebnis nicht mehr das gewünschte, wie könnte ich das optimieren?


    void addline()

    Soll Zeilen am Ende des Vector anfügen... leider klappt das auch hier nur sporadisch, und ich weiß nicht nach welchem Prinzip (Zeilen werden unvollständig oder gar nicht in den Stingvector anfgefügt). Wo ist hier die Inkonsistenz?

    allgemein: wie kann ich eine bessere Abruch-Bedingung schreiben? bei den ersten beiden Funktionen soll man mit Enter die Texteingabe beenden.

    Hier nun der Code meiner Klasse:

    Quellcode

    1. #include "AsciiEditor.h"
    3. #include <fstream>
    4. #include <iostream>
    5. #include <vector>
    6. #include <string>
    7. #include <iterator>
    9. class Editor
    10. {
    11. public:
    12. typedef std::vector<std::string> linevector;
    14. //Textdatei einelesen
    15. void einlesen()
    16. {
    17. std::ifstream ifs("C:/asci1.txt");
    18. std::string tmp;
    20. linevector::iterator li; //LineIterator
    22. while(getline(ifs, tmp))
    23. {
    24. lv.push_back(tmp);
    25. tmp = std::string();
    26. }
    28. }
    30. //Bestimmte Zeilen ansehen
    31. void viewline()
    32. {
    33. typedef std::vector<int> intvector;
    34. intvector intv;
    35. intvector::const_iterator inti;
    36. std::cout<<"Welche Zeilen sollen ausgegeebn werden?: \n\n";
    37. int eingabe;
    39. while (std::cin >> eingabe && eingabe != 999 )
    40. {
    41. intv.push_back(eingabe);
    42. }
    44. linevector::iterator li;
    45. for (inti=intv.begin(); inti!=intv.end(); inti++)
    46. {
    47. li = lv.begin() + *inti; //li LineIterator, lv Linevector von außen
    48. std::cout << *li << std::endl;
    49. }
    50. }
    52. //Bestimmte Zeilen entfernen
    53. void removeline()
    54. {
    55. typedef std::vector<int> intvector;
    57. intvector intv;
    58. intvector::const_iterator inti;
    59. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    60. int eingabe;
    62. while (std::cin >> eingabe && eingabe != 999 )
    63. {
    64. intv.push_back(eingabe);
    65. }
    67. linevector::iterator li;
    68. for (inti=intv.begin(); inti!=intv.end(); inti++)
    69. {
    70. li = lv.begin() + *inti; //li LineIterator, lv Linevector von außen
    71. lv.erase(li);
    72. }
    74. for (li=lv.begin(); li!=lv.end(); li++)
    75. {
    76. std::cout << *li << std::endl;
    77. }
    78. }
    80. //Zeilen hinzufügen
    81. void addline()
    82. {
    83. std::cout<<"Zeilen hinzufuegen: \n\n";
    85. std::string eingabe;
    86. std::string tmp;
    88. while(getline(std::cin >> eingabe, tmp)&& eingabe != "end!")
    89. {
    90. lv.push_back(tmp); //lv Linevector von außen
    91. tmp = std::string();
    92. }
    94. linevector::iterator li;
    95. for (li=lv.begin(); li!=lv.end(); li++)
    96. {
    97. std::cout << *li << std::endl; //li LineIterator, lv Linevector von außen
    98. }
    99. }
    101. private:
    102. linevector lv; //Linevector
    103. };
    106. int main(){
    108. Editor edit;
    109. edit.einlesen();
    110. edit.viewline();
    111. edit.removeline();
    112. edit.addline();
    113. return 0;
    114. }
    Alles anzeigen


    Ich würde mich echt freuen wenn ihr nen paar (oder alle?) meiner Fragen beantworten könnt..

    vielen Dank schonmal und viele Grüße...

    Julia :P

  • Hallo Julia,
    für removeline bietet es sich an die zu löschenden Indexe in nem set zu speichern weil sie darin dann sortiert vorliegen.Oder du sortierst halt den vector :wink: .
    Auf jeden Fall kannst du dann mit dem größten Lineindex anfangen und dich dann zum kleinsten vorarbeiten.
    Ohne es jetzt großartig getestet zu haben:

    Quellcode

    1. //Bestimmte Zeilen entfernen
    2. void removeline()
    3. {
    4. typedef std::set<int> index_set;
    5. index_set indexes;
    7. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    8. int eingabe;
    10. while (std::cin >> eingabe && eingabe != 999 )
    11. indexes.insert(eingabe);
    13. linevector::iterator li;
    14. index_set::const_reverse_iterator idx_rit;
    15. for (idx_rit=indexes.rbegin(); idx_rit!=indexes.rend(); ++idx_rit)
    16. {
    17. li = lv.begin() + *idx_rit; //li LineIterator, lv Linevector von außen
    18. lv.erase(li);
    19. }
    20. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    21. }
    23. //Zeilen hinzufügen
    24. void addline()
    25. {
    26. std::cout<<"Zeilen hinzufuegen: \n\n";
    27. std::string eingabe;
    29. while(std::getline(std::cin,eingabe)&& eingabe != "end!")
    30. lv.push_back(eingabe); //lv Linevector von außen
    32. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    33. }
    Alles anzeigen

    Generell solltest du in deinem Programm darauf achten ob die eingegebenen Indexe kleiner als lv.size() sind.

    Edit:Sehe gerade dass du noch mehr Fragen hattest....naja,morgen ist ja auch noch nen Tag :) :wink:

    Gruß void
    "Probleme kann man niemals mit derselben Denkweise lösen,
    durch die sie entstanden sind." (A. Einstein)

  • hey void :)

    danke für deine Hilfle.. es funktioniert auf jeden Fall schon beser :)

    Ich hab nun auch die Abfrage nach der Größe mit eingebaut...

    Das Ding.. wenn ich zB "1 2 3 4 5 6 7" zum löschen angebe und dann mit 999 die Abbruchbedingung erfülle zeigt er gar nichts mehr an :( Woran liegt? Und wie kann ich Enter als Abbruchbedingung festlegen? Hab schon mit /n rumprobiert..

    Quellcode

    1. //Bestimmte Zeilen entfernen
    2. void removeline()
    3. {
    4. typedef std::set<int> index_set;
    5. index_set indexes;
    7. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    8. int eingabe;
    9. const unsigned int lvsize = lv.size();
    11. while (std::cin >> eingabe && eingabe != 999 ){
    13. if (eingabe > lvsize)
    14. {
    15. std::cout<< eingabe << " liegt ausserhalb des Bildes ";
    16. }
    18. indexes.insert(eingabe);
    20. linevector::iterator li;
    21. index_set::const_reverse_iterator idx_rit;
    23. for (idx_rit=indexes.rbegin(); idx_rit!=indexes.rend(); ++idx_rit)
    24. {
    25. li = lv.begin() + *idx_rit; //li LineIterator, lv Linevector von außen
    26. lv.erase(li);
    27. }
    28. }
    30. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    32. }
    Alles anzeigen
    Forza Italia!!!!!!

  • Hi,
    wie soll die Eingabe denn generell aussehen?
    Möchtest du für jeden Wert mit return abschliessen oder alle Indexe in einer Zeile,durch Whitespaces getrennt,angeben?
    Hier ist jetzt mal ne aufgebohrte Version die die meisten Fehlerfälle abfängt.
    Der Hilfsfunktion get_indexes wird als zweiter Parameter nen boolscher Wert übergeben mit dem du steuern kannst ob die Werte einzeln(jeder mit return abgeschlossen) oder durch Whitespaces getrennt in einer Zeile eingelesen werden sollen.
    Die unterschiedlichen Anwendungen siehst du in viewline und removeline

    Quellcode

    1. #include <fstream>
    2. #include <iostream>
    3. #include <vector>
    4. #include <string>
    5. #include <iterator>
    6. #include <set>
    7. #include <sstream>
    9. class Editor
    10. {
    11. public:
    12. typedef std::vector<std::string> linevector;
    14. //Textdatei einelesen
    15. void einlesen()
    16. {
    17. std::ifstream ifs("C:/asci1.txt");
    18. std::string tmp;
    20. linevector::iterator li; //LineIterator
    22. while(getline(ifs, tmp))
    23. {
    24. lv.push_back(tmp);
    25. tmp = std::string();
    26. }
    27. }
    29. //Bestimmte Zeilen ansehen
    30. void viewline()
    31. {
    32. typedef std::vector<size_t> intvector;
    33. intvector intv;
    34. intvector::const_iterator inti;
    35. std::cout<<"Welche Zeilen sollen ausgegeebn werden?: \n\n";
    36. get_indexes(intv);
    38. linevector::iterator li;
    39. for (inti=intv.begin(); inti!=intv.end(); inti++)
    40. {
    41. li = lv.begin() + *inti; //li LineIterator, lv Linevector von außen
    42. std::cout << *li << std::endl;
    43. }
    44. }
    46. //Bestimmte Zeilen entfernen
    47. void removeline()
    48. {
    49. typedef std::set<size_t> index_set;
    50. index_set indexes;
    52. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    53. while (get_indexes(indexes,true));
    55. linevector::iterator li;
    56. index_set::const_reverse_iterator idx_rit;
    57. for (idx_rit=indexes.rbegin(); idx_rit!=indexes.rend(); ++idx_rit)
    58. {
    59. li = lv.begin() + *idx_rit; //li LineIterator, lv Linevector von außen
    60. lv.erase(li);
    61. }
    62. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    63. }
    65. //Zeilen hinzufügen
    66. void addline()
    67. {
    68. std::cout<<"Zeilen hinzufuegen: \n\n";
    69. std::string eingabe;
    70. std::cin.ignore(std::cin.rdbuf()->in_avail());
    71. do
    72. {
    73. std::getline(std::cin,eingabe);
    74. if(eingabe!="")
    75. lv.push_back(eingabe); //lv Linevector von außen
    76. else
    77. break;
    78. }while(std::cin.good());
    79. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    80. }
    82. private:
    83. linevector lv; //Linevector
    85. //Hilfsfunktion
    86. template<class container_type>
    87. bool get_indexes(container_type& int_container,bool single_values = false)
    88. {
    89. std::string str;
    90. std::getline(std::cin,str);
    91. if(str =="")
    92. return false;
    93. else
    94. {
    95. container_type::value_type tmp;
    96. std::istringstream isstr(str);
    97. if(!single_values)
    98. {
    99. do
    100. {
    101. isstr>>tmp;
    102. if(isstr && tmp<lv.size())
    103. int_container.insert(int_container.end(),tmp);
    104. else
    105. return false;
    106. }while(isstr);
    107. }
    108. else
    109. {
    110. isstr>>tmp;
    111. if(isstr && tmp<lv.size())
    112. int_container.insert(int_container.end(),tmp);
    113. else
    114. return false;
    115. }
    117. return true;
    118. }
    119. }
    121. };
    124. int main(){
    126. Editor edit;
    127. edit.einlesen();
    128. edit.viewline();
    129. edit.removeline();
    130. edit.addline();
    131. return 0;
    132. }
    Alles anzeigen

    Schau dir den Code halt mal an und bei Unklarheiten kannst du ja nachfragen. :wink:

    Ach ja,addline hab ich auch noch abgeändert.Die Version von heute Nacht hat nicht korrekt gearbeitet :oops: .

    Gruß void
    "Probleme kann man niemals mit derselben Denkweise lösen,
    durch die sie entstanden sind." (A. Einstein)

  • hey cool.. danke :) bin am studieren... also die eingaben sollen mit enter abgeschlossen werden.. außer bei addline()... da am besten durch ein wort oder so.. aber dass bekomme ich auch hin... nur templates hatten wir noch nicht.. ich gucke ob ichs trotzdem verstehe...

    aber auf jeden fall echt vielen dank.. ist mir ne superhilfe.. hab damit fast alles geschafft.. ich bastle grad noch an nem Rahmen für alles.. und dafür versuche ich den längsten String zu finden.. wenn ich soweit bein poste ich es nochmal.. bis denne... :)
    Forza Italia!!!!!!

  • Alle Funktionen sind nun implementiert, jedoch gibt es noch ein paar kleine Probleme die ich nicht behoben kann. Und in ein paar Tagen ist Abgabetermin für die Uni (ihr kennt das ja^^)

    Würde mich freuen wenn ihr ein paar (oder alle) Probleme rausbekommt.. und bitte nicht zu kompliziert, keine Templates (auch wenn es besser ist, aber die hatten wir noch nicht - sorry für deine Arbeit void :( ).. Hauptsache es läuft fehlerfrei...

    Die Probleme habe ich an die Stellen in den Quelltext geschrieben... :

    Quellcode

    1. #include "AsciiEditor.h"
    3. #include <fstream>
    4. #include <iostream>
    5. #include <vector>
    6. #include <string>
    7. #include <iterator>
    8. #include <set>
    11. class Editor
    12. {
    13. public:
    14. typedef std::vector<std::string> linevector;
    15. typedef std::set<int> stringset;
    16. unsigned int col;
    19. //Textdatei einelesen
    20. void einlesen()
    21. {
    23. /*Hier fehlt noch manuelle Angabe des Pfads oder Beispieldatei
    24. Auswahl: 1 -> pfad (ifs(pfad) oder 2 Beispiel-Datei*/
    26. std::cout <<"Ascii-Bild waehlen: Pfad angeben (1) oder Beispiel(2)";
    28. /* klappt leider nicht
    29. int wahl;
    30. std::cin >> wahl;
    31. if (wahl == 2){std::ifstream ifs("C:/asci1.txt");}
    32. if (wahl == 1){/*anuelle Pfadangabe}
    33. {std::cout << "Keine gueltige Auswahl";}
    34. */
    36. std::ifstream ifs("C:/asci1.txt");
    37. std::string tmp;
    39. linevector::iterator li; //LineIterator
    41. stringset maxstring;
    43. while(getline(ifs, tmp))
    44. {
    45. maxstring.insert(tmp.size());
    46. lv.push_back(tmp);
    47. tmp = std::string();
    50. }
    51. stringset::const_reverse_iterator aa;
    52. aa = maxstring.rbegin();
    53. col = *aa;
    55. for (li=lv.begin(); li!=lv.end(); li++)
    56. {
    57. std::cout << *li << std::endl;
    58. }
    59. }
    61. //Bestimmte Zeilen ansehen
    62. //Statt 999 besser "Enter" als Abbruchbedinung
    63. void viewline()
    64. {
    65. typedef std::vector<int> intvector;
    66. intvector intv;
    67. intvector::const_iterator inti;
    68. std::cout<<"Welche Zeilen sollen ausgegeebn werden?: \n\n";
    69. int eingabe;
    72. while (std::cin >> eingabe && eingabe != 999)
    73. {
    75. intv.push_back(eingabe);
    76. }
    78. linevector::iterator li;
    79. for (inti=intv.begin(); inti!=intv.end(); inti++)
    80. {
    81. li = lv.begin() + *inti; //li LineIterator, lv Linevector von außen
    82. std::cout << *li << std::endl;
    83. }
    84. }
    86. //Bestimmte Zeilen entfernen
    87. //Statt 999 besser "Enter" als Abbruchbedinung
    88. void removeline()
    89. {
    90. typedef std::set<int> index_set;
    91. index_set indexes;
    93. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    94. int eingabe;
    95. const unsigned int lvsize = lv.size();
    97. while (std::cin >> eingabe && eingabe != 999 ){
    99. if (eingabe > lvsize)
    100. {
    101. std::cout<< eingabe << " liegt ausserhalb des Bildes ";
    102. }
    104. indexes.insert(eingabe);
    106. linevector::iterator li;
    107. index_set::const_reverse_iterator idx_rit;
    109. for (idx_rit=indexes.rbegin(); idx_rit!=indexes.rend(); ++idx_rit)
    110. {
    111. li = lv.begin() + *idx_rit; //li LineIterator, lv Linevector von außen
    112. lv.erase(li);
    113. }
    114. }
    116. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    118. }
    120. //Zeilen hinzufügen
    121. //Problem: Fügt das erste Wort eine Zeile nicht ein
    123. void addline()
    124. {
    125. std::cout<<"Zeilen hinzufuegen: \n\n";
    127. std::string eingabe;
    128. std::string tmp;
    130. while(getline(std::cin >> eingabe, tmp)&& eingabe != "end!")
    131. {
    132. if(eingabe.size() > col)
    133. {
    134. std::cout << "Zeile muss kleiner als " << col << " sein!";
    135. break;
    136. }
    137. lv.push_back(tmp); //lv Linevector von außen
    138. tmp = std::string();
    139. }
    141. linevector::iterator li;
    142. for (li=lv.begin(); li!=lv.end(); li++)
    143. {
    144. std::cout << *li << std::endl; //li LineIterator, lv Linevector von außen
    145. }
    146. }
    148. //Rahmen zeigen -> OK
    149. void showline()
    150. {
    151. std::cout << "Symbol fuer Rahmen angeben: ";
    153. char symbol;
    155. std::cin >> symbol;
    157. std::string empty(col, ' ');
    158. std::string full(col + 4, symbol);
    160. std::cout << full << std::endl;
    161. std::cout << symbol << ' ' << empty << ' '<< symbol << std::endl;
    163. linevector::iterator li;
    165. for (li=lv.begin(); li!=lv.end(); ++li)
    166. {
    167. std::string max = *li;
    168. if(max.size() == col){
    169. std::cout << symbol << " " << *li << " " << symbol << std::endl;
    170. }
    171. else{
    172. unsigned int dif = col - max.size();
    173. std::string fill(dif, ' ');
    175. std::cout << symbol << " " << *li << fill << " " << symbol;
    176. }
    178. }
    179. std::cout << "\n" << symbol << ' ' << empty << ' '<< symbol << std::endl;
    180. std::cout << full << std::endl;
    181. }
    183. //Infos anzeigen lassen -> OK
    184. void infoline()
    185. {
    186. std::cout << "\nDas Ascii-Bild hat " << lv.size() <<" Zeilen und maximal " << col << " Spalten (ohne Rahmen).\n";
    188. }
    190. private:
    191. linevector lv; //Linevector
    192. stringset maxstring;
    193. };
    195. int main(){
    197. /*Sollte nur einmal edit.einlesen aufrufen, aber durch den main()-Aurufen
    198. weiter unten wird sie auch immer wieder neu geladen, ich weiß nicht wie ich
    199. die anderen Funktionen speriert aufrufen kann*/
    201. /*Sollte noch eine Exit-Funktion bzw. neues AsciiLaden-Funktion bekommen*/
    203. Editor edit;
    204. edit.einlesen();
    206. std::cout<<"\nWas wollen sie machen?\n"<<
    207. "Zeilen ansehen (1)?\nZeilen hinzufuegen (2)?\nZeilen entfernen (3)?\nZeileninfos anzeigen (4)?\nRahmen gestalten (5)\n\n";
    209. int wahl;
    210. std::cin >> wahl;
    211. switch(wahl)
    212. {
    213. case 1:edit.viewline();break;
    214. case 3:edit.removeline();break;
    215. case 2:edit.addline();break;
    216. case 5:edit.showline();break;
    217. case 4:edit.infoline();break;
    218. default: std::cout << "Zahl nicht guelting\n";
    219. }
    220. main();
    221. return 0;
    222. }
    Alles anzeigen
    Forza Italia!!!!!!

  • Hi Julia,
    hier ist schonmal der aktuelle Code(ohne templates).
    Dafür hab ich aber in removeline std::set durch nen vector ersetzt den ich mit sort sortiere um den gleichen Effekt zu erhalten.Dadurch brauch ich kein template mehr für die Hilfsfunktion get_indexes,sondern kann diese für std::vector<size_t> implementieren.
    Die set Geschichte um den längsten String zu finden halte ich für etwas oversized und hab das durch nen schlichtes if/else Konstrukt ersetzt.
    Die Geschichte mit dem alternativen Dateipfad hab ich minimal geändert(stream leeren).
    Die main hat jetzt 2 do/while Schleifen.
    addline habe ich ähnlich wie in meinen vorherigen Versionen implementiert,wobei ich es sinnvoller finden würde col neu zu setzen anstatt darauf zu bestehen dass die neue Zeile <=col sein muss.
    Schau dir den Code mal an.Hab den jetzt noch nicht groß getestet.
    Kannst ja mal schauen ob du noch irgendwelche Bugs findest.
    Alles andere können wir dann ja später besprechen :wink: .

    Quellcode

    1. #include <fstream>
    2. #include <iostream>
    3. #include <vector>
    4. #include <string>
    5. #include <iterator>
    6. #include <set>
    7. #include <sstream>
    8. #include <algorithm>
    10. class Editor
    11. {
    12. public:
    13. Editor():col(0){}
    14. typedef std::vector<std::string> linevector;
    15. typedef std::set<int> stringset;
    17. //Textdatei einelesen
    18. void einlesen()
    19. {
    20. std::cout <<"Ascii-Bild waehlen: Pfad angeben (1) oder Beispiel(2)";
    21. std::ifstream ifs;
    22. int wahl;
    23. std::cin >> wahl;
    24. if (wahl == 2){ifs.open("C:/asci1.txt");}
    25. else if (wahl == 1)
    26. {
    27. std::string temp_path;
    28. std::cout<<"Bitte den Pfad eingeben:";
    29. std::cin.ignore(std::cin.rdbuf()->in_avail());
    30. std::getline(std::cin,temp_path);
    31. ifs.open(temp_path.c_str());
    32. if(!ifs)
    33. {
    34. std::cout<<"Datei konnte nicht geoeffnet werden.\nProgramm wird beendet."<<std::endl;
    35. exit(-1);
    36. }
    37. }
    38. else {std::cout << "Keine gueltige Auswahl";exit(-1);}
    40. std::string tmp;
    42. linevector::iterator li; //LineIterator
    44. while(getline(ifs, tmp))
    45. {
    46. col = (tmp.size()>col)?tmp.size():col;
    47. lv.push_back(tmp);
    48. }
    50. for (li=lv.begin(); li!=lv.end(); li++)
    51. {
    52. std::cout << *li << std::endl;
    53. }
    54. }
    56. //Bestimmte Zeilen ansehen
    57. void viewline()
    58. {
    59. std::vector<size_t> intv;
    60. std::vector<size_t>::const_iterator inti;
    61. std::cout<<"Welche Zeilen sollen ausgegeebn werden?: \n\n";
    63. get_indexes(intv);
    65. linevector::iterator li;
    66. for (inti=intv.begin(); inti!=intv.end(); inti++)
    67. {
    68. li = lv.begin() + *inti; //li LineIterator, lv Linevector von außen
    69. std::cout << *li << std::endl;
    70. }
    71. }
    73. //Bestimmte Zeilen entfernen
    74. void removeline()
    75. {
    76. std::vector<size_t> indexes;
    78. std::cout<<"Welche Zeilen sollen geloescht werden?: \n\n";
    79. get_indexes(indexes);
    80. std::sort(indexes.begin(),indexes.end());
    82. linevector::iterator li;
    83. std::vector<size_t>::const_reverse_iterator idx_rit;
    85. for (idx_rit=indexes.rbegin(); idx_rit!=indexes.rend(); ++idx_rit)
    86. {
    87. li = lv.begin() + *idx_rit; //li LineIterator, lv Linevector von außen
    88. lv.erase(li);
    89. }
    90. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    92. }
    94. //Zeilen hinzufügen
    95. void addline()
    96. {
    97. std::cout<<"Zeilen hinzufuegen: \n\n";
    98. std::string eingabe;
    99. std::cin.ignore(std::cin.rdbuf()->in_avail());
    100. do
    101. {
    102. std::getline(std::cin,eingabe);
    103. if(eingabe!="!end")
    104. {
    105. if(eingabe.size() > col)
    106. {
    107. //Waere es hier nicht sinnvoller col neu zu setzen?
    108. std::cout << "Zeile muss kleiner als " << col << " sein!"<<std::endl;
    109. break;
    110. }
    111. lv.push_back(eingabe); //lv Linevector von außen
    112. }
    113. else
    114. break;
    115. }while(std::cin.good());
    116. std::copy(lv.begin(),lv.end(),std::ostream_iterator<std::string>(std::cout,"\n"));
    118. }
    120. //Rahmen zeigen -> OK
    121. void showline()
    122. {
    123. std::cout << "Symbol fuer Rahmen angeben: ";
    125. char symbol;
    127. std::cin >> symbol;
    129. std::string empty(col, ' ');
    130. std::string full(col + 4, symbol);
    132. std::cout << full << std::endl;
    133. std::cout << symbol << ' ' << empty << ' '<< symbol << std::endl;
    135. linevector::iterator li;
    137. for (li=lv.begin(); li!=lv.end(); ++li)
    138. {
    139. std::string max = *li;
    140. if(max.size() == col)
    141. std::cout << symbol << " " << *li << " " << symbol << std::endl;
    142. else
    143. {
    144. size_t dif = col - max.size();
    145. std::string fill(dif, ' ');
    147. std::cout << symbol << " " << *li << fill << " " << symbol<<std::endl;
    148. }
    149. }
    150. std::cout << symbol << ' ' << empty << ' '<< symbol << std::endl;
    151. std::cout << full << std::endl;
    152. }
    154. //Infos anzeigen lassen -> OK
    155. void infoline()
    156. {
    157. std::cout << "\nDas Ascii-Bild hat " << lv.size() <<" Zeilen und maximal " << col << " Spalten (ohne Rahmen).\n";
    158. }
    160. private:
    161. size_t col;
    162. linevector lv;
    164. //Hilfsfunktion
    165. bool get_indexes(std::vector<size_t>& index_vec,bool single_values = false)
    166. {
    167. std::string str;
    168. std::cin.ignore(std::cin.rdbuf()->in_avail());
    170. std::getline(std::cin,str);
    171. if(str =="")
    172. return false;
    173. else
    174. {
    175. size_t tmp;
    176. std::istringstream isstr(str);
    177. if(!single_values)
    178. {
    179. do
    180. {
    181. isstr>>tmp;
    182. if(isstr && tmp<lv.size())
    183. index_vec.push_back(tmp);
    184. else
    185. return false;
    186. }while(isstr);
    187. }
    188. else
    189. {
    190. isstr>>tmp;
    191. if(isstr && tmp<lv.size())
    192. index_vec.push_back(tmp);
    193. else
    194. return false;
    195. }
    197. return true;
    198. }
    199. }
    200. };
    202. int main()
    203. {
    204. char d;
    205. do
    206. {
    207. Editor edit;
    208. edit.einlesen();
    209. int wahl=0;
    210. do
    211. {
    212. std::cout<<"\nWas wollen sie machen?\n"<<
    213. "Zeilen ansehen (1)?\nZeilen hinzufuegen (2)?\nZeilen entfernen (3)?\nZeileninfos anzeigen (4)?\nRahmen gestalten (5)\nEnde(6)\n\n";
    215. std::cin >> wahl;
    216. switch(wahl)
    217. {
    218. case 1:edit.viewline();break;
    219. case 2:edit.addline();break;
    220. case 3:edit.removeline();break;
    221. case 4:edit.infoline();break;
    222. case 5:edit.showline();break;
    223. case 6:std::cout<<"Auf Wiedersehen"<<std::endl;break;
    224. default: std::cout << "Zahl nicht guelting\n";
    225. }
    226. } while(wahl != 6);
    228. std::cout<<"Neue Datei bearbeiten (j/n)? :";
    229. std::cin>>d;
    230. }while(d!= 'n' && d!='N');
    232. return 0;
    233. }
    Alles anzeigen


    Edit:Formatierung korrigiert.

    Gruß void
    "Probleme kann man niemals mit derselben Denkweise lösen,
    durch die sie entstanden sind." (A. Einstein)

  • Optimierung:
    Für den Fall dass der längste String gelöscht wird bietet es sich an jedes mal erst in showline die größte Länge zu ermitteln.

    Quellcode

    1. //global
    2. bool greater_string(const std::string& str1,const std::string& str2)
    3. {
    4. return str1.size()<str2.size();
    5. };
    6. ........
    7. void showline()
    8. {
    9. col = std::max_element(lv.begin(),lv.end(),greater_string)->size();
    10. std::cout << "Symbol fuer Rahmen angeben: ";
    12. char symbol;
    14. std::cin >> symbol;
    16. std::string empty(col, ' ');
    17. std::string full(col + 4, symbol);
    19. std::cout << full << std::endl;
    20. std::cout << symbol << ' ' << empty << ' '<< symbol << std::endl;
    22. linevector::iterator li;
    24. for (li=lv.begin(); li!=lv.end(); ++li)
    25. {
    26. std::string max = *li;
    27. if(max.size() == col)
    28. std::cout << symbol << " " << *li << " " << symbol << std::endl;
    29. else
    30. {
    31. size_t dif = col - max.size();
    32. std::string fill(dif, ' ');
    34. std::cout << symbol << " " << *li << fill << " " << symbol<<std::endl;
    35. }
    36. }
    37. std::cout << symbol << ' ' << empty << ' '<< symbol << std::endl;
    38. std::cout << full << std::endl;
    39. }
    Alles anzeigen

    Da col sonst nirgendwo benötigt wird kann col auch lokal definiert werden und sämtlicher anderer Code der sich auf col bezieht wegfallen.

    Edit:Korrektur-> max_element arbeitet ohne Predikat auf strings nicht wie erwartet.Deshalb greater_string hinzugefügt.

    Gruß void
    "Probleme kann man niemals mit derselben Denkweise lösen,
    durch die sie entstanden sind." (A. Einstein)