Discussion :

[daffy85]

bonjour
voila j'ai créer une fonction que j'utilise dans deux pages php différentes dans une des pages j'ai des variable qui ne sont pas utiliser voici ma fonction

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
function upload($file, $post){
	//on récupère la valeur du fichier d'entrée et de sortie de la liste déroulante
	$fichierEntree = $post['entreeFichier'];	
	$fichierSortie = $post['sortieFichier'];
	$projectionEntree = $post['projectionEntree'];
	$projectionSortie = $post['projectionSortie'];
 
	//chemin ou on depose le fichier à traiter
	$trajet = './upload/';
	//on recupère le fichier avec le bouton parcourir
	$fic = basename($file['fichierConvertit']['name']);
	//on identifie le chemin du fichier qu'on veut traiter
	$cheminFichier = $trajet.$fic;
	//on récupère les extensions du fichier d'entrée et du fichier de sortie
	$extensionEntree = strtolower(pathinfo($fic,PATHINFO_EXTENSION));
	var_dump($extensionEntree);
	$extensionSortie = strrchr($fichierSortie,'.');
	//on déplace le fichier pour pouvoir le traiter.
	if(move_uploaded_file($file['fichierConvertit']['tmp_name'],$trajet.$fic)){
		// si le fichier de sortie est différent de .ov2 ou .csv on créer un fichier de sortie .
		if($extensionSortie != '.ov2' && $extensionSortie != '.CSV' && $extensionSortie != '.xlsx'){
			//on créé le fichier de sortie.
			$nouveauxFichier = $trajet.'traitement/'.pathinfo($fic,PATHINFO_FILENAME).'_convertit'.$extensionSortie;
		}else{
			$nouveauxFichier = NULL;
		}
	return array('cheminFichier'=>$cheminFichier, 'nouveauxFichier'=>$nouveauxFichier, 'fichierEntree'=>$fichierEntree, 'fichierSortie'=>$fichierSortie, 'extensionEntree'=>$extensionEntree, 'extensionSortie'=>$extensionSortie, 'projectionSortie'=>$projectionSortie, 'projectionEntree'=>$projectionEntree, 'fichier'=>$fic, 'trajet'=>$trajet );
	}else{
		echo '<p class = "reponseFichier">erreur de chargment du fichier</br></p>';
	}
}

j'aimerais faire un truc du genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if( parametre){
$fichierEntree = $post['entreeFichier'];	
	$fichierSortie = $post['sortieFichier'];
	$projectionEntree = $post['projectionEntree'];
	$projectionSortie = $post['projectionSortie'];
}else{
$fichierEntree = null;	
	$fichierSortie = null;
	$projectionEntree = null;
	$projectionSortie = null
}

mais je ne sais pas quoi mettre dans les parametre
merci pour votre aide

[sabotage]

Pourquoi y a t'il ces 4 variables dans la fonction puisqu'elles ne sont pas utilisées ?

[daffy85]

je m'excuse je me suis mal expliquer en fait elles sont utiliser dans une des deux pages.

[sabotage]

oui mais elles ne sont pas utilisées dans la fonction donc pourquoi les mettre ?

[daffy85]

elle sont utilisé sinon j'ai un message qui m'indique que la variable n'est pas defini
tu me met le doute je regarde demain et je te dis sa.

[sabotage]

Tu devrais surtout revoir le sens fonctionnel de cette fonction.
c'est une fonction qui traite la reception d'un fichier uploadé, le seul paramètre dont elle a besoin c'est le seul que tu ne lui donnes pas : le nom du champs d'upload.
le seul retour qu'elle devrait avoir c'est le statut du traitement : succès ou erreur ; au lieu de cela tu renvoies tout une série de variable qui sont fixes (par exemple $cheminFichier n'est pas dépendant de la fonction, sa valeur est connue avant le traitement).

$files ne sert à rien, autant utiliser directement $_FILES
$post ce sont des variables qui n'ont rien à voir avec cette fonction ; que tu en aies besoin après, peut-être mais ne les fais pas transiter dans cette fonction.