Thread: Simple Editor

  1. #1
    Registered User
    Join Date
    Sep 2012

    Exclamation Simple Editor

    Hi there,

    I'm having a lot of trouble trying to write an apparently "simple" editor. What do you make of my code? It's already late to hand in.

    #include <stdio.h>
    #include <conio.h>
    #include <string.h>
    char text[23][80];
    int main() {
        int i=0, num;
        char c[80];
        char temp[80];
        FILE *g;
        printf("Please enter the file name:\n");
        g = fopen(c, "r");
        if(g == NULL) {
            printf("Problem! The file did not open sucessfully.");
        while(fgets (text[i], 80, g) !=NULL) {  
        for(i=0; i<23; i++) {
            printf("%s", text[i]);
        fgets(temp, sizeof(temp), stdin);
        sscanf(temp, "%c %d", c, &num);
        while(strcmp(temp, "Q") !=0) {
            if(strcmp(temp, "R") ==0) {
            if(strcmp(temp, "D") ==0) {
                for(i=21; i<num; i++) { 
                    strcpy(text[i], text[i+1]);
            if(strcmp(temp, "I") ==0) {
                for(i=22; i>num; i--) {
                    strcpy(text[i], text[i-1]);
            if(strcmp(temp, "S") ==0) {
                for(i=0; i<23; i++) {
                    fprintf(g, text[i]);

  2. #2
    Ticked and off
    Join Date
    Oct 2011
    La-la land
    Your system("cls"); only works in Windows and DOS. Instead, you could use CSI codes (ANSI escape codes). The CSI in the list is simply the string "\033[", so to for example clear the screen you can use
        printf("\033[H\033[2J"); /* Move cursor to upper left corner, then clear the screen. */
    For a text-mode editor, the ncurses library provides a lot of useful features, like automatic screen (terminal) size detection and so on. On Windows, pdcurses or win32a should do fine.

    As to the data structures, I'd define a dynamic string type (with a end-of-line type flag). Perhaps
    struct line {
        size_t        size;  /* Bytes allocated */
        size_t        used;  /* Bytes used */
        unsigned char newline;
        unsigned char data[];
    struct document {
        size_t        size;  /* Lines allocated */
        size_t        used;  /* Lines used */
        struct line  *line[];
    struct document *document_load(const char *const filename);
    int document_save(const struct document *const doc, const char *const filename);
    int document_insert(struct document *const doc, const size_t before_line, struct line *const line);
    int document_delete(struct document *const doc, const size_t line);
    int line_free(struct line *const line);
    struct line *line_new(const unsigned char *const from, const size_t length);
    struct line *line_grow(struct line *const line, const size_t minimum);
    struct line *line_copy(const struct line *const line);
    When inserting or deleting lines, only the pointers need to be updated. When appending a line, the struct line can be reallocated when necessary, so it can be grown longer. The line_ helper functions would allow you to manipulate the contents of a line as you wish.

    The reason for the newline constant for each line is that if you include it in the string itself, it will make your line editing and display functions overly complicated. Also, if instead of an actual character it is a constant, you can support different newline conventions without issues. (One of my pet peeves is how many editors mangle and get confused about newline conventions. I'd want an editor that supports NUL ('\0'), LF ('\n'), CR ('\r'), LF CR ("\r\n"), and CR LF ("\n\r") without me having to do anything about it.)

    It would be even better to use wchar_t data[]; in the struct line, so each element in that array would correspond to a glyph (character) in the user's locale. (For example, takes nowadays three chars, if you are using the UTF-8 character set as you should. "But I don't want to bother" is a poor excuse for mangling peoples names, for example.)

    Is there anything specific you want to ask?

  3. #3
    - - - - - - - - oogabooga's Avatar
    Join Date
    Jan 2008
    I can only assume that Nominal (more accurately, Actual) Animal's post is a joke. Just ignore it.

    What are the specs for your assignment?
    What is your simple editor supposed to do?
    The cost of software maintenance increases with the square of the programmer's creativity. - Robert D. Bliss

  4. #4
    Registered User
    Join Date
    Sep 2012
    Here is a copy of the task sheet.

    Write a simple editor.

    The editor should first ask the user for the name of a file to edit.
    You may assume that the file already exists.

    Files will be restricted to exactly 23 lines of text.

    The editor should display the text in the file starting at the top of the screen.

    The 24'th line is used by the user to give commands to the editor. The prompt should be Command:

    The only commands that are available are:

    D number which will delete the line with that number. All lines after that one will move up one.
    The new 23rd line at the bottom will be empty.

    I number Will insert the text typed on the 25'th line before the line with that number
    All lines will move down one to accommodate the new line.
    The old 23'rd line will be discarded.

    R number Will replace, with the text typed on the 25'th line, the line with that number

    S Will save away the changes, by writing the text back to disk.

    Q Will quit the program


    Command: I 6
    this new line will be placed between line 5 and line 6 of the file

    After each command the screen should be cleared using the function system("cls");.
    The text should then be displayed again.

    Line numbers given by the user will, in true computer science style, start from 0 and go to 22.

Popular pages Recent additions subscribe to a feed

Similar Threads

  1. Simple text editor help
    By Fink in forum C Programming
    Replies: 45
    Last Post: 01-18-2011, 02:12 PM
  2. developing a simple Text editor in C
    By rssrik in forum C Programming
    Replies: 10
    Last Post: 07-03-2007, 03:28 AM
  3. A good, simple web editor
    By confuted in forum A Brief History of
    Replies: 11
    Last Post: 09-19-2003, 07:20 PM
  4. simple text editor in C
    By adil75950 in forum C Programming
    Replies: 2
    Last Post: 03-12-2002, 01:15 PM