Mój kawałek kodu.

Hej,

Tym razem troche inna zabawa, "mój kawałek kodu". Zabawa polega na wklejeniu kawałka kodu nad którym teraz pracujecie, tudzież piszecie w wolnej chwili. Można oczywiście zadawać pytania odnośnie kodu, komentarze (krótki komentarz co kod ma robi jest mile widziany). Jako że jestem inicjatorem, zaczynam:


Poniższy kod wykorzystuje tzw. class policies (Alexandrescu, Modern C++ Design, 2005) do "konfigurowania" klas. Mająć klasę bazową, poprzez szablony możemy konfigurować jej implementacje (przypomina to wzorzec strategii).

Kod:

---

#include<list>
#include<vector>

template<typename T>
class IDataHolder
{
public:
        virtual void push(const T& value) = 0;
        virtual T pop() = 0;
};

template<typename T = void>
class DataList : public IDataHolder<T>
{
        std::list<T>        m_data;
public:
        virtual void push(const T& value) { m_data.push_back(value); }
        virtual T pop() { T t = m_data.front(); return t; }
};

template<typename T = void>
class DataVector : public IDataHolder<T>
{
        std::vector<T>        m_data;
public:
        virtual void push(const T& value) { m_data.push_back(value); }
        virtual T pop() { return m_data.front(); }       
};


template<typename T, typename C = IDataHolder<T>>
class CharQueue
{
        C m_data;
public:
        void push(const T& value) { m_data.push(value); }
        T pop() { return m_data.pop(); }       
};

int main()
{
        CharQueue<int, DataList<int>> x1;
        x1.push(10);
        int x = x1.pop();

        CharQueue<float, DataVector<float>> x2;
        x2.push(10.f);
        float f = x2.pop();

    return 0;
}

---

Kto następny?:)

Kod:

---

package assignment1;

import java.awt.*;

public class Square 
{ 
  private double x,y;   
 
  public Color color;   

  private int side;

  public Square()
  {
      x = 100;
      y = 100;
      side = 5;
          color = Color.orange;
  }
 
 
  public void draw(Graphics g)
  {
      g.setColor(color);
      g.fillRect( (int)(x), (int)(y), (int)(2*side), (int)(2*side) );
  }
 
  public void setSide(int s)
  {
      side = s;
  }
 
  public int getSide()
  {
      return side;
  }
   
  public int getX()
  {
        return (int) x;
  }
   
  public int getY()
  {
        return (int) y;
  }
 
  public void setX(int xIn)
  {
      x = xIn;
  }
 
  public void setY(int setY)
  {
      y = setY;
  }
}

---

Kod:

---

package assignment1;

import java.awt.*;

public class Ship extends Square 
{ 


    public Ship()
    {
        super();
    }

  public void draw(Graphics g)
  {
      int side = super.getSide();
      int x = super.getX();
      int y = super.getY();
     
      g.setColor(color);
      g.fillRect( (int)(x), (int)(y), (int)(2*side), (int)(2*side) );
       
      g.setColor(color);
      g.fillRect( (int)(x), (int)(y - (2*side)), (int)(2*side), (int)(2*side) );
     
      g.setColor(color);
      g.fillRect( (int)(x - (2*side)), (int)(y - (2*side)), (int)(2*side), (int)(2*side) );
     
      g.setColor(color);
      g.fillRect( (int)(x + (2*side)), (int)(y - (2*side)), (int)(2*side), (int)(2*side) );
     
      g.setColor(color);
      g.fillRect( (int)(x), (int)(y - (4*side)), (int)(2*side), (int)(2*side) );
  }
   
}

---

Kod:

---

package assignment1;

import java.awt.*;

public class FleetOfShips
{
    Ship[] fleetMembers;
    int total = 5;
 
    FleetOfShips()
    {
        fleetMembers = new Ship[total];
        for (int i = 0; i < total; i = i++)
        {
            fleetMembers[i] = new Ship(); 
            fleetMembers[i].setX (i * 10);
            fleetMembers[i].setY (i * 10);
        }
    }
   
    public void draw(Graphics g)
    {
        for (int i = 0; i < total; i = i++)
        {
            fleetMembers[i].draw(g);
        }
    }
   
    public void goSouth(int howFar)
    {
        for (int i = 0; i < total; i = i + 1)
        {   
            fleetMembers[i].setY (fleetMembers[i].getY() + howFar);
        }
    } 
}

---

Szykuje sie do studiow, teraz akurat ucze sie javy. Ten "program" ma za zadanie stworzyc kilka statkow i przy kliknieciu myszka maja sie poruszac o ustalony dystans w ustalonym kierunku.

Protipy jak uproscic, ulepszyc kod etc. bardzo mile widziane :D

Kilka tipów by się znalazło, jednak powiedz mi, to jest "skończony" projekt? Czy zamierzasz dodać jakieś nowe features (inne statki, inne ruchy statkow itd.) ?

Atm to bylo tylko zadanie do szkoly, dostalismy liste rzeczy ktore nasz program musi robic/uzyc. Wczesniej nic ciekawego raczej nie robilem, tylko jakis glupi kalkulator we flashu albo hello world, czy proceduralne programy w konsoli :p

