Error strcpy was not declared in this scope

In file included from /home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze.cpp:25:0,
                 from /home/ren/Compile/CGAL/cgal/build/Debug/Installation/src/CGAL_ImageIO/all_files.cpp:1:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h: In function ‘imformat* createAnalyzeFormat()’:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:201:54: error: ‘strcpy’ was not declared in this scope
   strcpy(f->fileExtension,".hdr,.hdr.gz,.img,.img.gz");
                                                      ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h: In function ‘int testAnalyzeHeader(char*, const char*)’:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:209:39: error: ‘memcmp’ was not declared in this scope
   if( !memcmp(magic,ANALYZE_LE_MAGIC,4) ||
                                       ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h: In function ‘int writeAnalyze(char*, _image*)’:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:222:21: error: ‘strlen’ was not declared in this scope
   length=strlen(name);
                     ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:225:42: error: ‘strncmp’ was not declared in this scope
   if ( strncmp( name+length-4, ".hdr", 4 ) == 0 ) {
                                          ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:238:47: error: ‘strncpy’ was not declared in this scope
   strncpy( outputName, name, length-extLength );
                                               ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:239:45: error: ‘strncmp’ was not declared in this scope
   if ( strncmp( name+length-7, ".hdr.gz", 7 ) == 0 )
                                             ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:240:52: error: ‘strcpy’ was not declared in this scope
     strcpy( outputName+length-extLength, ".hdr.gz" );
                                                    ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:242:49: error: ‘strcpy’ was not declared in this scope
     strcpy( outputName+length-extLength, ".hdr" );
                                                 ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:265:41: error: ‘strncmp’ was not declared in this scope
   if ( strncmp( name+length-3, ".gz", 3 ) == 0 ) {
                                         ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:266:52: error: ‘strcpy’ was not declared in this scope
     strcpy( outputName+length-extLength, ".img.gz" );
                                                    ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:269:49: error: ‘strcpy’ was not declared in this scope
     strcpy( outputName+length-extLength, ".img" );
                                                 ^
In file included from /home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze.cpp:25:0,
                 from /home/ren/Compile/CGAL/cgal/build/Debug/Installation/src/CGAL_ImageIO/all_files.cpp:1:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h: In function ‘int _readAnalyzeHeader(_image*, const char*, dsr*)’:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:470:101: error: ‘strlen’ was not declared in this scope
       im->user[i] = (char *) ImageIO_alloc((strlen("Data lost in the Analyze -> ImageIO conversion:") + 1));
                                                                                                     ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:532:36: error: ‘strcmp’ was not declared in this scope
  if( strcmp( name+length-4, ".hdr" ) )
                                    ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:542:27: error: ‘strcpy’ was not declared in this scope
  strcpy(data_filename,name);
                           ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h: In function ‘int writeAnalyzeHeader(const _image*)’:
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:598:37: error: ‘memset’ was not declared in this scope
    memset(&hdr,0, sizeof(struct dsr));
                                     ^
/home/ren/Compile/CGAL/cgal/CGAL_ImageIO/src/CGAL_ImageIO/analyze_impl.h:763:36: error: ‘strcpy’ was not declared in this scope
     strcpy(hdr.dime.vox_units,"mm.");
                                    ^
make[2]: *** [Installation/src/CGAL_ImageIO/CMakeFiles/CGAL_ImageIO.dir/all_files.cpp.o] Error 1
make[1]: *** [Installation/src/CGAL_ImageIO/CMakeFiles/CGAL_ImageIO.dir/all] Error 2

Posted By: Anonymous

I get this problem in a c++ problem compiling in Ubuntu g++ version 4.4.3.
I dont know the headers to include to solve this problem.. Thanks

centro_medico.cpp: In constructor ‘Centro_medico::Centro_medico(char*, char*, int, int, float)’:
centro_medico.cpp:5: error: ‘strcpy’ was not declared in this scope
centro_medico.cpp:13: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp:13: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp: In member function ‘Centro_medico& Centro_medico::operator=(const Centro_medico&)’:
centro_medico.cpp:26: error: ‘strcpy’ was not declared in this scope
centro_medico.cpp:39: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp:39: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp: In member function ‘bool Centro_medico::quitar_medico(int)’:
centro_medico.cpp:92: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp:92: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp: In member function ‘void Centro_medico::mostrar_especialidades(std::ostream&) const’:
centro_medico.cpp:123: error: ‘strcmpi’ was not declared in this scope
centro_medico.cpp: In member function ‘void Centro_medico::mostrar_horarios_consulta(char*) const’:
centro_medico.cpp:162: error: ‘strcmpi’ was not declared in this scope
centro_medico.cpp: In member function ‘void Centro_medico::crea_medicos()’:
centro_medico.cpp:321: warning: deprecated conversion from string constant to ‘char*’
centro_medico.cpp:321: warning: deprecated conversion from string constant to ‘char*’

medico.cpp

#include "medico.h"
#include <cstdlib>
#include <iostream>
#include <stdlib>  
#include<cstring>
#include<string>

long Medico::total_consultas=0; 
Medico::Medico(char *nom,char * espe,int colegiado,int trabajo)
{
int i;
strcpy(nombre,nom);
strcpy(especialidad,espe);
num_colegiado=colegiado;
num_horas_diarias=trabajo;
citas_medico= new Cita*[5]; // 5 Días de las semana, de Lunes a Viernes.
for (i=0;i<5;i++)
citas_medico[i]=new Cita[num_horas_diarias];
}



Medico::Medico(const Medico &m){
  int i;
  citas_medico=new Cita*[5];
  for (i=0;i<5;i++)
   citas_medico[i]=NULL;
 (*this) = m;
}

Medico &Medico::operator=(const Medico &m){
 int i,j;
 if (this != &m) { // Para evitar la asignación de un objeto a sí mismo
     strcpy(nombre,m.nombre);
     strcpy(especialidad,m.especialidad);     
     num_colegiado=m.num_colegiado;
     num_horas_diarias=m.num_horas_diarias;
     for (i=0;i<5;i++){
      delete citas_medico[i]; 
      citas_medico[i]=new Cita[num_horas_diarias];
      for(j=0;j<num_horas_diarias;j++){
       citas_medico[i][j] = m.citas_medico[i][j] ;
       }
     }     
  }
 return *this;
}

medico.h

#pragma once
#include <cstdlib>
#include <iostream>
using namespace std;
#include "cita.h"

class Medico
{
 private:
                char nombre[50];
                char especialidad[50];
                int num_colegiado;
                int num_horas_diarias;
                Cita **citas_medico;
                static long total_consultas;                
 public:
                void mostrar_calendario_citas(ostream &o=cout) const;
                bool asignar_cita(int d, int hor,Paciente *p=NULL);
                void anular_cita(int d, int hor);
                bool consultar_cita(char dni[10], int modificar=0);
                void modificar_cita(int d, int hor);                
                void vaciar_calendario_citas();
                void borrar_calendario_citas();                
                char* get_especialidad(char espec[50]) const;
                char* get_nombre(char n[50]) const;
                int get_num_colegiado() const;
                int get_num_horas() const;
                void set_num_horas(int horas);
                void mostrar_info(ostream &o=cout) const;
                static long get_total_consultas();
                Cita* operator[](int dia);
                void eliminar_calendario_citas();
                void crear_calendario_citas();    
                Medico(char *nom,char * espe,int colegiado,int trabajo);
                Medico(const Medico &m);
                Medico &operator=(const Medico &c);
                void operator delete(void*);
                ~Medico();
 };
 ostream& operator<<(ostream &o, Medico &c);
 ofstream& operator<<(ofstream &fichero, Medico &m);
 ifstream& operator>>(ifstream &fichero, Medico &m);

Solution

Observations:

• #include <cstring> should introduce std::strcpy().
• using namespace std; (as written in medico.h) introduces any identifiers from std:: into the global namespace.

Aside from using namespace std; being somewhat clumsy once the application grows larger (as it introduces one hell of a lot of identifiers into the global namespace), and that you should never use using in a header file (see below!), using namespace does not affect identifiers introduced after the statement.

(using namespace std is written in the header, which is included in medico.cpp, but #include <cstring> comes after that.)

My advice: Put the using namespace std; (if you insist on using it at all) into medico.cpp, after any includes, and use explicit std:: in medico.h.

strcmpi() is not a standard function at all; while being defined on Windows, you have to solve case-insensitive compares differently on Linux.

(On general terms, I would like to point to this answer with regards to «proper» string handling in C and C++ that takes Unicode into account, as every application should. Summary: The standard cannot handle these things correctly; do use ICU.)

warning: deprecated conversion from string constant to ‘char*’

A «string constant» is when you write a string literal (e.g. "Hello") in your code. Its type is const char[], i.e. array of constant characters (as you cannot change the characters). You can assign an array to a pointer, but assigning to char *, i.e. removing the const qualifier, generates the warning you are seeing.

OT clarification: using in a header file changes visibility of identifiers for anyone including that header, which is usually not what the user of your header file wants. For example, I could use std::string and a self-written ::string just perfectly in my code, unless I include your medico.h, because then the two classes will clash.

Don’t use using in header files.

And even in implementation files, it can introduce lots of ambiguity. There is a case to be made to use explicit namespacing in implementation files as well.

Answered By: Anonymous

Disclaimer: This content is shared under creative common license cc-by-sa 3.0. It is generated from StackExchange Website Network.

Observations:

• #include <cstring> should introduce std::strcpy().
• using namespace std; (as written in medico.h) introduces any identifiers from std:: into the global namespace.

Aside from using namespace std; being somewhat clumsy once the application grows larger (as it introduces one hell of a lot of identifiers into the global namespace), and that you should never use using in a header file (see below!), using namespace does not affect identifiers introduced after the statement.

(using namespace std is written in the header, which is included in medico.cpp, but #include <cstring> comes after that.)

My advice: Put the using namespace std; (if you insist on using it at all) into medico.cpp, after any includes, and use explicit std:: in medico.h.

strcmpi() is not a standard function at all; while being defined on Windows, you have to solve case-insensitive compares differently on Linux.

(On general terms, I would like to point to this answer with regards to «proper» string handling in C and C++ that takes Unicode into account, as every application should. Summary: The standard cannot handle these things correctly; do use ICU.)

warning: deprecated conversion from string constant to ‘char*’

A «string constant» is when you write a string literal (e.g. "Hello") in your code. Its type is const char[], i.e. array of constant characters (as you cannot change the characters). You can assign an array to a pointer, but assigning to char *, i.e. removing the const qualifier, generates the warning you are seeing.

OT clarification: using in a header file changes visibility of identifiers for anyone including that header, which is usually not what the user of your header file wants. For example, I could use std::string and a self-written ::string just perfectly in my code, unless I include your medico.h, because then the two classes will clash.

Don’t use using in header files.

And even in implementation files, it can introduce lots of ambiguity. There is a case to be made to use explicit namespacing in implementation files as well.