fix: Attempt get color from UnityObject

Author: jcs090218

Assets/JCSUnity/Scripts/JCS_UnityObject.cs

@@ -64,6 +64,7 @@ public class JCS_UnityObject : MonoBehaviour
         public void SetObjectType(JCS_UnityObjectType ob)
         {
             mObjectType = ob;
+
             UpdateUnityData();
         }
         public JCS_UnityObjectType GetObjectType() { return mObjectType; }
@@ -257,6 +258,7 @@ public Transform localTransform
                 }
 
                 Debug.LogError("Return default local position (This should not happens)");
+                
                 return transform;
             }
         }
@@ -335,6 +337,7 @@ public Vector3 position
                 }
 
                 Debug.LogError("Return default local position (This should not happens)");
+               
                 return transform.position;
             }
 
@@ -384,7 +387,9 @@ public Vector3 localPosition
                         return mTextMesh.transform.localPosition;
 #endif
                 }
+                
                 Debug.LogError("Return default local position (This should not happens)");
+                
                 return transform.localPosition;
             }
 
@@ -436,6 +441,7 @@ public Vector3 eulerAngles
                 }
 
                 Debug.LogError("Return default local rotation (This should not happens)");
+             
                 return transform.eulerAngles;
             }
 
@@ -487,6 +493,7 @@ public Vector3 localEulerAngles
                 }
 
                 Debug.LogError("Return default local rotation (This should not happens)");
+
                 return transform.localEulerAngles;
             }
 
@@ -536,7 +543,9 @@ public Vector3 localScale
                         return mTextMesh.transform.localScale;
 #endif
                 }
+
                 Debug.LogError("Return default local scale (This should not happens)");
+
                 return transform.localScale;
             }
 
@@ -560,6 +569,7 @@ public Vector3 localScale
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default local scale (This should not happens)");
             }
         }
@@ -586,7 +596,9 @@ public bool localEnabled
                         return mTextMesh.enabled;
 #endif
                 }
+
                 Debug.LogError("Return default visible (This should not happens)");
+
                 return false;
             }
 
@@ -625,14 +637,11 @@ private Color GetColor()
                 return mRenderer.material.GetColor(prop);
             }
 
-            try
-            {
-                return mRenderer.material.color;
-            }
-            catch
-            {
-                // ignore
-            }
+            Material mat = mRenderer.material;
+
+            // The `.color` and `GetColor("_Color")` is equivalent.
+            if (mat && mat.HasProperty("_Color"))
+                return mat.color;
 
             return Color.white;
         }
@@ -647,14 +656,11 @@ private void SetColor(Color val)
                 mRenderer.material.SetColor(prop, val);
             }
 
-            try
-            {
-                mRenderer.material.color = val;
-            }
-            catch
-            {
-                // ignore
-            }
+            Material mat = mRenderer.material;
+
+            // The `.color` and `GetColor("_Color")` is equivalent.
+            if (mat && mat.HasProperty("_Color"))
+                mat.color = val;
         }
 
         /// <summary>
@@ -681,6 +687,7 @@ public Color localColor
                 }
 
                 Debug.LogError("Return default Local Red (This should not happens)");
+
                 return new Color(255, 128, 64, 32);
             }
 
@@ -707,6 +714,7 @@ public Color localColor
 #endif
 
                 }
+
                 Debug.LogError("Set default Local Red (This should not happens)");
             }
         }
@@ -733,7 +741,9 @@ public float localAlpha
                         return mTextMesh.color.a;
 #endif
                 }
+
                 Debug.LogError("Return default Local Alpha (This should not happens)");
+
                 return 0;
             }
 
@@ -781,6 +791,7 @@ public float localAlpha
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default Local Alpha (This should not happens)");
             }
         }
@@ -807,7 +818,9 @@ public float localRed
                         return mTextMesh.color.r;
 #endif
                 }
+                
                 Debug.LogError("Return default Local Red (This should not happens)");
+
                 return 0;
             }
 
@@ -855,6 +868,7 @@ public float localRed
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default Local Red (This should not happens)");
             }
         }
@@ -881,7 +895,9 @@ public float localGreen
                         return mTextMesh.color.g;
 #endif
                 }
+
                 Debug.LogError("Return default Local Green (This should not happens)");
+
                 return 0;
             }
 
@@ -929,6 +945,7 @@ public float localGreen
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default Local Blue (This should not happens)");
             }
         }
@@ -955,7 +972,9 @@ public float localBlue
                         return mTextMesh.color.b;
 #endif
                 }
+
                 Debug.LogError("Return default Local Blue (This should not happens)");
+
                 return 0;
             }
 
@@ -1003,6 +1022,7 @@ public float localBlue
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default Local Blue (This should not happens)");
             }
         }
@@ -1029,7 +1049,9 @@ public Texture localMainTexture
                         return mTextMesh.material.mainTexture;
 #endif
                 }
+
                 Debug.LogError("Return default Local Blue (This should not happens)");
+
                 return null;
             }
 
@@ -1055,6 +1077,7 @@ public Texture localMainTexture
                         return;
 #endif
                 }
+
                 Debug.LogError("Set default Local Blue (This should not happens)");
             }
         }
@@ -1073,7 +1096,9 @@ public Sprite localSprite
                     case JCS_UnityObjectType.UI:
                         return mImage.sprite;
                 }
+
                 Debug.LogError("Failed to get sprite composite cuz current unity object setting does not have it.");
+
                 return null;
             }
 
@@ -1088,6 +1113,7 @@ public Sprite localSprite
                         mImage.sprite = value;
                         return;
                 }
+
                 Debug.LogError("Failed to set the sprite cuz the current unity object setting does not have sprite coposite.");
             }
         }
@@ -1111,7 +1137,9 @@ public bool localIsVisible
                     case JCS_UnityObjectType.SPRITE:
                         return mSpriteRenderer.isVisible;
                 }
+
                 Debug.LogError("Return default Local isVisible (This should not happens)");
+
                 return false;
             }
 
@@ -1138,7 +1166,9 @@ public bool localFlipX
 #endif
                         return JCS_Mathf.IsPositive(mRectTransform.localScale.x);
                 }
+
                 Debug.LogError("Return default Local FlipX (This should not happens)");
+
                 return false;
             }
 
@@ -1180,6 +1210,7 @@ public bool localFlipX
                         }
                         return;
                 }
+
                 Debug.LogError("Set default Local FlipX (This should not happens)");
             }
         }
@@ -1205,7 +1236,9 @@ public bool localFlipY
 #endif
                         return JCS_Mathf.IsPositive(mRectTransform.localScale.y);
                 }
+
                 Debug.LogError("Return default Local FlipY (This should not happens)");
+
                 return false;
             }
 
@@ -1247,6 +1280,7 @@ public bool localFlipY
                         }
                         return;
                 }
+
                 Debug.LogError("Set default Local FlipY (This should not happens)");
             }
         }