fix(formula): index row number

dream-num · Mar 18, 2024 · 254a505 · 254a505
1 parent 02016ab
commit 254a505
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/packages/engine-formula/src/engine/utils/char-kit.ts b/packages/engine-formula/src/engine/utils/char-kit.ts
@@ -14,6 +14,15 @@
  * limitations under the License.
  */
 
+/**
+ * Korean in Excel does not count as two characters. Here we calculate all Chinese, Japanese and Korean characters as two characters.
+ *
+ * ā -> 1
+ * ー -> 2
+ *
+ * @param str
+ * @returns
+ */
 export function charLenByte(str: string) {
     let byteCount = 0;
 

diff --git a/packages/engine-formula/src/functions/lookup/index/index.ts b/packages/engine-formula/src/functions/lookup/index/index.ts
@@ -23,15 +23,15 @@ import { BaseFunction } from '../../base-function';
 
 export class Index extends BaseFunction {
     override calculate(reference: BaseValueObject, rowNum: BaseValueObject, columnNum?: BaseValueObject, areaNum?: BaseValueObject) {
-        if (reference == null || rowNum == null) {
+        if (reference == null) {
             return ErrorValueObject.create(ErrorType.NA);
         }
 
         if (reference.isError()) {
             return reference;
         }
 
-        if (rowNum.isError()) {
+        if (rowNum?.isError()) {
             return rowNum;
         }