diff --git a/packages/facade/src/apis/sheet/__tests__/f-range.spec.ts b/packages/facade/src/apis/sheet/__tests__/f-range.spec.ts
index 3fb2b299ca..010895d6a7 100644
--- a/packages/facade/src/apis/sheet/__tests__/f-range.spec.ts
+++ b/packages/facade/src/apis/sheet/__tests__/f-range.spec.ts
@@ -174,7 +174,7 @@ describe('Test FRange', () => {
     });
 
     it('Range getCellData', () => {
-        const activeSheet = univerAPI.getActiveWorkbook()?.getActiveSheet();
+        const activeSheet = univerAPI.getActiveWorkbook()!.getActiveSheet();
         activeSheet?.getRange(0, 0)?.setValue(1);
         const range = activeSheet?.getRange(0, 0);
         expect(range?.getCellData()?.v).toBe(1);
diff --git a/packages/facade/src/apis/sheet/f-range.ts b/packages/facade/src/apis/sheet/f-range.ts
index b03169ee86..9f3ec933c6 100644
--- a/packages/facade/src/apis/sheet/f-range.ts
+++ b/packages/facade/src/apis/sheet/f-range.ts
@@ -412,21 +412,11 @@ export class FRange {
      * @param color The font color in CSS notation (such as '#ffffff' or 'white'); a null value resets the color.
      */
     setFontColor(color: string | null): this {
-        let style: IStyleTypeValue<IColorStyle | null>;
-
-        if (color === null) {
-            style = {
-                type: 'cl',
-                value: null,
-            };
-        } else {
-            style = {
-                type: 'cl',
-                value: {
-                    rgb: color,
-                },
-            };
-        }
+        const value: IColorStyle | null = color === null ? null : { rgb: color };
+        const style: IStyleTypeValue<IColorStyle | null> = {
+            type: 'cl',
+            value,
+        };
 
         const setStyleParams: ISetStyleCommandParams<IColorStyle | null> = {
             unitId: this._workbook.getUnitId(),