Maximum execution time

**seb-65** · 12/06/2012, 12h11

Bonjour,

J'ai un script qui permet de faire de la copie récurissive de fichiers/dossiers

Le problème est que quand les dossiers sont trop "rempli" j'obtiens le message d'erreur :

stderr: PHP Fatal error: Maximum execution time of 30 seconds exceeded in

Quel est selon vous la valeur recommandé pour max_execution_time

Merci

**Benjamin Delespierre** · 12/06/2012, 13h31

ça dépends. Si tu es confiant dans la capacité de ton script à faire son boulot sans planter, tu peux mettre 0. Si tu sais que de toute façon ça ne dépassera jamais X secondes, mets X tout simplement.

Note: tu peux définir le max_execution_time au runtime avec set_time_limit.

Et pour répondre explicitement à la question, 30 secondes c'est recommandé dans la plupart des cas (personnellement, je n'ai jamais conçu d'algorithme qui le dépasse).

**Jean-Georges** · 12/06/2012, 13h44

Tu peux également réinitialisée cette durée à chaque passage de la boucle.

**Celira** · 12/06/2012, 13h49

Je plussoie la valeur des 30 secondes. Si une page web met plus de 30 secondes à s'afficher, en règle générale, l'utilisateur va pas comprendre et réactualiser

Maintenant, tu peux dépasser cette valeur dans certains cas, par exemple : un script batch lancé par une tâche périodique, un export de données particulièrement fourni, ce genre de choses.
Par contre, si c'est une page web, il faut penser à au moins mettre un message du genre "ce traitement peut mettre quelques minutes à s'exécuter"...

**seb-65** · 12/06/2012, 14h24

Bonjour merci pour vos réponses

@Celira

Si une page web met plus de 30 secondes à s'afficher, en règle générale, l'utilisateur va pas comprendre et réactualiser

Non je pense pas... J'utilise JQuery/Ajax pour lancer ma fonction de copie récurssive. EN fait temps que la copie n'est pas terminée, un message apparait.

Là, mon problème c'est que la copie est trop longue donc timeout du script php !

@Jean-Georges

Tu peux également réinitialisée cette durée à chaque passage de la boucle.

Oui cela est une idée, mais comment faire pour modifier le timer ?

**Benjamin Delespierre** · 12/06/2012, 14h27

Tiens comme je l'ai dans mon presse-papier, voici un algo tout bête pour déplacer des fichiers:

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
function copy_dir ($from, $to, $reccursive = true, array & $files = null) {
    if (!is_dir($from) || !is_readable($from))
        throw new InvalidArgumentException("First parameter is not a valid directory or is not readable");
 
    if (!is_dir($to) || !is_writable($to))
        throw new InvalidArgumentException("Second parameter is not a vadlid directory or is not writeable");
 
    $from = realpath($from);
    $to   = realpath($to);
 
    if ($reccursive) {
        $dir = new RecursiveDirectoryIterator($from);
        $dir = new RecursiveIteratorIterator($dir);
    }
    else {
        $dir = new DirectoryIterator($from);
    }
 
    $files = array();
    foreach ($dir as $item) {
        $dirname          = realpath($item->getPath());
        $filename         = realpath($item->getPathname());
        $new_dirname      = str_replace($from, $to, $dirname);
        $new_filename     = str_replace($from, $to, $filename);
 
        if (!$reccursive && $item->isDir())
            continue;
        elseif(!isset($files[$new_dirname]) && !is_dir($new_dirname))
            $files[$new_dirname] = mkdir($new_dirname);
 
        $files[$new_filename] = copy($filename, $new_filename);
    }
 
    return !in_array(false, $files, true);
}

Mais c'est vrai que ça prends du temps.

**dré kam** · 12/06/2012, 19h25

Pour modifier le timer:

Code :

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

1
2
 
ini_set ('max_execution_time', 0);

**Benjamin Delespierre** · 13/06/2012, 10h15

Envoyé par Benjamin Delespierre

Note: tu peux définir le max_execution_time au runtime avec set_time_limit.

Bon, ini_set fonctionne aussi mais il vaut mieux utiliser les fonctions pré-définies

**seb-65** · 13/06/2012, 10h56

Bonjour,

Voici le code que j'utilise pour la copie récurssive, quand pensez-vous :

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
 
function copy_directory($source,$destination ) {
	if ( is_dir( $source ) ) {
		@mkdir( $destination );
		$directory = dir( $source );
		while ( FALSE !== ( $readdirectory = $directory->read() ) ) {
			if ( $readdirectory == '.' || $readdirectory == '..' ) {
				continue;
			}
			$PathDir = $source . '/' . $readdirectory;
			if ( is_dir( $PathDir ) ) {
				copy_directory( $PathDir, $destination . '/' . $readdirectory );
				continue;
			}
			copy( $PathDir, $destination . '/' . $readdirectory );
		}
		$directory->close();
		}else {
			copy( $source, $destination );
		}
	}

