[C] Passaggio di stringhe

[KITT]

ciao a tutti,
sto cercando di usare la funzione textbackground della libreria conio2 utilizzando il Dev-C++.

Ho scritto un algoritmo che colora una riga (cmq non credo che sia proprio il massimo):

for (i=1;i<=80;i++)
{
textbackground (YELLOW);
printf (" ");
}

Il mio problema è far diventare questo algoritmo una funzione che prendendo in ingresso un colore (stringa nomecolore) mi restituisca la riga colorata.

Io avevo pensato:

typedef char stringa[20]

void background (stringa colore)
{
int i;
for (i=1;i<=80;i++)
{
textbackground (colore);
printf (" ");
}
}

Aggiungo che i tipi vengono dichiarati in un file.h allegato al file main.c, inoltre le procedure sono implementate in un altro file.c tutto fa parte di progetto.dev

Il passaggio di parametri avviene correttamente (riesco a stampare la stringa colore).
L'errore segnalato è: [Warning] passing arg 1 of `textbackground' makes integer from pointer without a cast

Chi mi può dare una mano??

[oregon]

Penso che il valore dell'argomento sia di tipo numerico intero. Ovvero che YELLOW sia una costante numerica.

Dovresti trovare queste costanti all'interno del file .h

Quindi dovrebbe essere

void background (int colore)

[KITT]

Nello header file ho solo le dichiarazioni di tipi, ES typedef char stringa[20].

I prototipi delle funzioni insieme alle implementazioni sono nel file.c

Tra i prototipi c'è anche -> void background (stringa);

Non capisco perchè, è come se textbackground non riconosca che stringacolore in realtà contiene un nome di colore ad esso compatibile.

ES:
strcpy (colore,"YELLOW");
textbackground (colore) da quell'errore, ma in realtà colore = YELLOW.

[oregon]

Non e' che si capisca molto ...

Forse deve essere

textbackground ("YELLOW");

oppure, mostra il contenuto del file include e del .c

[Xaratroom]

YELLOW è una costante :

enum COLORS {
    /*  dark colors     */
    BLACK,          
    BLUE,
    GREEN,
    CYAN,
    RED,
    MAGENTA,
    BROWN,
    LIGHTGRAY,
    /*  light colors    */
    DARKGRAY, /* "light black" */
    LIGHTBLUE,
    LIGHTGREEN,
    LIGHTCYAN,
    LIGHTRED,
    LIGHTMAGENTA,
    YELLOW,
    WHITE
};

e per usare conio.h devi linkare conio.o (in Dev-C++\Lib\)...

[Xaratroom]

Giusto per la cronaca :
conio.h

#ifndef _CONIO_H_
#define _CONIO_H_

#ifdef __cplusplus
extern "C" {
#endif

struct text_info {
    unsigned char winleft;
    unsigned char wintop;
    unsigned char winright;
    unsigned char winbottom;
    unsigned char attribute;
    unsigned char normattr;
    unsigned char currmode;
    unsigned char screenheight;
    unsigned char screenwidth;
    unsigned char curx;
    unsigned char cury;
};

enum COLORS {
    /*  dark colors     */
    BLACK,          
    BLUE,
    GREEN,
    CYAN,
    RED,
    MAGENTA,
    BROWN,
    LIGHTGRAY,
    /*  light colors    */
    DARKGRAY, /* "light black" */
    LIGHTBLUE,
    LIGHTGREEN,
    LIGHTCYAN,
    LIGHTRED,
    LIGHTMAGENTA,
    YELLOW,
    WHITE
};
#define BLINK   0x80    /*  blink bit; doesn't work yet  */

#define _NOCURSOR      0
#define _SOLIDCURSOR   1
#define _NORMALCURSOR  2

/* 19 of 31 functions implemented, 62% */

int _conio_kbhit();
void _set_screen_lines(int nlines);
void _setcursortype(int _type);                                                 /* done */
void blinkvideo();
char *cgets(char *_str);                                 /* how does it work? */
void clreol();                                           /* what does it do? */
void clrscr();                                                                  /* done */
#define cprintf printf                                                          /* done */
int cputs(const char *_str);                                                    /* done */
#define cscanf scanf                                                            /* done */
void delline();
#define getch getchar                                                           /* done */
int getche();                                                                   /* done */
int gettext(int _left, int _top, int _right, int _bottom, void *_destin);
void gettextinfo(struct text_info *_r);                                         /* done */
void gotoxy(int x, int y);                                                      /* done */
void gppconio_init();                                                           /* done; does nothing */
void highvideo();                                                               /* maybe */
void insline();                                                                 /* done */
void intensevideo();
void lowvideo();
int movetext(int _left, int _top, int _right, int _bottom,
             int _destleft, int _desttop);
void normvideo();
int putch(int _c);                                                              /* done */
int puttext(int _left, int _top, int _right, int _bottom, void *_source);
void textattr(int _attr);                                                       /* done */
void textbackground(int _color);                                                /* done */
void textcolor(int _color);                                                     /* done */
void textmode(int _mode);
int ungetch(int);
int wherex();                                                                   /* done */
int wherey();                                                                   /* done */
void window(int _left, int _top, int _right, int _bottom);                      /* done */

#ifdef __cplusplus
}
#endif

#endif _CONIO_H_

conio.c

