Word values

Question

codewarsというサイト内で問題を解いています。https://www.codewars.com/kata/598d91785d4ce3ec4f000018/train/c
文字列を与えられ、アルファベットのそれぞれの文字が値を持つとして、(a = 1, b = 2, c = 3 ....z = 26)

["abc","abc abc"] →　[6,24] 　

上の例　：　一つ目の要素"abc" →a(=1)+b(=2)+c(=3) ×1
二つ目の要素"abc abc" →　{ (1+2+3)+ (1+2+3) } ×2

このように、それぞれの文字が持つ値の和と要素の位置の掛け算を値として返す関数を作成しています。下のようにコードを作ったのですが、問題点があれば教えていただけないでしょうか？また、コードのより良い書き方や、不要な部分を教えていただければ嬉しいです。

#include <stddef.h>
#include <stdlib.h> //malloc
#include <string.h> //strlen
#include <ctype.h>  //isalpha

const int* name_value(size_t n, const char *const words[n]) {
  
    size_t i;
    const int* answer = malloc(sizeof(int) * n );
    
  
    for(i = 0; i < m; i++)
    {
      int l = strlen(words[i]);
      for(size_t j = 0; j < (size_t)l; j++)
      {
       
        if (!isalpha((int)words[i][j]))continue;  //空白だったらスキップ

        answer[i] = answer[i] + (words[i][j] - 96);  //文字を数値に変換
        
        
      }
      
      answer[i] = answer[i] * (i + 1);
      
      
    }
  
  
  return answer;
}

エラー

fixture.c:10:5: warning: implicit declaration of function 'tester' is invalid in C99 [-Wimplicit-function-declaration]
    tester(4, words, expected);
    ^
fixture.c:15:5: warning: implicit declaration of function 'tester' is invalid in C99 [-Wimplicit-function-declaration]
    tester(3, words, expected);
    ^
2 warnings generated.
solution.c:12:20: error: use of undeclared identifier 'm'
    for(i = 0; i < m; i++)
                   ^
solution.c:20:19: error: read-only variable is not assignable
        answer[i] = answer[i] + (words[i][j] - 96); 
        ~~~~~~~~~ ^
solution.c:25:17: error: read-only variable is not assignable
      answer[i] = answer[i] * (i + 1);
      ~~~~~~~~~ ^
3 errors generated.
setup.c:11:17: warning: format specifies type 'int' but the argument has type 'size_t' (aka 'unsigned long') [-Wformat]
                i, expected[i], submitted[i]);
                ^
/usr/include/criterion/internal/assert.h:529:62: note: expanded from macro 'cr_assert'
#define cr_assert(...)                  CR_EXPAND(cr_assert_(__VA_ARGS__))
                                                             ^~~~~~~~~~~
/usr/include/criterion/internal/assert.h:159:28: note: expanded from macro 'cr_assert_'
                CR_VA_TAIL(__VA_ARGS__)            \
                           ^~~~~~~~~~~
/usr/include/criterion/internal/preprocess.h:41:103: note: expanded from macro 'CR_VA_TAIL'
#define CR_VA_TAIL(...)                   CR_EXPAND(CR_VA_TAIL_HELPER(CR_VA_TAIL_SELECT(__VA_ARGS__), __VA_ARGS__))
                                                                                                      ^~~~~~~~~~~
note: (skipping 30 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
/usr/include/criterion/internal/preprocess.h:35:43: note: expanded from macro 'CR_EXPAND'
#define CR_EXPAND(x)                      x
                                          ^
/usr/include/criterion/internal/preprocess.h:35:43: note: expanded from macro 'CR_EXPAND'
#define CR_EXPAND(x)                      x
                                          ^
/usr/include/criterion/internal/preprocess.h:35:43: note: expanded from macro 'CR_EXPAND'
#define CR_EXPAND(x)                      x
                                          ^
1 warning generated.

テストコード

#include <criterion/criterion.h>
#include <stddef.h>

int* name_value(size_t n, const char *const words[n]);

Test(Example_Tests, should_pass_these_two_tests) {
  {
    const char *const words[4] = {"abc", "abc", "abc", "abc"};
    const int expected[4] = {6, 12, 18, 24};
    tester(4, words, expected);
  }
  {
    const char *const words[3] = {"codewars", "abc", "xyz"};
    const int expected[3] = {88, 12, 225};
    tester(3, words, expected);
  }
}

修正版

#include <stddef.h>
#include <stdlib.h> //malloc
#include <string.h> //strlen
#include <ctype.h>  //isalpha

const int* name_value(size_t n, const char *const words[n]) {
  
    size_t i;
    int* answer = calloc(n,sizeof(int) );
    
  
    for(i = 0; i < n; i++)
    {
      int l = strlen(words[i]);
      for(size_t j = 0; j < (size_t)l; j++)
      {
       
        if (!isalpha(words[i][j]))continue;  //空白だったらスキップ

        answer[i] +=  (words[i][j] - 'a' - 1);  //文字を数値に変換
        
        
      }
      
      answer[i] *= (i + 1);
      
      
    }
  
  
  return answer;
}

Comments

[kazuma-s]

エラーメッセージのうち、warning よりもまず error を見てください。
自分の書いたコードのエラーが表示されています。

Answer 1

const int* answer = だと answer[i] が書き込み禁止になります。
初期化すらできません。というか 0 に初期化せずに和を計算していますね。
なぜ const を付けたんですか？

for(i = 0; i < m; i++) の m はどこにも宣言されていません。
n の間違いでしょう。

isalpha((int)words[i][j]) の (int) は不要。

words[i][j] - 96 の 96 が分かりにくい。('a' - 1) のほうが良い。

answer[i] = answer[i] + は answer[i] += と書けます。
answer[i] = answer[i] * は answer[i] *= と書けます。

Comments

[Merrifield]

>>なぜ const を付けたんですか？
戻り値がconstだったので、付けたのですが必要なのでしょうか？あと、初期化は必要でしたね。

>>for(i = 0; i < m; i++) の m はどこにも宣言されていません。
n の間違いでしょう。
これはそうですね。

>>isalpha((int)words[i][j]) の (int) は不要
(int)は必要なかったのですね。

>>answer[i] = answer[i] + は answer[i] += と書けます。
answer[i] = answer[i] * は answer[i] *= と書けます。
その書き方で良いのですね。ありがとうございます。

[Merrifield]

>>words[i][j] - 96 の 96 が分かりにくい。('a' - 1) のほうが良い。
これはwords[i][j] - ('a' - 1)と書いたほうが良いということでしょうか？

[kazuma-s]

word[i][j] が 'a' のとき、式の値が 1 になることが明白ですよね。
'a' - 96 と書いたら、それが 1 になることはどうやって分かるんですか？

ところで、全体の修正の結果はどうなったんですか？

[Merrifield]

そういうものなのですね。

修正結果は、質問文に載せました。