fix off-by-one in item display

[c128-kasse] / src / kasse.c

diff --git a/src/kasse.c b/src/kasse.c
index d5d55e71845738ea89cc3abbe5d93de64249eb65..0710b2adad7013008c0e6874906da2023480e73f 100644 (file)
--- a/src/kasse.c
+++ b/src/kasse.c
@@ -36,35 +36,62 @@ static void print_screen() {
                cprintf("Einnahme %ld konnte nicht umgerechnet werden\r\n", money);
                exit(1);
        }
-       cprintf("C128-Kassenprogramm (phil_fry, sECuRE, sur5r) " GV "\r\
+       textcolor(TC_CYAN);
+       cprintf("C128-Kassenprogramm (phil_fry, sECuRE, sur5r) " GV "\r");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf("\
 \r\nUhrzeit:     %s (wird nicht aktualisiert)\r\
 Eingenommen: %s, Verkauft: %ld Dinge, Drucken: %s\r\n",
        time, profit, items_sold, (printing == 1 ? "ein" : "aus"));
-       cprintf("\xB0\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xAE\r\n");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf("      \xB0\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB2\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xAE\r\n");
        for (; i < min(status.num_items, 15); ++i) {
                if (format_euro(profit, sizeof(profit), status.status[i].price) == NULL) {
                        cprintf("Preis %ld konnte nicht umgerechnet werden\r\n", status.status[i].price);
                        exit(1);
                }
 
-               cprintf("\x7D%2d: %-" xstr(MAX_ITEM_NAME_LENGTH) "s \x7D%s, %3dx \x7D",
-                       i, status.status[i].item_name, profit, status.status[i].times_sold);
-               if ((i+16) < status.num_items) {
+               cprintf("      \x7D");
+               textcolor(TC_YELLOW);
+               cprintf("%2d", i);
+               textcolor(TC_LIGHT_GRAY);
+               cprintf(": %-" xstr(MAX_ITEM_NAME_LENGTH) "s \x7D%s, %3dx \x7D",
+                       status.status[i].item_name, profit, status.status[i].times_sold);
+               if ((i+15) < status.num_items) {
 
-                       if (format_euro(profit, sizeof(profit), status.status[i+16].price) == NULL) {
-                               cprintf("Preis %ld konnte nicht umgerechnet werden\r\n", status.status[i+16].price);
+                       if (format_euro(profit, sizeof(profit), status.status[i+15].price) == NULL) {
+                               cprintf("Preis %ld konnte nicht umgerechnet werden\r\n", status.status[i+15].price);
                                exit(1);
                        }
-                       cprintf("%2d: %-" xstr(MAX_ITEM_NAME_LENGTH) "s \x7D%s, %3dx \x7D",
-                               i+16, status.status[i+16].item_name, profit, status.status[i+16].times_sold);
+                       textcolor(TC_YELLOW);
+                       cprintf("%2d", i+15);
+                       textcolor(TC_LIGHT_GRAY);
+                       cprintf(": %-" xstr(MAX_ITEM_NAME_LENGTH) "s \x7D%s, %3dx \x7D",
+                               status.status[i+15].item_name, profit, status.status[i+15].times_sold);
                } else cprintf("              \x7D                \x7D");
                cprintf("\r\n");
        }
-       cprintf("\xAD\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xBD\r\n");
-       cprintf(
-"   s) Daten sichern                 d) Drucken umschalten\r\
-   g) Guthabenverwaltung            z) Zeit setzen\r\
-   f) Freitext verkaufen            q) Beenden\r\n");
+       cprintf("      \xAD\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xB1\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\x60\xBD\r\n");
+       textcolor(TC_YELLOW);
+       cprintf("   s");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf(") Daten sichern                                  ");
+       textcolor(TC_YELLOW);
+       cprintf("g");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf(") Guthabenverwaltung\r\n");
+       textcolor(TC_YELLOW);
+       cprintf("   z");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf(") Zeit setzen         ");
+       textcolor(TC_YELLOW);
+       cprintf("f");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf(") Freitext verkaufen      ");
+       textcolor(TC_YELLOW);
+       cprintf("q");
+       textcolor(TC_LIGHT_GRAY);
+       cprintf(") Beenden\r\n");
 }
 
 /*
@@ -94,6 +121,7 @@ static void print_log(char *name, int item_price, int einheiten, char *nickname,
        sprintf(print_buffer, "%c[%3u] %s - %-" xstr(MAX_ITEM_NAME_LENGTH) "s - %s - %s - %d - an %s\r",  17,
                        status.transaction_id, time, name, price, rest,
                        einheiten, (*nickname != '\0' ? nickname : "Unbekannt"));
+       status.transaction_id++;
        print_the_buffer();
 }
 
@@ -150,10 +178,18 @@ static signed int buy(char *name, unsigned int price) {
                 * to NULL if no such credit could be found */
                credit = find_credit(nickname);
                if (credit != NULL) {
-                       if ((signed int)credit->credit < ((signed int)price * einheiten)) {
-                               cprintf("Sorry, %s hat nicht genug Geld :-(\r\n", nickname);
-                               get_input();
-                               return 0;
+                       while ((signed int)credit->credit < ((signed int)price * einheiten)) {
+                               if (format_euro(rest, 10, credit->credit) == NULL) {
+                                       cprintf("Preis %d konnte nicht umgerechnet werden\r\n", credit->credit);
+                                       exit(1);
+                               }
+                               cprintf("\r\n%s hat nicht genug Geld (%s). e) einzahlen a) abbruch \r\n", nickname, rest);
+                               c = cgetc();
+                               if (c == 'e') {
+                                       deposit_credit(nickname);
+                               } else {
+                                       return 0;
+                               }
                        }
                        /* substract money */
                        credit->credit -= (price * einheiten);
@@ -163,15 +199,19 @@ static signed int buy(char *name, unsigned int price) {
                                exit(1);
                        }
 
+                       textcolor(TC_LIGHT_GREEN);
                        cprintf("\r\nVerbleibendes Guthaben fuer %s: %s. Druecke RETURN...\r\n",
                                nickname, rest);
+                       textcolor(TC_LIGHT_GRAY);
                        toggle_videomode();
                        cprintf("\r\nDein Guthaben betraegt noch %s.\r\n", rest);
                        toggle_videomode();
                        get_input();
                        matches++;
                } else {
+                       textcolor(TC_LIGHT_RED);
                        cprintf("\r\nNickname nicht gefunden in der Guthabenverwaltung! Abbruch, druecke RETURN...\r\n");
+                       textcolor(TC_LIGHT_GRAY);
                        get_input();
                        return 0;
                }
@@ -308,12 +348,6 @@ int main() {
                        save_credits();
                        cprintf("Statefile/Creditfile gesichert, druecke RETURN...\r\n");
                        get_input();
-               } else if (*c == 'd') {
-                       /* enable/disable printing */
-                       printing = (printing == 1 ? 0 : 1);
-                       cprintf("Drucken ist nun %s, druecke RETURN...\r\n", 
-                               (printing == 1 ? "eingeschaltet" : "ausgeschaltet"));
-                       get_input();
                } else if (*c == 'g') {
                        credit_manager();
                } else if (*c == 'z') {