Removed try-catch and replaced by robust implementation in ArtQrCode …

…small renderer
codebude · Apr 22, 2024 · af88109 · af88109
1 parent 3a994a6
commit af88109
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/QRCoder/ASCIIQRCode.cs b/QRCoder/ASCIIQRCode.cs
@@ -99,7 +99,14 @@ public string GetGraphic(bool drawQuietZones = true, bool invert = false, string
                     try
                     {
                         var current = moduleData[col + quietZonesOffset][row + quietZonesOffset] ^ invert;
-                        var next = moduleData[col + quietZonesOffset][(row + 1) + quietZonesOffset] ^ invert;
+                        var nextRowId = row + quietZonesOffset + 1;
+
+                        // Set next to whitespace "color"
+                        var next = false ^ invert;
+                        // Fill next with value, if in data range
+                        if (nextRowId < QrCodeData.ModuleMatrix.Count)
+                            next = moduleData[col + quietZonesOffset][nextRowId] ^ invert;
+
                         if (current == WHITE && next == WHITE)
                             lineBuilder.Append(palette.WHITE_ALL);
                         else if (current == WHITE && next == BLACK)
@@ -109,14 +116,6 @@ public string GetGraphic(bool drawQuietZones = true, bool invert = false, string
                         else
                             lineBuilder.Append(palette.BLACK_ALL);
                     }
-                    catch (Exception)
-                    {
-                        if (drawQuietZones)
-                            lineBuilder.Append(palette.WHITE_BLACK);
-                        else
-                            lineBuilder.Append(palette.BLACK_ALL);
-                    }
-
                 }
                 lineBuilder.Append(endOfLine);
             }