array, list or even other solution

Report · Jun 04, 2011

i find this really complicated. i got a flashpage that uploads images. In fact for each image selected, it uploads 3 different formats (thumb_, midle_, max_)

So I upload image1.jpg and I get thumb_image1.jpg, middle_image1jpg, and max_image1.jpg

I can upload unto 9 images at at time, they will all have these three formats.

After uploading data get posted to the last file where I need to rename files and write them to db

The data that get's posted has the format:

MPuploadFileName_0 = sea.jpg

MPuploadFileName_1 = sea1.jpg

MPuploadFileName_3 = sea2.jpg

etc.

Dont forget, each image has 3 formats. In fact the MPuploadFileName_0 is hte 'main' filename and the others are details of it.

Each file upload get's an ID from the DB (don't ask me why, I didn't invent the software just have to work with it:-)) So I get latest ID, add 1 and write for exemple 104256 to db

now system supposes there is a 104256_max.jp, 104256_middle.jpg, 104256_thumb.jpg

This was for the 'main image'

This works, that is already that

Now the more difficult, for the 'detail images' the system supposes they are called 104256_detail2_max.jpg, 104256_detail2_middle.jpg, 104256_detail2_max.jpg, 104256_detail3_max.jpg, 104256_detail3_middle.jpg, 104256_detail3_max.jpg, etc

The number of uploaded files is in a variable named "filecount"

So I have to turn MPuploadFileName_1 = sea1.jpg that has been uploaded as thumb_sea1.jpg, middle_sea1jpg, and max_sea1.jpg into

104256_detail2_max.jpg, 104256_detail2_middle.jpg, 104256_detail2_max.jpg

This is what I tried:

ERROR;

Complex object types cannot be converted to simple values.

58 :           <cfset ThisFileNametoChange = "thumb_"& ThisCurrentFileName />

I hope i explained it well, any help would be greatly appreciated.

Report · Jun 05, 2011

<cfset ThisCurrentFileName = ["MultiPowUploadFileName_" & i] />Complex object types cannot be converted to simple values.
58 :           <cfset ThisFileNametoChange = "thumb_"& ThisCurrentFileName />

This error means that somewhere you're trying to use an array, object or struct as a number or string. It cannot be "ThisFileNameToChange" as that's the varible you're setting. It cannot be "thumb_" as that's a string literal, so it must be that the variable "ThisCurrentFileName" is not what you think it is.

Dump it out to the screen, see what it is. I'd guess you've set it to be the entire form scope rather than one variable, or something like that.

Report · Jun 05, 2011

<cfset ThisCurrentFileName = ["MultiPowUploadFileName_" & i] />

That's the culprit. ColdFusion will read square brackets as an object. You could do this of course:

The same applies to all the other square brackets. Hence, <cfset ThisNewFileName = request.currentimgid & "_detail" & teller & "_thumb.jpg"> in place of <cfset ThisNewFileName = [request.currentimgid & "_detail" & teller & "_thumb.jpg"]>, and so on.

Report · Jun 05, 2011

hi thanks for you answer. However though if I use

<cfset ThisCurrentFileName = "MultiPowUploadFileName_" & i /> the dump says ThisCurrentFileName = MultiPowUploadFileName_1

BUt MultiPowUploadFileName_1is a variable and has a value that I need to get

If I use <cfset ThisCurrentFileName = MultiPowUploadFileName_ & i /> it says MultiPowUploadFileName_ is undefined

which is true, how do I find a workaround? I feel I am so close but cannot seem to find the solution.

Bianca

Report · Jun 05, 2011

Hi again:-)

I used what you said in a former post about arrays:

So for each "ThisCurrentFileName" there are three versions: _thumb, _middle and _max and I think that is where it is going wrong but I don't know how to do this. Should I use an array after all? Can you help me find the right way?

Bianca

Report · Jun 05, 2011

bianca_homedev wrote:
Hi again:-)
I used what you said in a former post about arrays:
<cfloop from="1" to="#filesCount#" index="i">

<cfset ThisCurrentFileName = ["MultiPowUploadFileName_" & i] />
So for each "ThisCurrentFileName" there are three versions: _thumb, _middle and _max and I think that is where it is going wrong but I don't know how to do this. Should I use an array after all? Can you help me find the right way?
Bianca

You could do like this:

or perhaps loop through the list of file types, like this

Report · Jun 05, 2011

If you set

and do a cfdump then it says

ThisCurrentFileName= MultiPowUploadFileName_1

Then if I try to determine what filename should be renamed

cfdump says

ThisFileNametoChange = thumb_MultiPowUploadFileName_1

Of course this is not what I am looking for because in fact MultiPowUploadFileName_1= desert.jpg and MultiPowUploadFileName_2 = pinguin.jpg etc.

Please, please help me out

Bianca

Report · Jun 05, 2011

It looks like you want to be using bracket notation here:

form[MultiPowUploadFileName_1]

(I'm presuming the keys you're after are in the form scope here).

http://help.adobe.com/en_US/ColdFusion/9.0/Developing/WSc3ff6d0ea77859461172e0811cbec22c24-7fb2.html

--

Adam

