Create instagram_pic

[Epic-R-R]

Describe your change:

• Add an algorithm?
• Fix a bug or typo in an existing algorithm?
• Documentation change?

Checklist:

• I have read CONTRIBUTING.md.
• This pull request is all my own work -- I have not plagiarized.
• I know that pull requests will not be merged if they fail the automated tests.
• This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
• All new Python files are placed inside an existing directory.
• All filenames are in all lowercase characters with no spaces or dashes.
• All functions and variable names follow Python naming conventions.
• All function parameters and return values are annotated with Python type hints.
• All functions have doctests that pass the automated testing.
• All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
• If this pull request resolves one or more open issues then the commit message contains Fixes: #{$ISSUE_NO}.

[ghost]

Pull Request Report

@Epic-R-R Hello! I'm a bot made to check all the pull request Python files. First of all, I want to say thank you for your time and interest in this project and for opening a pull request.

I have detected errors in some of the Python files submitted in this pull request. Please read through the report and make the necessary changes. You can take a look at the relevant links provided after the report.

What are node paths?

The report contain headings and a checklist where the items are paths to the class/function/parameter where the error is present. Node paths are double colon :: separated names and can be any of the following format:

• Class path: [file_name]::[class_name]
• Function path: [file_name]::[function_name]
• Function parameter path: [file_name]::[function_name]::[parameter_name]
• Method path: [file_name]::[class_name]::[function_name]
• Method parameter path: [file_name]::[class_name]::[function_name]::[parameter_name]

Following functions require tests [doctest/unittest/pytest]:

• web_programming/instagram_pic.py::DownloadSingleFile

Following function parameters require type hints:

• web_programming/instagram_pic.py::DownloadSingleFile::URL

Relevant links:

• Contributing guidelines: https://github.com/TheAlgorithms/Python/blob/master/CONTRIBUTING.md
• Project Euler solution guidelines: https://github.com/TheAlgorithms/Python/blob/master/project_euler/README.md
• Type hints: https://docs.python.org/3/library/typing.html
• doctest: https://docs.python.org/3/library/doctest.html
• unittest: https://docs.python.org/3/library/unittest.html
• pytest: https://docs.pytest.org/en/stable/

[cclauss]

Suggested change

	import requests
	from bs4 import BeautifulSoup
	import datetime
	import requests
	from bs4 import BeautifulSoup
	from datetime import datetime

Please run isort on the imports.

[cclauss]

Suggested change

	print("Downloading image...")
	print(f"Downloading image from {url} ...")

Use f-strings as discussed in CONTRIBUTING.md

[cclauss]

Suggested change

	req = requests.get(url)
	soup = BeautifulSoup(req.content, "html.parser")
	soup = BeautifulSoup(requests.get(url).content, "html.parser")

Avoid creating variables that are only used on the next line unless:

1. The lines are already close to the max_line_length of 88 chars per line -- or --
2. The variable name clarifies something that is unclear

[cclauss]

Suggested change

	metaTag = soup.find_all("meta", {"property": "og:image"}) #get all meta tags
	imgURL = metaTag[0]["content"]
	# The image URL is in the content field of the first meta tag with the property og:image
	image_url = soup.find("meta", {"property": "og:image"})["content"]

[cclauss]

Python variable names are in snake_case as discussed in CONTRIBUTING.md.

[cclauss]

Suggested change

	r = requests.get(imgURL)
	fileName = datetime.datetime.now().strftime("%Y-%m-%d_%H:%M:%S") + ".jpg"
	with open(fileName, "wb") as fp:
	fp.write(r.content)
	image_data = requests.get(imgURL).content
	file_name = f"{datetime.now():%Y-%m-%d_%H:%M:%S}.jpg"
	with open(file_name, "wb") as fp:
	fp.write(image_data)

Avoid single-letter variable names -- They make code look like it was written in the 1970's. The reader of this code does not care about the response, but they do care about the image_data so focus their attention on that.

Use f-strings which are more expressive especially with complex types like datetimes.

[cclauss]

Suggested change

	print("Done. Image saved to disk as " + fileName)
	print(f"Done. Image saved to disk as {file_name}.")

[cclauss]

Suggested change

	url = input("Enter image url: ")
	url = input("Enter image url: ").strip()

As discussed in CONTRIBUTING.md.

[erdum]

Plz add type hints in your function parameters like this, def test(a: int, b: int) -> int:
Like that you have to define the types of parameters and the type of return of the function,
And plz jus install doctest pip3 install doctest, then read this tutorial on how to use doctest in your code https://www.geeksforgeeks.org/testing-in-python-using-doctest-module/, then commit changes and hope pre-commit check will be clear thanks.

[Epic-R-R]

I corrected the code and commit it, but it still gives an error in pre-commit isort field

[erdum]

Yes i saw that your flake8 and isort is failing

[erdum]

Install isort in your machine and run isort in this file locally

[erdum]

And also install flake8 to see what styling mistake you done and if you can't corrected manually then install black and run black on this file it will correct all your styling mistake and flake8 check should be clear

[Epic-R-R]

thanks, I use isort and then All checks have passed

[cclauss]

Nice work! Thanks for doing this.

[erdum]

cclauss please also review my pull request