レビューに基づいてコードを修正

step1は不要な初期化やnullチェックを削除
step3は初期化を適切に行うように修正

Author: colorbox

383/step1.cpp

@@ -8,19 +8,19 @@ magazineにある文字を高速にチェックできないといけないので
 magazineに存在しない文字や、消費し尽くした文字を指定されたらransomは成立しなくなるのでその時点でfalseを返せる。
 falseを返さずに最後までチェックし終えたら
 */
+/*
+mapで`[]`を使用することで不要な初期化処理やnullチェックを削除
+*/
 class Solution {
 public:
     bool canConstruct(string ransomNote, string magazine) {
         map<char, int> character_count;
         for (char c: magazine) {
-            if (!character_count[c]) {
-                character_count[c] = 0;
-            }
             character_count[c]++;
         }
 
         for (char c: ransomNote) {
-            if (!character_count[c] || character_count[c] <= 0) {
+            if (character_count[c] <= 0) {
                 return false;
             }
             character_count[c]--;

383/step3.cpp

@@ -1,10 +1,16 @@
 /*
 97という定数が気持ち悪かったので、'a'に変更、それ以外はstep2と変わらず。
 */
+/*
+初期化をきちんとするように修正
+*/
 class Solution {
 public:
     bool canConstruct(string ransomNote, string magazine) {
         int char_count[26];
+        for(int i = 0; i < 26; i++) {
+          char_count = 0;
+        }
 
         for (char c: magazine) {
             char_count[c - 'a']++;