動画ファイルをフレームごとの画像として保存したい

前提・実現したいこと

プログラミング大初心者です。
PythonでOPEN CVを使って動画ファイルからフレームを切り出して保存するプログラムをインターネット上にあるものを参考に実行しています。
その際に以下のエラーが発生しました。
また、このプログラムの意味がほとんどわかっていないので解説いただけると幸いです。

発生している問題・エラーメッセージ

 File "<ipython-input-16-7e09426af7b6>", line 7
    def save_all_frames(video_path, der_path, new, ext='jpg')
                                                             ^
SyntaxError: invalid syntax

該当のソースコード

import cv2
import os

video_path = 'C0597.mp4'
dir_path = 'framecap'

def save_all_frames(video_path, der_path, basename, ext='jpg')
  cap = cv2.VideoCapture(video_path)

    if not cap.isOpened():
        return

    os.makedirs(dir_path, exist_ok=True)
    base_path = os.path.join(dir_path, basename)

    digit = len(str(int(cap.get(cv2.CAP_PROP_FRAME_COUNT))))

    n = 0

    while True:
        ret, frame = cap.read()
        if ret:
            cv2.imwrite('{}_{}.{}'.format(base_path, str(n).zfill(digit), ext), frame)
            n += 1
        else:
            return

save_all_frames('data/temp/sample_video.mp4', 'data/temp/result', 'sample_video_img')

save_all_frames('data/temp/sample_video.mp4', 'data/temp/result_png', 'sample_video_img', 'png')

試したこと

補足情報（FW/ツールのバージョンなど）

行動規範の内容に同意します

回答3件

ベストアンサー

動画のファイル名が合ってて、コードを実行した時のカレントディレクトリに動画ファイルがあるなら、これで動くと思います
ファイル名や置き場所が違うなら、そこは自分で直してください

修正前のを「#」付けて残してあるので、それと見比べてください

python
1import cv2
2import os
3
4video_path = 'C0597.mp4'
5dir_path = 'framecap'
6
7#def save_all_frames(video_path, der_path, basename, ext='jpg')
8def save_all_frames(video_path, dir_path, basename, ext='jpg'):
9  #cap = cv2.VideoCapture(video_path)
10    cap = cv2.VideoCapture(video_path)
11
12    if not cap.isOpened():
13        return
14
15    os.makedirs(dir_path, exist_ok=True)
16    base_path = os.path.join(dir_path, basename)
17
18    digit = len(str(int(cap.get(cv2.CAP_PROP_FRAME_COUNT))))
19
20    n = 0
21
22    while True:
23        ret, frame = cap.read()
24        if ret:
25            cv2.imwrite('{}_{}.{}'.format(base_path, str(n).zfill(digit), ext), frame)
26            n += 1
27        else:
28            return
29
30#save_all_frames('data/temp/sample_video.mp4', 'data/temp/result', 'sample_video_img')
31
32#save_all_frames('data/temp/sample_video.mp4', 'data/temp/result_png', 'sample_video_img', 'png')
33save_all_frames(video_path, dir_path, 'sample_video_img', 'png')

投稿2021/06/22 10:55

jbpb0

総合スコア7653

shktech

2021/06/23 05:41

ご回答ありがとうございました。この通りにやると実行することができました。本当に助かりました。 #save_all_frames('data/temp/sample_video.mp4', 'data/temp/result', 'sample_video_img') この部分が必要なくて、video_path,dir_pathをちゃんと指定してあげることで解決できたという認識で間違いないのでしょうか？

jbpb0

2021/06/23 06:05 編集

違います他にもあります・def save_all_frames(... の最後の「:」が抜けてる (ppaulさんの回答と同じ) ・def save_all_frames(... の引数名の間違い：der_path → dir_path ・cap = cv2.VideoCapture(... のインデントの間違い：次の if not... に合わせる・一つ目の save_all_frames('data/... が要らない (質問者さんが上のコメントに書いてること) の四つが間違い「#」付いてるのと、よーーーく見比べてください

shktech

2021/06/24 03:28

1.2.4番目の間違いは理解しました。 3番目の間違いについては、インデントが違うことで違うブロックだと認識してしまうからそろえる必要があるということでしょうか？

jbpb0

2021/06/24 05:55

> インデントが違うことで違うブロックだと認識してしまうからそろえる必要がある cap = cv2.VideoCapture(... のインデントを元に戻して実行したら分かりますけど、エラーになります

shktech

2021/06/24 09:31

インデントを０にして実行してみると確かにエラーが出ました。本当に、本当に助かりました。ありがとうございました。

行動規範の内容に同意します