Le 16/02/2013 13:52, niconil a écrit :
>
> In actions/actionpackdata/code/file.cpp:
>
> > @@ -35,7 +35,15 @@
> >  {
> >  	QScriptValue File::constructor(QScriptContext *context, QScriptEngine *engine)
> >  	{
> > -		return CodeClass::constructor(new File, context, engine);
> > +		if(context->argumentCount() > 1)
> > +		{
> > +			throwError(context, engine, "ParameterCountError", tr("Incorrect parameter count"));
> > +			return false;
> > +		}
> > +		if(context->argumentCount() == 1)
> > +			return CodeClass::constructor(new File(context->argument(0).toString()), context, engine);
> > +		else
> > +			return CodeClass::constructor(new File, context, engine);
>
> bon j'ai regardé du coté de QFile
> ce genre de cas n'arrive pas car open() ne peut pas etre appeler en 
> passant un nom de fichier en paramètre
>
> maintenant si on se concentre sur Actionaz, nous parlons de l'objet File
> je ne veux pas refaire toute l'API de cet objet et trop modifier 
> Actionaz pour un gain trop peui perceptible
>
> donc deux choix:
> 1- je reviens en arrière et je reviens à la version simple de File
> un constructeur File(),
> une méthode exists(),
> et une méthode static exists(ficname)
>
> 2- je modifie open pour que open soit appelée simplement par open()
> pareil pour write()
> pour éviter les trucs que tu redoutes : 
> File("fichier").open("autreFichier").write("test")
> et se rapprocher de l'API QFile
>
> disons que pour aujourd'hui, je voterai pour la solution 1
> (coté perso, j'ai des réponses à toutes mes questions et j'ai vu 
> suffisamment de chose, donc aucun problème pour défaire ce que j'ai fait)
> mais la décision te revient ...
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/Jmgr/actionaz/pull/13/files#r3039261>.
>
Je suis bien d'accord, la solution 1 me paraît la meilleure pour l'instant.