• Odpowiedz

@Minniat, post #1

• Odpowiedz

@Minniat, post #1

• Odpowiedz

@MDW, post #3

• Odpowiedz

@szuler, post #2

• Odpowiedz

@MDW, post #3

• Odpowiedz

@Minniat, post #5

• Odpowiedz

@szuler, post #7

• Odpowiedz

@Minniat, post #8

• Odpowiedz

@Minniat, post #9

class line
{
private:
   struct Node line_node; // węzeł
   char *text; // tekst w linijce
   int linelen; // długość tekstu
   int lineused; // ilość używanych znaków

public:
   line(int len); // konstruktor pobiera długość linijki i alokuje pamięć
   ~line(); // destruktor zwalnia pamięć
   void ClearLine(); // kasuje linię
   void SetText(char *newtext); // wstawia tekst do linii
   void Print(); // wyświetla tekst
   void Backspace(); // usuwa znak przed kursorem
   void Delete(); // usuwa znak w miejscu kursora
   void AddChar(char c); // dodaje nowy znak w miejscu kursora
   void SplitLine(); // dzieli linię na dwie (póki co brak implementacji)
}
*cursorline;

int cursorpos; // pozycja x kursora

line :: line(int len)
{
   printf("Constructing line (len=%d)...n",len);
   text = (char *)malloc(len);
   linelen = len;
}

line :: ~line()
{
   printf("Deconstructing line...n" );
   free(text);
}

void line :: ClearLine()
{
   int i;

   printf("Clearing line...n" );
   text[0] = 'n';
   text[1] = '';
   for (i = 2; i < linelen; i++)
      text[ i ] = ' ';
   lineused = strlen(text);
}

void line :: SetText(char *newtext)
{
   printf("Setting text to %s", newtext );
   strcpy(text, newtext);
   lineused = strlen(text);
}

void line :: Print()
{
   printf("Printing line...n" );
   printf("CHARS: %d LINE: %s", lineused, text);
}

void line :: Backspace()
{
   int i;

   printf("Backspacing...n" );
   if (cursorpos > 0)
   {
      cursorpos--;
      for (i = cursorpos; i < lineused; i++)
         text[ i ] = text[i + 1];
      text[ i ] = '';
   }
   lineused--;
   cursorpos--;
}

void line :: Delete()
{
   int i;

   printf("Deleting...n" );
   for (i = cursorpos; i < lineused; i++)
      text[ i ] = text[i + 1];
   text[ i ] = '';
   lineused--;
}

void line::AddChar(char c)
{
   int i;

   printf("Adding char -%c-...n", c);
   lineused++;
   for (i = lineused; i > cursorpos; i--)
      text[ i ] = text[i - 1];
   text[cursorpos] = c;
   cursorpos++;
}

void line::SplitLine()
{
}

int main()
{
   line testline(100);
   char test[]="Hello!n";
   cursorline = &testline;
   testline.ClearLine();
   testline.Print();
   testline.SetText(test);
   testline.Print();
   cursorpos = 5;
   printf("Setting cursor at %d...n", cursorpos);
   testline.AddChar(' ');
   testline.AddChar('W');
   testline.AddChar('o');
   testline.AddChar('r');
   testline.AddChar('l');
   testline.AddChar('d');
   testline.Print();
   return 0;
}

• Odpowiedz

@Minniat, post #10

• Odpowiedz

@Minniat, post #10

line new_line(2); new_line.SetText("Dupa zbita");

Konstruktor z parametrem to zły pomysł, zrób konstruktor bezparametrowy i jak już musisz to dodaj metodę do zmieniania wielkości bufora...
Stały bufor to w ogóle bardzo zły pomysł... Sprawdzaj zakresy - czy nie wychodzisz poza bufor, i czy nie trzeba w takim przypadku realokować bufora... Bez tego ten kod to po prostu generator zwiech...

Jaki jest sens w klasie line (z założenia przechowującej jedna linie tekstu) trzymać na sile znak nowej linii?

Poza tym: piszesz w c++, a używasz funkcji biblioteki standardowej c...
Korzystanie z malloc/free w c++ to też średni pomysł... W c++ od tego masz operatory new/delete...


Nie używaj zmiennych globalnych:
int cursorpos; // pozycja x kursora

cursorline = &testline; - po co Ci to jak z tego nie korzystasz? ;)

Inne sugestie:
-przeczytaj Pasje C++ i Symfonie C++ Grębosza.
-wywal ten kod i użyj po prostu std::string, bo to co napisałes to taka... odrobinę prymitywna implementacja tej klasy.

• Odpowiedz

@MinisterQ, post #12

