added lexer

[mmiyahar1205]

lexer(ファイルの情報をdataに保存する)を作った

• memory leakとかチェックできてないので要チェックかも

[kurrrru]

• 依存関係からして、mlx_init関係を呼び出すのは、read_fileが完了した後の方がいいと思う
• open, malloc, mlx_initとか、失敗したらエラーメッセージ出して終了する系の関数をwrapしたxopen, xmalloc, xmlx_initを用意してるから、使っていいよ。というかむしろ、mlx_initとかの前だったら、exitしたらリソースを解放してくれるから、積極的にx...を使いたいかも
  • mlx_initとかは、ちゃんと解放してくれるかは分からないから、明示的に解放した方が良さそう
• readfile内のwhileループ終了時はnext_lineはNULLのはずだから、freeいらなそう
• handle_elements内の、msg_exit("Invalid or error processing element in file\n");の\nは不要かも
• handle_elements内、statusという変数なくてもいいかも
• init_data内で、dataはmallocしなくてもいいかも（すでにスタック領域にメモリ確保された状態だから）
• process_elementの実装見た感じ、ft_strcmpを今からでも作った方が分かりやすいような気がする
• read_fileやset_...の各関数も、返す値はマクロを使った方が分かりやすいと思う（0/1でも0/-1でもいいけどできるだけ統一した方がいい）
• elements_lenみたいな関数を作った方がよいかも
  • 今の感じだと、char *の配列の長さはいくつが正しいのか、ぱっと見で分かりづらいかも
• parse_colorは0~255の値を扱う
  • 255以下であることのチェックも必要かも？
• 数値以外の文字がないかのチェックはした方がいいかも
• 全部は見切れてないので明日みます

[mmiyahar1205]

commit comment 1

対応完了

• 依存関係からして、mlx_init関係を呼び出すのは、read_fileが完了した後の方がいいと思う
• open, malloc, mlx_initとか、失敗したらエラーメッセージ出して終了する系の関数をwrapしたxopen, xmalloc, xmlx_initを用意してるから、使っていいよ。というかむしろ、mlx_initとかの前だったら、exitしたらリソースを解放してくれるから、積極的にx...を使いたいかも
• readfile内のwhileループ終了時はnext_lineはNULLのはずだから、freeいらなそう
  • mlx_initとかは、ちゃんと解放してくれるかは分からないから、明示的に解放した方が良さそう
• handle_elements内の、msg_exit("Invalid or error processing element in file\n");の\nは不要かも
• init_data内で、dataはmallocしなくてもいいかも（すでにスタック領域にメモリ確保された状態だから）
• handle_elements内、statusという変数なくてもいいかも
• process_elementの実装見た感じ、ft_strcmpを今からでも作った方が分かりやすいような気がする
• parse_colorは0~255の値を扱う
  • 255以下であることのチェックも必要かも？

未対応

• read_fileやset_...の各関数も、返す値はマクロを使った方が分かりやすいと思う（0/1でも0/-1でもいいけどできるだけ統一した方がいい）
• elements_lenみたいな関数を作った方がよいかも
  • 今の感じだと、char *の配列の長さはいくつが正しいのか、ぱっと見で分かりづらいかも
• 数値以外の文字がないかのチェックはした方がいいかも

[kurrrru]

• data.img.image = xmlx_new_image(data.mlx, 1000, 1000);を追加したい
• x...の関数、失敗したらexitする実装にしているから、NULLチェックしなくていいよ
• norm errorはlibft.hのプロトタイプ宣言のインデントがあっていない
• ft_lstnewのエラー処理がされていないかも
• src/wrapにxft_lstnewを作った方がいいかも
• 座標として-1が与えられることはあり得るため、-1のときエラーとするのは良くないかも
• splitした後の単語数が想定と違うなどの場合は、エラーメッセージを出してexitしてもいいかも
• 数値以外の文字があったときをエラーとするために、convertible_to_floatという作った関数を使える
• splitもxsplitあるよ

[mmiyahar1205]

対応完了

• data.img.image = xmlx_new_image(data.mlx, 1000, 1000);を追加したい
• x...の関数、失敗したらexitする実装にしているから、NULLチェックしなくていいよ
• norm errorはlibft.hのプロトタイプ宣言のインデントがあっていない
• 座標として-1が与えられることはあり得るため、-1のときエラーとするのは良くないかも
• splitもxsplitあるよ
• 数値以外の文字があったときをエラーとするために、convertible_to_floatという作った関数を使える
• splitした後の単語数が想定と違うなどの場合は、エラーメッセージを出してexitしてもいいかも
• ft_lstnewのエラー処理がされていないかも
• src/wrapにxft_lstnewを作った方がいいかも

イェーイ