Copie d'un Répertoire

**Mangacker** · 14/04/2009, 15h46

Bonjour,

encore une fois, je fais appel à vos connaissances...

voici mon problème :
j'ai besoin de copier le contenu d'un répertoire dans un autre afin d'effectuer des modifications sur les fichiers copiés.

j'utilise donc la fonction présentée ici (code de blondelle) :
http://www.developpez.net/forums/d43...r/#post2602788

ce qui me donne le code suivant :

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
 
//---------------------------------------------------------------------------
void __fastcall TufmImages::save_finalCanClose(TObject *Sender, bool &CanClose)
{
	// copie complète du répertoire source dans le répertoire final
 
	AnsiString tmpSource = ExtractFileDir(fichierSource);
	AnsiString tmpFinal = ExtractFileDir(save_final->FileName);
 
	tmpSource += "\0";
	tmpFinal += "\0";
 
//	ShowMessage(tmpSource);
//	ShowMessage(tmpFinal);
 
	CopyDir(tmpSource.c_str(), tmpFinal.c_str());
}
//---------------------------------------------------------------------------
void TufmImages::CopyDir(char* SourceDir, char* DestDir)
{
//	SourceDir = "C:\\tmp\\test\0";
//	DestDir = "C:\\tmp\\new\0";
 
	SHFILEOPSTRUCT FileOpStr;
	ZeroMemory(&FileOpStr, sizeof(FileOpStr));
	FileOpStr.hwnd = NULL;
	FileOpStr.wFunc = FO_COPY;
	FileOpStr.pFrom = SourceDir;
	FileOpStr.pTo = DestDir;
	SHFileOperation(&FileOpStr);
}
//---------------------------------------------------------------------------

mais j'ai l'erreur suivante :

Cannot copy file: Cannot read from the source file or disk.

avez - vous une idée d'où peut provenir cette erreur?? sachant que si je retire les // ici :

Code :

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

1
2
3
 
//	SourceDir = "C:\\tmp\\test\0";
//	DestDir = "C:\\tmp\\new\0";

mon code fonctionne (mais ce n'est pas le but...

)

merci encore une fois pour votre aide
à bientôt

**getz85** · 14/04/2009, 15h51

Je sais pas d'ou peut venir ton problème, mais as-tu vérifé si la valeur que prenait tmpSource et tmpFinal lors de l'éxécution de ton code était bonne?

**Mangacker** · 14/04/2009, 15h54

woahoo jamais vu de réponse aussi rapide merci...

oui les valeurs "m'ont l'air" correctes

ShowMessage(tmpSource); donne :
"C:\tmp\test"

ShowMessage(tmpFinal); donne :
"C:\tmp\new

**getz85** · 14/04/2009, 16h02

Peut-être faut-il doubler les antislash dans tes ansiString avant d'appeler la fonction CopyDir.

**nirgal76** · 14/04/2009, 16h11

Je ne vois pas bien l'interêt de tes 2 lignes :

Code :

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

1
2
3
 
tmpSource += "\0";
tmpFinal += "\0";

puisque le zéro terminal sera ajouté par c_str() au moment du passage des argument à la fonction

**Mangacker** · 14/04/2009, 16h17

c'est ce à quoi j'avais pensé mais comme la valeur vient directement d'une fonction ExtractFileDir je pensais que c'était ok.

de plus, si j'essaye de les doubler à l'aide de la fonction AnsiReplaceText

ce qui me donne :

Code :

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

1
2
 
AnsiReplaceText("C:\tmp\test\0", "\\", "\\\\");

malheureusement là j'obtiens une chaîne bizarre :

Code :

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

1
2
 
"C:    mp    est"

**Mangacker** · 14/04/2009, 16h18

Envoyé par nirgal76

Je ne vois pas bien l'interêt de tes 2 lignes :

Code :

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

1
2
3
 
tmpSource += "\0";
tmpFinal += "\0";

puisque le zéro terminal sera ajouté par c_str() au moment du passage des argument à la fonction

ah merci je ne le savais pas...

