Discussion :

[ChipsterJulien]

Bonjour.
j'ai fait un petit programme qui est en fait un split. C'est très très basique mais c'était pour le plaisir
Voici donc le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#include <iostream>
#include <string>
#include <fstream>
#include <vector>
using namespace std;
 
int main(int argc, char *argv[]) {
 
	ifstream SOURCE("Fichier.txt", ios::in);
	if(!SOURCE) {
		cerr << "Impossible d'ouvrir le fichier !" << endl;
		exit(1);
	}
 
	string s;
	vector<string> mots;
 
	while(getline(SOURCE, s)) {
		bool done = false;
		while(!done) {
			int emplacement = s.find(" ");
			if(emplacement == string::npos) {
				done = true;
			}
			else {
				string mot = s.substr(0, emplacement);
				mots.push_back(mot);
				s = s.substr(emplacement+1, s.size());
			}
		}
	}
 
	for(size_t i = 0; i < mots.size(); ++i)
		cout << mots[i] << endl;
 
	mots.clear();
 
	(void) argc;
	(void) argv;
 
	return 0;
}

Mon soucis c'est qu'à la compilation j'ai ce warning :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
g++ -Wall -W -pedantic -ansi split.cpp -o split
split.cpp: In function ‘int main(int, char**)’:
split.cpp:22: attention : comparaison entre des expressions entières signée et non signée

J'aimerai bien éviter de l'avoir mais ... je ne sais pas comment modifier mon code pour le plus avoir ceci

[edit] Je viens de trouver une solution mais je ne sais pas si c'est vraiment super. J'ai modifier la ligne pour avoir ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(emplacement == (int) string::npos) {

Y a t il une meilleure solution ?

[BoudBoulMan]

Le compilateur t'indique que tu compares deux variables de types différents (un int et un unsigned int). La solution est donc bien de caster l'une des deux variables dans le type de l'autre variable, ce que tu as fait en mettant (int)string::npos.
Tu aurais pu caster la variable 'emplacement' mais cela ne donne aucun avantage par rapport au cast que tu as fait.

Je ne pense pas qu'il y ait d'autres solutions.

[babar63]

Dans ton cas emplacement ne prends pas de valeurs négatives je suppose? Si c'est le cas alors il est préférable de le déclarer directement en unsigned int, cela t'évitera aussi le cast par la même occasion...

[Neitsa]

Bonjour,

attention, string::find et string::pos renvoient tous les deux un size_t (généralement un typedef sur unsigned int).

[Mat007]

Salut,

Il faut juste utiliser le type correct :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
...
std::string::size_type emplacement = s.find(" ");
if(emplacement == string::npos) {
  done = true;
}
...

MAT.

[ChipsterJulien]

Bonjour,

attention, string::find et string::pos renvoient tous les deux un size_t (généralement un typedef sur unsigned int).

Salut,

Il faut juste utiliser le type correct :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
...
std::string::size_type emplacement = s.find(" ");
if(emplacement == string::npos) {
  done = true;
}
...

MAT.

Exact, il faut mettre un size_t emplacement
Merci

[Mat007]

Exact, il faut mettre un size_t emplacement

Techniquement c'est un std::string::size_type, mais bon oui en pratique derrière tout ça c'est des unsigned int.

MAT.

[Médinoc]

Ou plutôt des size_t, non?
(ce n'est pas forcément la même chose quand tu passes en 64 bits)

[Mat007]

Ou plutôt des size_t, non?

Non non c'est bien des std::string::size_type, cf. par exemple http://cppreference.com/cppstring/find.html.

MAT.

[ChipsterJulien]

Non non c'est bien des std::string::size_type, cf. par exemple http://cppreference.com/cppstring/find.html.

MAT.

Ok, j'ai intérêt de faire attention alors car moi j'ai utilisé size_t donc je risquerai de faire des bourdes de programmations

[Médinoc]

Je ne disais pas ça pour string_type, mais pour unsigned int.

[corrector]

Non non c'est bien des std::string::size_type, cf. par exemple http://cppreference.com/cppstring/find.html.

Allons plutôt à la source de l'information : C++ draft (N2451). Suivez le guide!

Commençons par 21.2 String classes [string.classes] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

typedef basic_string<char> string;

défini par 21.3 Class template basic_string [basic.string] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
template<class charT, class traits = char_traits<charT>,
class Allocator = allocator<charT> >
class basic_string {
    typedef typename Allocator::size_type size_type;

allons voir 20.6.1 The default allocator [default.allocator] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
template <class T> class allocator {
public:
typedef size_t size_type;

Voilà : pas la peine de s'embarrasser avec une notation lourde, size_t est strictement équivalent à string::size_type.

CQFD

[ChipsterJulien]

Allons plutôt à la source de l'information : C++ draft (N2451). Suivez le guide!

Commençons par 21.2 String classes [string.classes] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

typedef basic_string<char> string;

défini par 21.3 Class template basic_string [basic.string] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
template<class charT, class traits = char_traits<charT>,
class Allocator = allocator<charT> >
class basic_string {
    typedef typename Allocator::size_type size_type;

allons voir 20.6.1 The default allocator [default.allocator] :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
template <class T> class allocator {
public:
typedef size_t size_type;

Voilà : pas la peine de s'embarrasser avec une notation lourde, size_t est strictement équivalent à string::size_type.

CQFD

Merci pour ces renseignements

[nicroman]

Ca va être chaud alors pour couper un ficher de 6Go dans un programme 32 bits !

Moi qui croyais qu'une position dans un fichier était un fpos_t... soit... un unsigned __int64 qu'on soit sous 32 ou 64 bits.

[Médinoc]

Sous POSIX, il me semble que c'est un off_t ou (hélas) un ssize_t, mais donc rien qui fasse 64bits sous 32bits...

À ma connaissance, il n'y a rien en standard pour ça, mais il y a des extensions sous certains systèmes (Windows propose fseek64() pour les fichier C)...

[corrector]

Ca va être chaud alors pour couper un ficher de 6Go dans un programme 32 bits !

Je ne vois pas du tout le rapport.