Kod:

---

private void bgThread_DoWork(object sender, DoWorkEventArgs e)
        {
            string host = this.host.Text;
            string acc = this.acc.Text;
            string pass = this.pass.Text;



            int loop = 0;
            r = new Reported();
            this.BeginInvoke(showReport, r);
            this.BeginInvoke(setStatusText, status, "Working", Color.Green);

            if (this.loop.Checked)
                setLoopValue(host, ref loop);

       
            if (command.Enabled)
                this.BeginInvoke(addTextToForm, r, "Run command " + command.Text + " for " + (loop > 1 ? "arry ip's" : "one ip"));
            else this.BeginInvoke(addTextToForm, r, "Try run shell on " + (loop > 1 ? "arry ip's" : "one ip"));
            if (paraller.Checked)
            #region paraller loop
            {
                pOptions.CancellationToken = cancelSrc.Token;
                pOptions.MaxDegreeOfParallelism = System.Environment.ProcessorCount;
                try
                {
                    for (int j = (twoLevelArry.Checked ? getStartIp(host, 0) : 0); j < (twoLevelArry.Checked ? 256 : 1); ++j)
                    {
                        Parallel.For(getStartIp(host, 1), (loop > 0 ? 256 : getStartIp(host, 1) + 1), pOptions, i =>
                        {
                           
                            pOptions.CancellationToken.ThrowIfCancellationRequested();
                            doEachLoop(nextIp(host, i), acc, pass);
                        });
                        nextIpSecondLevel(ref host, j);
                      }
                }
                catch
                {
                    this.BeginInvoke(setStatusText, status, "Canceled", Color.Red);
                    this.BeginInvoke(addTextToForm, r, "Canceled");
                    return;
                }
            }
            #endregion
            else
            #region normal loop
                for (int j = (twoLevelArry.Checked ? getStartIp(host, 0) : 0); j < (twoLevelArry.Checked ? 256 : 1); ++j)
                {
                    for (int i = 0; i <= loop; i++)
                    {
                        if (((BackgroundWorker)sender).CancellationPending)
                        {
                            e.Cancel = true;
                            this.BeginInvoke(setStatusText, status, "Canceled", Color.Red);
                            this.BeginInvoke(addTextToForm, r, "Canceled");
                            return;
                        }
                        doEachLoop(host, acc, pass);
                        nextIp(ref host);
                    }
                    nextIpSecondLevel(ref host, j);
                }
                #endregion

            this.BeginInvoke(setStatusText, status, "Done", Color.Blue);
        }

---

Nie uzywacie komentarzy w kodzie?

@Haan zależy czy robię coś skomplikowanego czy nie.

Kod:

---


<?php
/**
 * User: [...]
 * Date: 4/4/14
 * Time: 3:41 PM
 */

namespace Infrastructure\PDO;

/**
 * Class DB
 * @author chriso , edited by [...]
 * @description Provides a lazy-connecting extension around PDO.
 * @see http://stackoverflow.com/questions/5484790/auto-connecting-to-pdo-only-if-needed
 * @package Infrastructure\PDO
 */
class LazyPDOConnection extends \PDO {

    protected $_config = array();

    protected $_connected = false;

    public function __construct($dsn, $user = null, $pass = null, $options = null) {
        //Save connection details for later
        $this->_config = array(
            'dsn' => $dsn,
            'user' => $user,
            'pass' => $pass,
            'options' => $options
        );
    }

    public function checkConnection() {
        if (!$this->_connected) {
            parent::__construct($this->_config['dsn'],$this->_config['user'],$this->_config['pass'],$this->_config['options']);
            $this->_connected = true;
        }
    }

    public function query($query) {
        $this->checkConnection();
        return parent::query($query);
    }

    public function exec($query) {
        $this->checkConnection();
        return parent::exec($query);
    }


    public function commit() {
        $this->checkConnection();
        return parent::commit();
    }

    public function prepare($statement, $driver_options = array()) {
        $this->checkConnection();
        return parent::prepare($statement, $driver_options);
    }

}

---

Lazy-loading dla PDO.

Cytuj:

---

Haan napisał

Nie uzywacie komentarzy w kodzie?

---

@Haan
Było ciężko napisać, ma być ciężko zrozumieć.

Ja komentuje tylko rzeczy niezrozumiale dla mnie, ewentualnie calle do pojebanych metod zewnetrznych, czyli w sumie patrz pt 1, reszta się sama dokumentuje lub proste znalezienie danej metody w klasie wszystko tłumaczy (maksymalnie 1 poziom póki co także nie ma problemów), coś niegrzecznego, ale stworzonego z lenistwa do mojej małej zabawki wrzucę jak wlacze kompa. i może coś jeszcze ciekawego znajdę i przysiade do kodzenia w najblizszym czasie to zobaczymy

Dobry kod OOP sam powinien umieć się udokumentować.
Tak poza tym, to algorytmy warto komentować.

moje "cudo" xD

Kod:

---