j'essaye de les enlever et je te redis

Envoyé par edition du message par mangacker

ça ne change rien que je laisse ou pas ces lignes...

elles sont là (comme indiqué par blondelle je crois) pour indiquer qu'il faut copier le contenu du répertoire et pas le répertoire lui-meme

si c'est pas pour ça, ça serait cool de m'expliquer ^^

a+

**pottiez** · 14/04/2009, 16h31

Envoyé par Mangacker

c'est ce à quoi j'avais pensé mais comme la valeur vient directement d'une fonction ExtractFileDir je pensais que c'était ok.

C'est ok

Envoyé par Mangacker

de plus, si j'essaye de les doubler à l'aide de la fonction AnsiReplaceText

ce qui me donne :

Code :

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

1
2
 
AnsiReplaceText("C:\tmp\test\0", "\\", "\\\\");

malheureusement là j'obtiens une chaîne bizarre :

Code :

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

1
2
 
"C:    mp    est"

La par contre tu essaye un chaine brut que tu met toi directement dans ton code, du coup il faut doubler tes antislash, de plus comment veut tu trouver des double antislash dans une chaine en dur ou tu voit bien qu'il n'y en as pas

**getz85** · 14/04/2009, 16h32

Envoyé par Mangacker

c'est ce à quoi j'avais pensé mais comme la valeur vient directement d'une fonction ExtractFileDir je pensais que c'était ok.

de plus, si j'essaye de les doubler à l'aide de la fonction AnsiReplaceText

ce qui me donne :

Code :

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

1
2
 
AnsiReplaceText("C:\tmp\test\0", "\\", "\\\\");

malheureusement là j'obtiens une chaîne bizarre :

Code :

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

1
2
 
"C:    mp    est"

Alors là je comprends pas trop ce résultat....En plus j'ai rien sous la main pour tester

Essaye de faire ta propre fonction pour ajouter l'antislash peut-être...ou attends quelqu'un de plus éxpérimenté que moi

**pottiez** · 14/04/2009, 16h39

Envoyé par Suffocation

Alors là je comprends pas trop ce résultat....En plus j'ai rien sous la main pour tester

Essaye de faire ta propre fonction pour ajouter l'antislash peut-être...ou attends quelqu'un de plus éxpérimenté que moi

Ce résultat est normal, ici dans le text "C:\tmp\test\0", les \ sont des caractère d'échappement ce caractère associé à un caractère t donne une tabulation (d'où le grand espace entre chaque groupe de lettre qui se suive), si tu l'avait associé à un caractère n tu aurais eu des retour à la lignes et pour avoir un carcatère \ d'affciher il faut doubler ce caractère d'échappement

**getz85** · 14/04/2009, 16h44

Envoyé par pottiez

Ce résultat est normal, ici dans le text "C:\tmp\test\0", les \ sont des caractère d'échappement ce caractère associé à un caractère t donne une tabulation (d'où le grand espace entre chaque groupe de lettre qui se suive), si tu l'avait associé à un caractère n tu aurais eu des retour à la lignes et pour avoir un carcatère \ d'affciher il faut doubler ce caractère d'échappement

oui c'est logique une fois bien expliqué

**Mangacker** · 14/04/2009, 17h02

Envoyé par pottiez

Ce résultat est normal, ici dans le text "C:\tmp\test\0", les \ sont des caractère d'échappement ce caractère associé à un caractère t donne une tabulation (d'où le grand espace entre chaque groupe de lettre qui se suive), si tu l'avait associé à un caractère n tu aurais eu des retour à la lignes et pour avoir un carcatère \ d'affciher il faut doubler ce caractère d'échappement

donc si je veux m'en sortir, il faut que je crée ma propre fonction pour doubler les '\' et mettre le résultat dans mes variables tmpSource et tmpFinal...

**totoche76** · 14/04/2009, 17h07

Hello,

Le code fourni par Blondelle était correct car il utilisait un tableau de char dans lequel il rajoutait un '\0', en plus du '\0' de fin de chaîne (cf MSDN - SHFILEOPSTRUCT).
Par contre, si tu joues avec des AnsiStrings, tu paumes tout ce qui peut dépasser de ton 0 terminal...