Ok, je vais utiliser set_time_limit(0); avant le lancement de ma fonction

**Benjamin Delespierre** · 13/06/2012, 11h34

quand pensez-vous

Assez souvent en général

Mis à part le @ qui se tiend là comme une belle verrue et que la logique de contrôle laisse à désirer, elle à l'air de faire son travail.

Cela étant, je trouve la mienne un poil plus simple et plus souple.

Voici le bench que j'ai effectué:

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
function copy_directory($source,$destination ) {
	if ( is_dir( $source ) ) {
		@mkdir( $destination );
		$directory = dir( $source );
		while ( FALSE !== ( $readdirectory = $directory->read() ) ) {
			if ( $readdirectory == '.' || $readdirectory == '..' ) {
				continue;
			}
			$PathDir = $source . '/' . $readdirectory;
			if ( is_dir( $PathDir ) ) {
				copy_directory( $PathDir, $destination . '/' . $readdirectory );
				continue;
			}
			copy( $PathDir, $destination . '/' . $readdirectory );
		}
		$directory->close();
		}else {
			copy( $source, $destination );
		}
	}
 
function copy_dir ($from, $to, $reccursive = true, array & $files = null) {
    if (!is_dir($from) || !is_readable($from))
        throw new InvalidArgumentException("First parameter is not a valid directory or is not readable");
 
    if (!is_dir($to) || !is_writable($to))
        throw new InvalidArgumentException("Second parameter is not a vadlid directory or is not writeable");
 
    $from = realpath($from);
    $to   = realpath($to);
 
    if ($reccursive) {
        $dir = new RecursiveDirectoryIterator($from);
        $dir = new RecursiveIteratorIterator($dir);
    }
    else {
        $dir = new DirectoryIterator($from);
    }
 
    $files = array();
    foreach ($dir as $item) {
        $dirname          = realpath($item->getPath());
        $filename         = realpath($item->getPathname());
        $new_dirname      = str_replace($from, $to, $dirname);
        $new_filename     = str_replace($from, $to, $filename);
 
        if (!$reccursive && $item->isDir())
            continue;
        elseif(!isset($files[$new_dirname]) && !is_dir($new_dirname))
            $files[$new_dirname] = mkdir($new_dirname);
 
        $files[$new_filename] = copy($filename, $new_filename);
    }
 
    return !in_array(false, $files, true);
}
 
set_time_limit(0);
 
define('SOURCE', 'D:/temp/test/1');
define('DESTINATION', 'D:/temp/test/%s');
 
echo "<pre>";
for ($i = 0; $i < 10; $i++) {
  echo "TEST #$i\n";
  mkdir($dest_a = sprintf(DESTINATION, "$i.1"));
  mkdir($dest_b = sprintf(DESTINATION, "$i.2"));
 
  _st();
  copy_directory(SOURCE, $dest_a);
  echo "copy_directory() took " . _nd() . " seconds\n";
 
  _st();
  copy_dir(SOURCE, $dest_b);
  echo "copy_dir() took " . _nd() . " seconds\n";
}

Résultat:

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
TEST #0
copy_directory() took 1.122 seconds
copy_dir() took 2.201 seconds
TEST #1
copy_directory() took 2.876 seconds
copy_dir() took 4.613 seconds
TEST #2
copy_directory() took 3.635 seconds
copy_dir() took 3.054 seconds
TEST #3
copy_directory() took 2.643 seconds
copy_dir() took 2.923 seconds
TEST #4
copy_directory() took 2.643 seconds
copy_dir() took 2.301 seconds
TEST #5
copy_directory() took 2.466 seconds
copy_dir() took 2.397 seconds
TEST #6
copy_directory() took 2.811 seconds
copy_dir() took 2.837 seconds
TEST #7
copy_directory() took 3.259 seconds
copy_dir() took 2.406 seconds
TEST #8
copy_directory() took 2.027 seconds
copy_dir() took 1.604 seconds
TEST #9
copy_directory() took 1.964 seconds
copy_dir() took 1.545 seconds

On remarque que copy_directory() est relativement stable en performances alors que la rapidité de copy_dir() augmente avec le nombre d'itération (probablement dû au cache de PHP).

Les tests ont été effectués avec un Core2 Duo 2.26Ghz et 2,86Mo de RAM (128Mo pour PHP) sous WinXP SP3 avec PHP 5.3.10 et un jeu de données de ~45Mo répartis sur 3 sous dossiers.