FTP PHP versioning synchronizer

[bullx]

Bonjour à tous,

J'ai développé un petit système de synchronisation entre répertoire local de travail et ftp distant.
Il permet de faire un get : récupération des fichier du serveur et un submit : envoi des fichiers modifiés sur le serveur.
On peut aussi filtrer avec des expressions régulière les fichiers qui doivent échapper à la synchronisation.

Pouvez-vous me dire ce que vous en pensez ?
http://my.opera.com/metrallik/blog/2...sioning-system

Cordialement, bullx

[stealth35]

c'est vraiment fouillis, de plus si on a pas l'extension ftp on la dans l'os (alors que le wrapper ftp est toujours la)
t'utilises pas les les iterators pour parcourir tes dossiers,

c'est quoi ca : $tmpArr1=explode("\n",file_get_contents($tmpFile)); a part file mais en moins bien ?
surtout que c'est vraiment pas la meilleurs solution pour savoir si le fichier est identique un hash aurai plus judicieux et beaucoup plus rapide

et y'a des commentaires en français et en anglais

[bullx]

Le ton est un peu rude mais merci quand même pour ces précieux conseils.

J'ai honte de mon explode ...

Je connaissais pas les itérateurs (encore la honte ?)

Pour le wrapper, c'est peut-être bête mais j'avais l'impression que c'était susceptible de générer du trafic inutile (identification à chaque manipulation).

Pour les commentaires, c'est quoi le mieux ?

Je vais utiliser un hash pour les comparaison.

[stealth35]

oui en effet en relisant ça fait pas très sympa, rajoute des ( ) un peu partout

sinon pour les commentaire c'est mieux de tout mettre en anglais, pour la syntaxe n’hésite pas a passer les lignes c'est plus clair

Code :

if(!is_dir(dirname("$cHistDir/$f"))) mkdir(dirname("$cHistDir/$f"),0777,true);

Code :

1
2
3
4
5
6
$dir = dirname("$cHistDir/$f");
 
if(!is_dir($dir))
{
    mkdir($dir, 0777, true);
}

[bullx]

Bon alors je ne suis pas trop satisfait.

J'ai tout changé en utilisant les itérateurs et le wrapper ftp (ce qui m'a pris plusieurs heures) mais ça ne marche pas (encore).

J'ai un problème, le serveur passe en mode ASCII ... Donc j'ai essayé avec type=i; à la fin mais ça plante.

Si je peux pas utiliser de wrapper alors je ne peux pas non plus utiliser d'itérateur et tout ce que j'ai fait est inutile.

Par ailleurs, quand bien même j'arrive à ajouter type=X au bout des adresses, il faudra que je transforme chacune d'entre elle ou que j'ajoute (encore) une variable de configuration pour définir un type ...

[stealth35]

montre ce que t'as fait

[bullx]

J'ai mis à jour sur http://my.opera.com/metrallik/blog/2...sioning-system ; la dernière version y est disponible.

[stealth35]

aère ton code
c'est pas très lisible ça :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
if($argv[1]=='diff'){
  if(empty($argv[2])) exit("Sorry, diff on whole project not supported yet\n");
  if(!is_file($argv[2])) exit("Error : file does not exist locally\n");
  if(!is_file("$cRemDir/$argv[2]")) exit("Error : remote file does not exist\n");
  $tmpFile="$cHistDir/${argv[2]}.ftp.".time();
  if(!is_dir(dirname($tmpFile))) mkdir(dirname($tmpFile),0777,true);
  copy("$cRemDir/$argv[2]",$tmpFile);
  if(md5_file($tmpFile)==md5_file($argv[2])) echo "Files are identical (same hash)\n";
  # À faire : Eventuellement mettre à jour le repository (si les fichiers y sont et sont identiques)
  else{
    echo "On server and not in wa : \n";
    print_r(array_diff(file($tmpFile,FILE_IGNORE_NEW_LINES),file($argv[2],FILE_IGNORE_NEW_LINES)));
    echo "In wa and not on server : \n";
    print_r(array_diff(file($argv[2],FILE_IGNORE_NEW_LINES),file($tmpFile,FILE_IGNORE_NEW_LINES)));
  }
}

contrairement a :

Code :

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
if($argv[1] == 'diff')
{
    if(empty($argv[2]))
    {
        exit "Sorry, diff on whole project not supported yet\n";
    }
 
    if(!is_file($argv[2]))
    {
        exit "Error : file does not exist locally\n";
    }
 
    if(!is_file("$cRemDir/$argv[2]"))
    {
        exit "Error : remote file does not exist\n";
    }
 
    $tmpFile = "$cHistDir/${argv[2]}.ftp.".time();
 
    if(!is_dir(dirname($tmpFile)))
    {
        mkdir(dirname($tmpFile), 0777, true);
    }
 
    copy("$cRemDir/$argv[2]", $tmpFile);
 
    if(md5_file($tmpFile) == md5_file($argv[2]))
    {
        echo "Files are identical (same hash)\n";
    }
}
else
{
    echo "On server and not in wa : \n";
    print_r(array_diff(file($tmpFile,FILE_IGNORE_NEW_LINES), file($argv[2],FILE_IGNORE_NEW_LINES)));
 
    echo "In wa and not on server : \n";
    print_r(array_diff(file($argv[2],FILE_IGNORE_NEW_LINES), file($tmpFile,FILE_IGNORE_NEW_LINES)));
}

c'est inutile d’économiser des lignes de code

[bullx]

Moi, ça me va. Et puis, cela ne m'aide pas pour mon problème d'ASCII avec le wrapper ftp...

J'ai cherché vite fait des histoires de contexte de wrapper mais sans succès.

[stealth35]

Envoyé par bullx

Moi, ça me va. Et puis, cela ne m'aide pas pour mon problème d'ASCII avec le wrapper ftp...

J'ai cherché vite fait des histoires de contexte de wrapper mais sans succès.

il faut rajouter le type pour le wrapper ftp