[W2] Coping with Issues and Concerns (LAB 1)

LAB 1

Lab 1 was about reviewing classmate's amazing work and giving each others advice. Thanks to Calvin Ho, I had made some changes to my code.

Issues that came up in my review

#1 Using list comprehension

Calvin suggested me to use list comprehension, and also kindly gave me a link that had examples, which helped me understand it way clearer.



so I changed my code from:


to:



#2 Moving dependant code inside the block

Calvin also suggested moving some code blocks inside the "opening file" scope. Calvin was totally right! So I changed some indentation to my code and I think the code is more readable!

Issue that I filed


Paul's command line args implementation didn't match the requirements, so I told him it would be better to fix it to UNIX like styles. (link)




Comments

Post a Comment

Popular posts from this blog

About Me

  print("Hi! My name is Jongwon Jang")      I'm a 3rd year student studying at Seneca College located in North York, Toronto and thrilled to study open source development with wonderful developers all around the world. I'm passionate about web development, especially using libraries such as React and currently exploring new tools and techs for back-end development. Also, I am planning to learn python in depth until the course ends and hoping I'll be good at it.      I chose OSD600 over other courses, because I really wanted to know how programmers collaborate and learn how real life programming works. I've done several assignments during my studies, however, I only had this passive learning experience and wasn't actively searching and learning the things I really wanted to learn. I hope this opportunity will make me a better programmer than before.      Among countless trending repositories on Github, I picked a repo called "learn-python...
Read more

[W2] Release 0.1 - getting out of Tutorial Land

Image
This week we had to create a cli tool with our choice of language. I chose Python and started researching how to create a cli tool with it. Since I've never used Python (only watched several tutorial videos), it was frustrating at first. PyCharm couldn't recognize the import statements and configuring virtual environment for the project was totally confusing.  I started by searching the regEx which will collect all URLs starting with either "http://" or "https://". Then I put those links in a List and removed duplicates by using a Dictionary (since Dictionary cannot have duplicate keys, it removed the duplicates. I could have done with Set as well!) Then I made requests to the links and collected status_codes for the request. If the status code was 200, it would print the message "GOOD" with a color green. If the status code was 400, 404, or couldn't get the response within 1.5s, it would print the message "BAD" with the color red. Ot...
Read more

[W4] Lab 3

Image
For Lab 3, I made some changes for my CLI tool, detectURL. 1. I made my program to exit with the right exit code (0, 1, or 3) 2 I made my program to support 3 flags, --all, --bad, and --good. I first created  Issue #5  and Issue #6  as the Lab's requirement. Then I created a new branches from master by using the command: git checkout -b issue-5 I made the changes which are: b5224c  and 9869105 I merged each of the commit to the master branch and pushed the master branch to GitHub. Now it supports 3 new flags and if you check the exit code after execution, it will have different exit codes for different scenarios.
Read more