Merge pull request #5 from infinotiver/deepsource-autofix-019ece25

infinotiver · web-flow · commit fd9f9c74810a · 2024-06-29T21:19:37.000+05:30
refactor: fix out of bounds array access in for-loop
diff --git a/Tutorials/Basic/challenge.js b/Tutorials/Basic/challenge.js
@@ -18,7 +18,7 @@ function countVowelsAndConsonants(str) {
    let digit_count = 0;
    let digit_sum = 0
 
-   for(var i = 0; i <= str.length; i++) {
+   for(var i = 0; i < str.length; i++) {
     if(vowels.includes(str[i])) {
         vowel_count++
     }

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ function countVowelsAndConsonants(str) {`
`18`	`18`	`let digit_count = 0;`
`19`	`19`	`let digit_sum = 0`
`20`	`20`
`21`		`- for(var i = 0; i <= str.length; i++) {`
	`21`	`+ for(var i = 0; i < str.length; i++) {`
`22`	`22`	`if(vowels.includes(str[i])) {`
`23`	`23`	`vowel_count++`
`24`	`24`	`}`