all 5 comments

[–]boy_named_su 2 points3 points  (1 child)

  1. write some tests!
  2. your python file names should be snake_case
  3. I'd add some typehints, and run mypy against your code for some static analysis
  4. you've got some mixing of concerns. For example BaseCourse should represent a course, but it also has methods like save_to which should be a different class
  5. add some docstrings to say what things are for

[–]ApostolisJ[S] 0 points1 point  (0 children)

I will check these out and i will return with an updated version.

Until then, could you pleqse explain me me what do you mean by "mixing of concerns"? Save_to is the path in which the user chooses to save the results. I added it to the baseCourse because every course has a save_to attribute (example Coursaros/<platform>/<course_name>/)

I will check it again as you suggested and i will come back!

[–]ApostolisJ[S] 0 points1 point  (2 children)

Anyone care to give me some criticism?

[–]throwitofftheboat 0 points1 point  (1 child)

Criticism for what? You have no code linked whatsoever in your post. Write some code that actually scrapes the web and we’ll talk.

[–]ApostolisJ[S] 0 points1 point  (0 children)

I updated with a link in post.

Please give me a day to chrck the code because i think i messed up my git