Cfrtjryk

Is my code correct?

import requests

import bs4

import time

import urllib.request



def input_link():



    start_url = "https://en.wikipedia.org/wiki/Special:Random"

    response = requests.get(start_url)

    html = response.text

    soup = bs4.BeautifulSoup(html,"html.parser")

    return soup



def right_link(soup):



    for element in soup.find_all("p"):

        if element.find("a"):

            article_link = element.find("a").get('href')

            link = "http://en.wikipedia.org" + article_link

            return link



def loop(link):



    for element in range(50):

        response = requests.get(link)

        html = response.text

        soup = bs4.BeautifulSoup(html,"html.parser") 

        for element in soup.find_all("p"):

            if element.find("a"):

                link = element.find("a").get('href')



    return("http://en.wikipedia.org"+link)



def continue_crawl(link):



    target_url = "https://en.wikipedia.org/wiki/Philosophy"

    if link == target_url:

        print("We've found the target article!")

    elif len(link) > 50:

        print("The search has gone on suspiciously long, aborting search!")

    elif link == link:

        print("We've arrived at an article we've already seen, aborting search!")



def main():



    for i in range(10):

        x = input_link()

        y = right_link(x)

        z = loop(y)

        a = continue_crawl(z)

        time.sleep(2) # slow down otherwise wiki server will block you



main()

This is essentially a wild guess, given that you haven't really stated your requirements - nor the definition of 'correct' - but:

It appears that you're crawling Wikipedia to play the degrees-of-separation game. The way you're doing it is essentially the worst one. You shouldn't be scraping the web page itself. You should be calling the API. Better yet - download a dump of the Wikipedia database and operate on that.

There are other minor problems: you're missing a shebang, and you aren't checking the module name before calling main.