From 392f1903f72938159927b49a463ddf4ba242b394 Mon Sep 17 00:00:00 2001
From: Eric Banks <ebanks@broadinstitute.org>
Date: Wed, 18 Apr 2012 12:57:37 -0400
Subject: [PATCH] Handling some of the NumberFormatExceptions seen via Tableau
 that are really user errors.

---
 .../sting/utils/codecs/refseq/RefSeqCodec.java             | 2 ++
 .../sting/utils/codecs/vcf/AbstractVCFCodec.java           | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/public/java/src/org/broadinstitute/sting/utils/codecs/refseq/RefSeqCodec.java b/public/java/src/org/broadinstitute/sting/utils/codecs/refseq/RefSeqCodec.java
index efcd3ecf0..cb392f29c 100644
--- a/public/java/src/org/broadinstitute/sting/utils/codecs/refseq/RefSeqCodec.java
+++ b/public/java/src/org/broadinstitute/sting/utils/codecs/refseq/RefSeqCodec.java
@@ -73,6 +73,8 @@ public class RefSeqCodec implements ReferenceDependentFeatureCodec<RefSeqFeature
         } catch ( UserException.MalformedGenomeLoc e ) {
             Utils.warnUser("RefSeq file is potentially incorrect, as some transcripts or exons have a negative length ("+fields[2]+")");
             return null;
+        } catch ( NumberFormatException e ) {
+            throw new UserException.MalformedFile("Could not parse location from line: " + line);
         }
     }
 
diff --git a/public/java/src/org/broadinstitute/sting/utils/codecs/vcf/AbstractVCFCodec.java b/public/java/src/org/broadinstitute/sting/utils/codecs/vcf/AbstractVCFCodec.java
index 2c4c4f607..a1127e35d 100755
--- a/public/java/src/org/broadinstitute/sting/utils/codecs/vcf/AbstractVCFCodec.java
+++ b/public/java/src/org/broadinstitute/sting/utils/codecs/vcf/AbstractVCFCodec.java
@@ -473,7 +473,12 @@ public abstract class AbstractVCFCodec implements FeatureCodec, NameAwareCodec {
     protected static Allele oneAllele(String index, List<Allele> alleles) {
         if ( index.equals(VCFConstants.EMPTY_ALLELE) )
             return Allele.NO_CALL;
-        int i = Integer.valueOf(index);
+        final int i;
+        try {
+            i = Integer.valueOf(index);
+        } catch ( NumberFormatException e ) {
+            throw new TribbleException.InternalCodecException("The following invalid GT allele index was encountered in the file: " + index);
+        }
         if ( i >= alleles.size() )
             throw new TribbleException.InternalCodecException("The allele with index " + index + " is not defined in the REF/ALT columns in the record");
         return alleles.get(i);