class DBO extends PDO
{
    public function query($q, $t=0)
    {
        try
        {
            $z = $this->prepare($q);
            $z->setFetchMode(PDO::FETCH_OBJ);
            $z->execute();
            switch($t)
            {
                case QT::SINGLE:
                    return $z->fetch();
                case QT::MULTI:
                      return $z->fetchAll();
                case QT::NR:
                    return ;//
            }
            return $z->fetch();
        }
        catch(PDOException $e)
        {
            echo $e->getMessage();
        }
        return NULL;
    }
    public function __construct($a, $b, $c)
    {
        parent::__construct($a, $b, $c);
        $this->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );
    }
}

abstract class QT
{
    const SINGLE = 0;
    const MULTI = 1;
    const NR = 2;
}

---

wyjątek traktuję tak, bo to dev mode póki co
ogólnie pewnie niektórych rozbolą oczy czy głowy, ale jak wspominałem: jestem leniwy, a to takie coś, co przede wszystkim ma działać
(jakby jakiś protip inny niż "spal to" to poproszę :D)

W sumie nie rozumiem po co zatracasz funkcjonalnosc query() zamiast dodac nowa ( queryAndFetch xD )

rly, nawet nie zerknąłem, że ma metodę o tej nazwie, no to podmienię :D
anyway, w sumie tylko te 3 co oplotłem swoją funkcją mi są potrzebne. chyba

Kod:

---

class string;

        //Prototype of string class
      _self.init();

 
        proc _self.puts;
        puts(_self);
        end

      proc _self.execute;
        system(_self);
        end

        proc _self.pos(1);
        pos(_self.pos.arg0,_self);       
        return $0

        proc _self.concat(1);
        concat(_self,_self.concat.arg0);
        end
        proc _self.comparestr(1);
        comparestr(_self,_self.comparestr.arg0);
        return $0
        proc _self.comparetext(1);
        comparetext(_self,_self.comparetext.arg0);
        return $0
        proc _self.MsgBox;
        ShowWindow(_self);
        end;



        //Constructor
        proc _self.init
        _self=" ";
      "End"
endclass;

---

Fragment stdliba mojego języka.
Narazie pracuje nad dziedziczeniem, ale utknąłem w pętli między konstruktorami ^^

Kod:

---

            float crossPoint01 =
            (
            ((startPoint.Key * endPoint.Value - startPoint.Value * endPoint.Key) * (startTriangle.vertex0.X - startTriangle.vertex1.X)) -
            ((startTriangle.vertex0.X * startTriangle.vertex1.Y - startTriangle.vertex0.Y * startTriangle.vertex1.X) * (startPoint.Key - endPoint.Key))
            ) / (
            ((startPoint.Key - endPoint.Key) * (startTriangle.vertex0.Y - startTriangle.vertex1.Y)) - (startPoint.Value - endPoint.Value) * (startTriangle.vertex0.X - startTriangle.vertex1.X)
            );

---

Równanie prostych dla boku trójkąta i raya zdefiniowanego przez punkty startPoint i endPoint. Część mojego raytracera, którego uzywam w projekcie.

Prosta klasa dla callbacków.

Kod:

---

#include <vector>

template<class RT, class AT, class OT>
class Callback
{
        typedef RT (*tCallbackF)(AT arg);
        typedef RT (OT::*tCallbackM)(AT arg);

        typedef std::vector<tCallbackF> tCallbackListF;
        typedef std::pair<OT*,tCallbackM> tCallbackMp;
        typedef std::vector<tCallbackMp> tCallbackListM;

        tCallbackListF                        m_callbackListF;
        tCallbackListM                        m_callbackListM;
       
public:
        void registerListener(OT*, tCallbackM method);
        void registerListener(tCallbackF method);

        void notify(AT arg);
       
};

template<class RT, class AT, class OT>
void Callback<RT, AT, OT>::notify(AT arg)
{
        for (auto f : m_callbackListF)
                f(arg);
       
        for (auto m : m_callbackListM)
                (m.first->*m.second)(arg);
}

template<class RT, class AT, class OT>
void Callback<RT, AT, OT>::registerListener(tCallbackF method)
{
        m_callbackListF.push_back(method);
}

template<class RT, class AT, class OT>
void Callback<RT, AT, OT>::registerListener(OT* obj, tCallbackM method)
{
        tCallbackMp pair(obj, method);
        m_callbackListM.push_back(pair);
}

---

Oraz użycie:

Kod:

---

#include "Callback.h"

class Test
{
public:
        Test() { }
        void foo(int x) { std::cout << "Test::foo() called: " << x << std::endl; }
};

void bar(int x)
{
        { std::cout << "bar() called: " << x << std::endl; }
}

int main()
{
        Test test;

        Callback<void, int, Test> callback;
        callback.registerListener(&test, &Test::foo);
        callback.registerListener(&bar);

        callback.notify(10);

        return 0;
}

---

Zadanie na konkurs MasterCoder numer siedem - Algorytm Dijkstry na nietypowym nieskierowanym grafie , kolejka priorytetowa została napisana w oparciu o implementację z neta xD

https://anonfiles.com/file/a886d423f...d67ca4fdc2758e