Use Optional Chaining (?.)

js/src/bin/arrow2csv.ts

@@ -147,7 +147,7 @@ function batchesToString(state: ToStringState, schema: Schema) {
         },
         transform(batch: RecordBatch, _enc: string, cb: (error?: Error, data?: any) => void) {
 
-            batch = !(state.schema && state.schema.length) ? batch : batch.select(...state.schema);
+            batch = !state.schema?.length ? batch : batch.select(...state.schema);
 
             if (state.closed) { return cb(undefined, null); }
 

js/src/io/node/iterable.ts

@@ -66,7 +66,7 @@ class IterableReadable<T extends Uint8Array | any> extends Readable {
             }
             if (!this.push(r.value) || size <= 0) { break; }
         }
-        if ((r && r.done || !this.readable) && (this.push(null) || true)) {
+        if ((r?.done || !this.readable) && (this.push(null) || true)) {
             it.return && it.return();
         }
         return !this.readable;
@@ -105,7 +105,7 @@ class AsyncIterableReadable<T extends Uint8Array | any> extends Readable {
             }
             if (!this.push(r.value) || size <= 0) { break; }
         }
-        if ((r && r.done || !this.readable) && (this.push(null) || true)) {
+        if ((r?.done || !this.readable) && (this.push(null) || true)) {
             it.return && it.return();
         }
         return !this.readable;

js/src/table.ts

@@ -185,7 +185,7 @@ export class Table<T extends { [key: string]: DataType } = any>
 
         const chunks = selectArgs<RecordBatch<T>>(RecordBatch, args);
 
-        if (!schema && !(schema = chunks[0] && chunks[0].schema)) {
+        if (!schema && !(schema = chunks[0]?.schema)) {
             throw new TypeError('Table must be initialized with a Schema or at least one RecordBatch');
         }
 

js/src/util/args.ts

@@ -186,7 +186,7 @@ function _selectFieldArgs<T extends { [key: string]: DataType }>(vals: any[], re
             ({ [idx]: field = idx } = keys);
             if (val instanceof DataType && (values[++valueIndex] = val)) {
                 fields[++fieldIndex] = Field.new(field, val as DataType, true) as Field<T[keyof T]>;
-            } else if (val && val.type && (values[++valueIndex] = val)) {
+            } else if (val?.type && (values[++valueIndex] = val)) {
                 val instanceof Data && (values[valueIndex] = val = Vector.new(val) as Vector);
                 fields[++fieldIndex] = Field.new(field, val.type, true) as Field<T[keyof T]>;
             }

js/src/util/buffer.ts

@@ -177,7 +177,7 @@ export async function* toArrayBufferViewAsyncIterator<T extends TypedArray>(Arra
         yield* pump((function*(it: Iterator<any>) {
             let r: IteratorResult<any> = <any> null;
             do {
-                r = it.next(yield r && r.value);
+                r = it.next(yield r?.value);
             } while (!r.done);
         })(source[Symbol.iterator]()));
     };

js/test/Arrow.ts

@@ -29,14 +29,14 @@ Object.defineProperty(Object, Symbol.hasInstance, {
     writable: true,
     configurable: true,
     value(inst: any) {
-        return inst && inst.constructor && inst.constructor.name === 'Object';
+        return inst?.constructor && inst.constructor.name === 'Object';
     }
 });
 Object.defineProperty(ArrayBuffer, Symbol.hasInstance, {
     writable: true,
     configurable: true,
     value(inst: any) {
-        return inst && inst.constructor && inst.constructor.name === 'ArrayBuffer';
+        return inst?.constructor && inst.constructor.name === 'ArrayBuffer';
     }
 });
 

js/test/unit/builders/utils.ts

@@ -193,7 +193,7 @@ export function validateVector<T extends DataType>(vals: (T['TValue'] | null)[],
         // debugger;
         // vec.get(i);
         throw new Error([
-            `${(vec as any).VectorName}[${i}]: ${e && e.stack || e}`,
+            `${(vec as any).VectorName}[${i}]: ${e?.stack || e}`,
             `nulls: [${nullVals.join(', ')}]`,
             `values: [${vals.join(', ')}]`,
         ].join('\n'));