web-dev-qa-db-ja.com

エラー:strcpyはこのスコープで宣言されていません

この問題は、Ubuntu g ++バージョン4.4.3でコンパイルしているc ++問題で発生します。この問題を解決するために含めるヘッダーがわかりません。

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);
25
Chak

観察:

  • _#include <cstring>_はstd :: strcpy()を導入する必要があります。
  • _using namespace std;_(medico.hで記述)は、_std::_の識別子をグローバル名前空間に導入します。

_using namespace std;_は別として、アプリケーションが大きくなると(グローバルネームスペースに大量の識別子が1つ混入するため)多少不器用になるため、neverヘッダーファイルでusingを使用します(以下を参照してください!)、_using namespace_は、導入された識別子に影響しませんafterステートメント。

(_using namespace std_はmedico.cppに含まれるヘッダーに書き込まれますが、_#include <cstring>_はafter thatになります。)

私のアドバイス:_using namespace std;_をmedico.cppに入れ、すべてのインクルードの後にし、明示的な_std::_ medico.h内.


strcmpi()は標準関数ではありません。 Windowsで定義されている場合、Linuxでは大文字と小文字を区別しない比較を異なる方法で解決する必要があります。

(一般的に言えば、すべてのアプリケーションがそうであるように、Unicodeを考慮に入れるCおよびC++での「適切な」文字列処理に関して この回答 を指し示したいと思います。要約:標準cannotこれらのことを正しく処理します; do use [〜#〜] icu [〜#〜] 。)


_warning: deprecated conversion from string constant to ‘char*’
_

「文字列定数」とは、コードに文字列リテラル(_"Hello"_など)を記述する場合です。そのタイプは_const char[]_、つまりconstant文字の配列です(文字を変更できないため)。配列をポインターに割り当てることができますが、_char *_に割り当てる、つまりconst修飾子を削除すると、表示されている警告が生成されます。


OTの明確化:ヘッダーファイル内のusingは、そのヘッダーを含むすべてのユーザーの識別子の可視性を変更します。これは通常、ヘッダーファイルのユーザーが望んでいるものではありません。たとえば、コードで_std::string_と自己記述__::string_を完全に使用できます。2つのクラスが衝突するため、medico.hを含めない限りです。

ヘッダーファイルでusingを使用しないでください。

44
DevSolar

あなたが言う時:

 #include <cstring>

g ++コンパイラは、それ自体に含まれる<string.h>宣言をstd:: ANDグローバル名前空間に配置する必要があります。何らかの理由で、あたかもそれをしていないかのように見えます。 strcpyの1つのインスタンスをstd::strcpyに置き換えてみて、問題が解決するかどうかを確認してください。

1
anon

このエラーは、次のような状況で発生することがあります。

#ifndef NAN
#include <stdlib.h>
#define NAN (strtod("NAN",NULL))
#endif

static void init_random(uint32_t initseed=0)
{
    if (initseed==0)
    {
        struct timeval tv;
        gettimeofday(&tv, NULL);
        seed=(uint32_t) (4223517*getpid()*tv.tv_sec*tv.tv_usec);
    }
    else
        seed=initseed;
#if !defined(CYGWIN) && !defined(__INTERIX)
    //seed=42
    //SG_SPRINT("initializing random number generator with %d (seed size %d)\n", seed, RNG_SEED_SIZE)
    initstate(seed, CMath::Rand_state, RNG_SEED_SIZE);
#endif
}

次のコード行が実行時に実行されない場合:

#ifndef NAN
#include <stdlib.h>
#define NAN (strtod("NAN",NULL))
#endif

次のようなコードのエラーが発生します。 initstatestdlib.hファイルと含まれない

In file included from ../../shogun/features/SubsetStack.h:14:0, 
                 from ../../shogun/features/Features.h:21, 
                 from ../../shogun/ui/SGInterface.h:7, 
                 from MatlabInterface.h:15, 
                 from matlabInterface.cpp:7: 
../../shogun/mathematics/Math.h: In static member function 'static void shogun::CMath::init_random(uint32_t)': 
../../shogun/mathematics/Math.h:459:52: error: 'initstate' was not declared in this scope
0
MRS1367