Report · Jun 05, 2011

I still don't understand.

If I do

<cfset ThisCurrentFileName = ["MultiPowUploadFileName_" & i] />
<cfdump var="#ThisCurrentFileName#">
Result is

array
1	MultiPowUploadFileName_1

I have MultiPowUploadFileName from 1 to max 9 But MultiPowUploadFileName_1 has a value, a filename, how do I get that one?

If i set <cfset FiletoRename = #MultiPowUploadFileName_1# > I get an desert.jpg, so far so good but then

I get an error Complex object types cannot be converted to simple values.And I understand why but I don't know what is the right way to do this.

Report · Jun 05, 2011

The square brackets have different meanings depending on how you use them.

form["MultiPowUploadFileName_" & i]

Means "return me th value of the submitted field called MultiPowUploadFileName_1"

["MultiPowUploadFileName_" & i]

Means "create an array, containing the string "MultiPowUploadFileName_1". This is what you've accidentally done, hence your error.

You just need to prepend "form" to the start so you're referencing the submitted variable not an implicit array.

Report · Jun 05, 2011

Unluckily there is no form scope, the variables are posted from a flash

application and have no scope, they are just called multipowupload_0,

multipowupload_1 and each variable stands for a filename that has been

uploaded.

Thank you for thinking with me.

Bianca

Report · Jun 05, 2011

All variables in CF have a scope, if as you say they've been "posted" (in the http sense of the word) then yes, they're in the FORM scope.

Report · Jun 05, 2011

Every variable is in a scope. And if a variable comes from an external source, it will be in either the form scope or the URL scope (depending on whether it came to CF via a POST or a GET request). Or, I guess, the cookie scope.

Do a dump of the FORM and URL scopes. I bet Owain's right arm you'll see your variables in one or other of them.

--

Adam

Report · Jun 05, 2011

Not cool Adam, that's my favourite arm.

Report · Jun 05, 2011

All right, all right.

Owain's left arm.

(I dunno where your sense of community is, mate)

--

Adam

Report · Jun 05, 2011

But that's my second favourite!

/predictable

Report · Jun 05, 2011

Bianca_homedev wrote:
I have MultiPowUploadFileName from 1 to max 9 But MultiPowUploadFileName_1 has a value, a filename, how do I get that one?
If i set <cfset FiletoRename = #MultiPowUploadFileName_1# > I get an desert.jpg, so far so good but then
<cfset RealFiletoRename = "thumb_" & #ThisCurrentFileName# >
I get an error Complex object types cannot be converted to simple values.And I understand why but I don't know what is the right way to do this.

If you're sure all 9 indices are defined, you could do the following. Here, realFiletoRename dynamically holds the current file name.

In fact, you might even decide to store each file name separately for later use. Use for example, an array, as follows:

Alternatively, you could make the logic even much simpler with cfswitch. Here the index can take any of the values 1,2,...,9.

Report · Jun 06, 2011

...OR...

Just put in the word "form" which has seemingly been forgotten? If ever I find myself reaching for the evaluate() function, that's when I know I'm doing something shoddy.

Report · Jun 06, 2011

Hi, Your answer helped me a great way around. I have one weird problem

though. I use

Filescount is the number of files uploaded. (In case there are less then 9)

Now I upload 4 files, and the code only seems to work for file no 3, it

skips the rest. Do you have any ideas?

Bianca

Report · Jun 06, 2011

Okay, well if you're uploading four files, then doing effectively this:

</cfloop>

And only three are getting processed, then the only explanation is that the form.Filescount variable is wrong somehow. Chuck in a bit of debugging - dump the form scope at the top of the page - see what you have. Is the Filescount variable correct?

Then in your loop - put something out to the screen to show you what it's processing, see where the issue is. Should be a fairly simply one to sort if you look at your code logically.

Report · Jun 06, 2011

@Bianca

You could of course use <cfloop from="1" to="#filescount#" index="idx">, as Owain has pointed out. If you still can't get it to work, then it might help to show us just the piece of code that needs fixing.

Report · Jun 06, 2011

Hi,

Here's the code I use. Weird enough it only treats the last image that gets

uploaded and not the other ones. I checked the filescount variable and that

one works fine.

</cfloop

Report · Jun 06, 2011

Your code hasn't appeared, but I suspect you're using one variable to store the dynamic filenames, like this:

</cfloop>

This code overwrites the value something_1 with something_2, and then overwrites the value something_2 with something_3. The final value of filename is therefore something_3.

If that is indeed what's happening in your case, then you should use an array or a struct. For example

</cfloop>

Report · Jun 06, 2011

Oooh, I though it was executing the code for each step, I am indeed using

I was thinking that like this it would rename file 1 to 3 one by one, am I

wrong?

Bianca

Report · Jun 06, 2011

Bianca_homedev wrote:
I was thinking that like this it would rename file 1 to 3 one by one, am I
wrong?

It would. But then, after each such renaming, you would have to do any business that depends on the index within the loop, before the loop closes.

</cfloop>

As I have said, store the values in an array or struct if you wish to use them after the loop.

array, list or even other solution

Complex object types cannot be converted to simple values.

1 Correct answer