Essaye ceci:

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
void __fastcall TufmImages::save_finalCanClose(TObject *Sender, bool &CanClose)
{
	// copie complète du répertoire source dans le répertoire final
 
	AnsiString tmpSource = ExtractFileDir(fichierSource);
	AnsiString tmpFinal = ExtractFileDir(save_final->FileName);
 
  // Un 0 pour terminer la chaine, et un autre pour terminer le buffer 
  char* tmpSourceChar = new char[tmpSource.Length() + 2];
  ZeroMemory(tmpSourceChar, tmpSource.Length() + 2);
  StrCopy(tmpSourceChar, tmpSource.c_str());
 
  char* tmpFinalChar = new char[tmpFinal.Length() + 2];
  ZeroMemory(tmpFinalChar, tmpFinal.Length() + 2);
  StrCopy(tmpFinalChar, tmpFinal.c_str());
 
	//CopyDir(tmpSource.c_str(), tmpFinal.c_str());
	CopyDir(tmpSourceChar, tmpFinalChar);
  delete []tmpSourceChar;
  delete []tmpFinalChar;
}

**pottiez** · 14/04/2009, 17h12

Envoyé par Mangacker

donc si je veux m'en sortir, il faut que je crée ma propre fonction pour doubler les '\' et mettre le résultat dans mes variables tmpSource et tmpFinal...

Si tu entre tes chemins de répertoire via l'interface graphique tu n'as pas besoin de le faire, le compilo le fait à ta place, part contre si tu met tes chemins en dur dans ton code, la c'est à toi de faire attention au doublement des antislash

**totoche76** · 14/04/2009, 17h14

Envoyé par Mangacker

donc si je veux m'en sortir, il faut que je crée ma propre fonction pour doubler les '\' et mettre le résultat dans mes variables tmpSource et tmpFinal...

J'ai rien compris à cette histoire de '\'....

Si tu dois dans ton code écrire le caractère '\', il faut que tu le doubles, et c'est tout:

Code :

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

1
2
char* toto="ben\\oui";  // Un antislash
char* tutu="ben\\\\oui"; // Double antislash

La chaîne que tu reçois en retour de ExtractFileDir contient des antislash corrects, y'a pas à y toucher...

**gandf** · 14/04/2009, 20h48

Pour la fonction de remplacement, il y a aussi cette fonction :

Code :

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

txttmp = StringReplace(txttmp, "\\", "\\\\", TReplaceFlags() << rfReplaceAll);

**Mangacker** · 15/04/2009, 08h14

Envoyé par totoche76

La chaîne que tu reçois en retour de ExtractFileDir contient des antislash corrects, y'a pas à y toucher...

ok merci

le prob c'est que quand je l'affiche il supprime 1 '/' sur 2 c'est pour ça que j'essayais de les doubler..

merci à tous mon problème est maintenant Résolu

à bientôt pour une nouvelle question.
bye

**Mangacker** · 15/04/2009, 09h25

Envoyé par gandf

Pour la fonction de remplacement, il y a aussi cette fonction :

Code :

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

txttmp = StringReplace(txttmp, "\\", "\\\\", TReplaceFlags() << rfReplaceAll);

merci je ne connaissais pas cette fonction je la testerai si j'ai un peu de temps

sinon une dernière question :
quand j'execute mon programme, je voudrai l'arborescence suivante

dossier de base :
c:\tmp\base\

dossier de destination :
c:\tmp\destination\

mais en fait, il me rajoute un niveau dans mes répertoires :
c:\tmp\destination\base

savez - vous comment l'en empêcher?

**totoche76** · 15/04/2009, 10h22

Hello,

Tu lui demandes de copier le répertoire '<..>\base', donc il recrée le répertoire 'base', normal dirais-je...

Si tu veux uniquement copier les fichiers contenus dans tmpSource, travailles avec tmpSource + "\\*.*", ça devrais mieux aller