Throw error instead of console.assert

Author: Alexis Girault

examples/Dicom/src/index.js

@@ -12,7 +12,7 @@ const outputFileInformation = curry(async function outputFileInformation (output
   const files = event.target.files || dataTransfer.files
 
   // Parse DICOM metadata
-  const patientDict = await parseDicomFiles(files)
+  const patientDict = await parseDicomFiles(files, true)
 
   // Select DICOM serie
   outputTextArea.textContent = "Please select serie..."

examples/Dicom/src/parseDicomFiles.js

@@ -33,30 +33,26 @@ class DICOMEntity {
     const name = this.constructor.name
     const tags = this.constructor.tags
     const primaryTag = this.constructor.primaryTag
-    console.assert(
-      tags.includes(primaryTag),
-      `The primary tag of the ${name} class ("${primaryTag}") is not included in its list of tags ([${tags}]).`
-      )
+    if (!tags.includes(primaryTag)) {
+      throw Error(`The primary tag of the ${name} class ("${primaryTag}") is not included in its list of tags ([${tags}]).`)
+    }
     tags.forEach((tag) => {
-      console.assert(
-        tag in DICOM_DICTIONARY,
-        `The tag "${tag}" associated with the ${name} class is not defined in DICOM_DICTIONARY.`
-        )
+      if (!(tag in DICOM_DICTIONARY)) {
+        throw Error(`The tag "${tag}" associated with the ${name} class is not defined in DICOM_DICTIONARY.`)
+      }
     })
   }
 
   extractTags(dicomMetaData) {
+    const name = this.constructor.name
     const tags = this.constructor.tags
     const primaryTag = this.constructor.primaryTag
     tags.forEach((tag) => {
       const value = dicomMetaData.string(DICOM_DICTIONARY[tag])
       if (this[tag] === undefined) {
         this[tag] = value
-      } else if (value !== undefined) {
-        console.assert(
-          this[tag] === value,
-          `Inconsistent value for ${tag} property of ${this[primaryTag]}`
-          )
+      } else if (value !== undefined && this[tag] !== value) {
+        throw new Error(`Inconsistent value for the "${tag}" property of ${name} "${this[primaryTag]}": received "${this[tag]}" but already had "${value}".`)
       }
     })
   }
@@ -160,7 +156,7 @@ class DICOMSerie extends DICOMEntity {
   }
 }
 
-const parseDicomFiles = async (fileList) => {
+const parseDicomFiles = async (fileList, skipOnFailure = false) => {
   var patientDict = {}
 
   const parseFile = async (file) => {
@@ -186,7 +182,17 @@ const parseDicomFiles = async (fileList) => {
   const parseFiles = [...fileList].map(parseFile)
   const logName = `Parsed ${fileList.length} DICOM files in`
   console.time(logName)
-  await Promise.all(parseFiles)
+  if (skipOnFailure) {
+    parseFiles.forEach(async (promise) => {
+      try {
+        await promise
+      } catch (error) {
+        console.error(error)
+      }
+    })
+  } else {
+    await Promise.all(parseFiles)
+  }
   console.timeEnd(logName)
   return patientDict
 }