#ifndef _CONIO_C_
#define _CONIO_C_

/* Please keep all functions alphabetical! */

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <windows.h>
#include "conio.h"

int __FOREGROUND = LIGHTGRAY;
int __BACKGROUND = BLACK;

void _setcursortype(int _type) {
  CONSOLE_CURSOR_INFO Info;
  Info.bVisible = TRUE;
  if (_type == _NOCURSOR)
     Info.bVisible = FALSE;
  else if (_type == _SOLIDCURSOR)
     Info.dwSize = 100;
  else if (_type == _NORMALCURSOR)
     Info.dwSize = 1;
  SetConsoleCursorInfo (GetStdHandle (STD_OUTPUT_HANDLE), &Info);
}

void clreol() {
  /* What does this function do? */
}

void clrscr() {
  COORD coord;
  DWORD written;
  CONSOLE_SCREEN_BUFFER_INFO info;

  coord.X = 0;
  coord.Y = 0;
  GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &info);
  FillConsoleOutputCharacter (GetStdHandle(STD_OUTPUT_HANDLE), ' ',
    info.dwSize.X * info.dwSize.Y, coord, &written);
  gotoxy (1, 1);
}

int cputs(const char *_str) {
  printf ("%s\n", _str);
  return 0;
}

int getche() {
  int ch;
  ch = getch ();
  printf ("%c\n", ch);
  return ch;
}

void gettextinfo(struct text_info *_r) {
  CONSOLE_SCREEN_BUFFER_INFO Info;
  GetConsoleScreenBufferInfo (GetStdHandle (STD_OUTPUT_HANDLE), &Info);
  _r->winleft = Info.srWindow.Left;
  _r->winright = Info.srWindow.Right;
  _r->wintop = Info.srWindow.Top;
  _r->winbottom = Info.srWindow.Bottom;
  _r->attribute = Info.wAttributes;
  _r->normattr = LIGHTGRAY | BLACK;
/*  _r->currmode = ; */ /* What is currmode? */
  _r->screenheight = Info.dwSize.Y;
  _r->screenwidth = Info.dwSize.X;
  _r->curx = wherex ();
  _r->cury = wherey ();
}

void gotoxy(int x, int y) {
  COORD c;
  c.X = x - 1;
  c.Y = y - 1;
  SetConsoleCursorPosition (GetStdHandle(STD_OUTPUT_HANDLE), c);
}

void gppconio_init() {
  /* Do nothing */
}

void highvideo() {
  if (__FOREGROUND <= BROWN)
     textcolor (__FOREGROUND + 9);
  if (__BACKGROUND <= BROWN)
     textbackground (__BACKGROUND + 9);
}

void insline() {
  printf ("\n");
}

int putch(int _c) {
  printf ("%c", _c);
  return _c;
}

void textattr(int _attr) {
  SetConsoleTextAttribute (GetStdHandle(STD_OUTPUT_HANDLE), _attr);
//  printf ("%d\n", text_info.screenheight);
}

void textbackground(int _color) {
  if (_color == BLINK)
     _color = WHITE;
  __BACKGROUND = _color;
  textattr (__FOREGROUND | (_color + 29));
}

void textcolor(int _color) {
  if (_color == BLINK)
     _color = WHITE;
  __FOREGROUND = _color;
  textattr(_color | __BACKGROUND);
}

int wherex() {
  CONSOLE_SCREEN_BUFFER_INFO info;
  GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &info);
  return info.dwCursorPosition.X;
}

int wherey() {
  CONSOLE_SCREEN_BUFFER_INFO info;
  GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &info);
  return info.dwCursorPosition.Y - 2;
}

void window(int _left, int _top, int _right, int _bottom) {
  SMALL_RECT R;
  R.Left = _left;
  R.Top = _top;
  R.Right = _right;
  R.Bottom = _bottom;
  SetConsoleWindowInfo (GetStdHandle(STD_OUTPUT_HANDLE), TRUE, &R);
  gotoxy (_left, _top);
}

#endif _CONIO_C_

[KITT]

Grazie mille per l'aiuto.

Per l'uso della libreria conio2 ho seguito le istruzioni riportate sul sito blooshed da dove ho scaricato Dev-C++ e chiaramente l'ho linkata nel mio progetto.

Per YELLOW pensavo si potesse sostituire con una variabile con lo stesso valore.

Ho pensato ad una soluzione per rimediare, usare uno SWITCH.
ES:

 void background (int colore)
 {
     int i;
     for (i=1;i<=80;i++) 
    {
         switch(colore)
         {
            case 1:
               textbackground (YELLOW);
            case 2:
                textbackground (RED);
           ......
          }
          printf (" ");
    }
  }

Un CASE per ogni colore possbile. Sapete indicarmi una soluzione migliore?

[XWolverineX]

Lo switch è ottimo.
Forse (ma non so se corre piu' veloce) memorizzi in un intero il risultato e solo dopo il costrutto chiami la funzione.

[KITT]

Grazie ancora. Proverò

[oregon]

No scusa ... allora e' come ti avevo detto io ... quelle sono semplici costanti numeriche ... e quindi non vedo perche' tu non possa scrivere

void background (int colore)
{
  int i;
  for (i=1;i<=80;i++)
  {
    textbackground(colore); 
    printf (" ");
  }
}

e le stringhe non c'entrano nulla ...