App\Http\Controllers\WebScrapingController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getPage()	0	6	1
A	getData()	0	14	1

<?php

namespace App\Http\Controllers;

use App\Http\Requests;
use Goutte\Client;

class WebScrapingController extends Controller
{
    protected $crawler;

    const NEWS_URL = 'https://news.ycombinator.com/';

    /**
     * Initialize controller
     */
    public function __construct()
    {
        $this->client = new Client();

The property client does not exist. Did you maybe forget to declare it?

    }

    /**
     * Return all data to the Stripe API dashboard
     * @return mixed
     */
    public function getPage()
    {
        $links = $this->getData(self::NEWS_URL);

        return view('api.scraping')->withLinks($links);
    }

    /**
     * Scrape the Links
     * @param $siteToCrawl
     * @return array
     */
    public function getData($siteToCrawl)
    {
        $crawler = $this->client->request('GET', $siteToCrawl);

        $arr = $crawler->filter('.title a[href^="http"], a[href^="https"]')->each(function($element) {
            $links = [];

            array_push($links, $element->text());

            return $links;
        });

        return $arr;
    }
}