Przede wszystkim dziękuję za komentarz, ale proszę, nie pisz tylko wad mojej implementacji metod klasy line. Poza tym to tylko zarys programu.

"Konstruktor z parametrem to zły pomysł, zrób konstruktor bezparametrowy i jak już musisz to dodaj metodę do zmieniania wielkości bufora..."
1) Jednak ten parametr jest wygodny w użyciu. Zmiana wielkości bufora jest za to odpowiednim pomysłem.

"Stały bufor to w ogóle bardzo zły pomysł... Sprawdzaj zakresy - czy nie wychodzisz poza bufor, i czy nie trzeba w takim przypadku realokować bufora... Bez tego ten kod to po prostu generator zwiech..."
2) OK. Dodam realokację bufora. Póki co metoda line::SetText() nie sprawdza, czy wychodzimy poza bufor.

"Jaki jest sens w klasie line (z założenia przechowującej jedna linie tekstu) trzymać na sile znak nowej linii?"
3) Przydaje się to tylko do testu. W edytorze tego nie będzie.

"Nie używaj zmiennych globalnych:
int cursorpos; // pozycja x kursora"
4) Ależ kursor jest tylko jeden. Jego pozycja i linia, w której się znajduje może być globalne. No chyba, że stworzę klasę kursor...

"-wywal ten kod i użyj po prostu std::string, bo to co napisałes to taka... odrobinę prymitywna implementacja tej klasy."
5) Napisałem własną implementację. Poza tym dochodzi metoda JoinLine() i SplitLine(), które są potrzebne w edytorze tekstu.



Ostatnia modyfikacja: 08.09.2008 13:05:40

• Odpowiedz

@Minniat, post #13

• Odpowiedz

@MinisterQ, post #14

• Odpowiedz

@Minniat, post #15

clear ();

string& erase ( size_t pos = 0, size_t n = npos );
iterator erase ( iterator position );
iterator erase ( iterator first, iterator last );

string& erase ( size_t pos = 0, size_t n = npos );
iterator erase ( iterator position );
iterator erase ( iterator first, iterator last );

string& insert ( size_t pos1, const string& str );
 string& insert ( size_t pos1, const string& str, size_t pos2, size_t n );
 string& insert ( size_t pos1, const char* s, size_t n);
 string& insert ( size_t pos1, const char* s );
 string& insert ( size_t pos1, size_t n, char c );
iterator insert ( iterator p, char c );
    void insert ( iterator p, size_t n, char c );
template
    void insert ( iterator p, InputIterator first, InputIterator last );

string substr ( size_t pos = 0, size_t n = npos ) const;

std::string string_1;
std::string string_2;
std::string string_3;

string_1 = "Hello ";
string_2 = "World";
string_3 = string_1 + string_2;

std::cout << string_3 << std::endl;

• Odpowiedz

@szuler, post #16

• Odpowiedz

@szuler, post #16

• Odpowiedz

@rzookol, post #18

• Odpowiedz

@Minniat, post #10

Patrząc z punktu widzenia wzorca projektowego MVC skupiłeś się na razie na M czyli na modelu. A mówiąc w sposób zrozumiały dla wszystkich musisz to jeszcze wyświetlić . Do tego obsłużyć kursor zarówno jeśli chodzi o przemiaszczanie (w tym myszą) jak i rysowanie. Podejrzewam że czeka Cie niemiła niespodzianka gdy zaczniesz zastanawiać się nad rozwiązaniem problemu Taba. Tak czy inaczej jeśli chodzi o edytor programisty zastosowanie podejścia, w którym każda linia jest trzymana osobno (czyli takiego jak Twoje) nie jest złe. Ułatwi i przyspieszy to bardzo parsowania tekstu gdy będziesz chciał zrealizować podświetlanie. Jednak sam model (czyli to co masz) to początek początku. Czeka Cie jeszcze dużo pracy zwłaszcza jeśli chcesz to zrobić porządnie i nie wyświetlać całego tekstu co operację. Proponowałbym posłuchać Szulera i wykorzystać stl. Czas nie jest z gumy a gotowy sprawdzony kod umożliwia efektywniejsze wykorzystanie go. Dodatkowo rozdzielając operacje na jednej linii i wykorzystując do nich stringa od operacji na całym modelu (czyli te wykonywane na więcej niż jednej linii) ułatwiasz sobie pisanie testów. Nie widzę również sensu wykorzystywania list execowych, według mnie one nie są do tego, lepiej użyć kontenerów z stl. To umożliwi Ci wykorzystanie funkcji tam zawartych operujących właśnie na tych kontenerach. Odpadnie Ci dzięki temu trochę pracy.

• Odpowiedz

@smith, post #20

• Odpowiedz

@Minniat, post #21

• Odpowiedz