Save GetPixels to String

Scripting
1

Hi there

I am trying to convert the info I get from the GetPixels array into a more simple array - but my code is not working :frowning: - see below.

MapArrayData variable is not changing

What have a done wrong? No error kicks back. Please help?

thanks

colArray = texture.GetPixels(0);

for (x=0; x<colArray.length; x++) {
if(colArr[×].ToString==“RGBA(1.000,1.000,1.000,1.000)”) {MapArrayData = MapArrayData + “white ,”;}
if(colArr[×].ToString==“RGBA(0.000,0.000,0.000,1.000)”) {MapArrayData = MapArrayData + “black ,”;}
}

2

There are several issues with that:

  1. Your variable is “colArray”, but in the loop you’re referring to “colArr”, which either doesn’t exist or is a different variable.

  2. Functions are always called using (), even if there are no parameters. So you need ToString(), not ToString.

  3. However, using ToString() to compare color values is a really bad idea. One, it’s a lot slower than just comparing color values directly. Two, the formatting is not guaranteed to stay the same, so while it might work in one version of Unity, it might stop working in another version. So ditch the ToString() stuff and just compare colors, which will always work for sure and is way faster.

  4. A minor thing, but MapArrayData should be mapArrayData, to be consistent with Unity naming conventions (makes code easier to follow when variables and functions are easily distinguishable). Also you can use mapArrayData += “white”; for shorter code.

–Eric