Ich programiere schlecht!

[elk]

Hi,
Ich mache grad ein 2d weihnachtsmannbattle ( 2 weihnachtsmänner werden über die tastatur gesteuert und werfen sich mit schnebällen ab).

So habe ich das gemacht:
-Ein Schneeball (mc_sr1) bekommt die weihnachtsmannsteuerung.
-Wenn der Schneeball auf ein Hinderniss trifft geht er wieder in den weihnachtsmann(mc_santaR).
-wenn weihnachtsmann auf hinderniss trifft wird er ein bissl zurückgestuppst

Das finde ich nciht so toll, da man nicht mehrmals hintereinander schießen kann. Außerdem siht der quelltext echt ******e aus ( geht aber )


Kann mir einer Sagen wie ich die Schiessteuerung besser hinbekomme als oben beschrieben?

PHP-Code:

var i = 1;
var xy = 1;
var k = 1;
var M = 1;
this.mc_santar.onEnterFrame = function() {
    if (Key.isDown(Key.RIGHT)) {
        this._x += 5;
        _root.mc_santar.gotoAndStop(2);
    }
    if (Key.isDown(Key.LEFT)) {
        this._x -= 5;
        _root.mc_santar.gotoAndStop(3);
    }
    if (Key.isDown(Key.UP)) {
        this._y -= 5;
        _root.mc_santar.gotoAndStop(1);
    }
    if (Key.isDown(Key.DOWN)) {
        this._y += 5;
        _root.mc_santar.gotoAndStop(4);
    }
    if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.RIGHT))) {
        this._y -= 2;
        this._x -= 2;
        _root.mc_santar.gotoAndStop(5);
    }
    if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.LEFT))) {
        this._y -= 2;
        this._x += 2;
        _root.mc_santar.gotoAndStop(6);
    }
    if ((Key.isDown(Key.UP)) and (Key.isDown(Key.RIGHT))) {
        this._y += 2;
        this._x -= 2;
        _root.mc_santar.gotoAndStop(7);
    }
    if ((Key.isDown(Key.UP)) and (Key.isDown(Key.LEFT))) {
        this._y += 2;
        this._x += 2;
        _root.mc_santar.gotoAndStop(8);
    }
};
this.mc_sr1.onEnterFrame = function() {
    if (_root.mc_santar.hitTest(_root.mc_sr1._x, _root.mc_sr1._y, true)) {
        if (Key.isDown(Key.RIGHT)) {
            this._x += 5;
        }
        if (Key.isDown(Key.LEFT)) {
            this._x -= 5;
        }
        if (Key.isDown(Key.UP)) {
            this._y -= 5;
        }
        if (Key.isDown(Key.DOWN)) {
            this._y += 5;
        }
        if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.RIGHT))) {
            this._y -= 2;
            this._x -= 2;
        }
        if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.LEFT))) {
            this._y -= 2;
            this._x += 2;
        }
        if ((Key.isDown(Key.UP)) and (Key.isDown(Key.RIGHT))) {
            this._y += 2;
            this._x -= 2;
        }
        if ((Key.isDown(Key.UP)) and (Key.isDown(Key.LEFT))) {
            this._y += 2;
            this._x += 2;
        }
    }
};
keyListener = new Object();
Key.addListener(keyListener);
keyListener = new Object();
keyListener.onKeyUp = function() {
    if (Key.getCode() == Key.SPACE) {
        boom();
    }
};
Key.addListener(keyListener);
function boom() {
    if (_root.mc_santar.hitTest(_root.mc_sr1._x, _root.mc_sr1._y, true)) {
        if (_root.x == 1) {
            //trace("oben");
            schuss(1, 0, 0, 0, 0, 0, 0, 0);
        }
        if (_root.x == 2) {
            //trace("rechts");
            schuss(0, 1, 0, 0, 0, 0, 0, 0);
        }
        if (_root.x == 3) {
            //trace("links");
            schuss(0, 0, 1, 0, 0, 0, 0, 0);
        }
        if (_root.x == 4) {
            //trace("unten");
            schuss(0, 0, 0, 1, 0, 0, 0, 0);
        }
        if (_root.x == 5) {
            //trace("ru");
            schuss(0, 0, 0, 0, 1, 0, 0, 0);
        }
        if (_root.x == 6) {
            //trace("lu");
            schuss(0, 0, 0, 0, 0, 1, 0, 0);
        }
        if (_root.x == 7) {
            //trace("ro");
            schuss(0, 0, 0, 0, 0, 0, 1, 0);
        }
        if (_root.x == 8) {
            //trace("lo");
            schuss(0, 0, 0, 0, 0, 0, 0, 1);
        }
    }
}
function schuss(o, r, l, u, ru, lu, ro, lo) {
    onEnterFrame = function () {
        if (_root.mc_map.hitTest(_root.mc_santar._x, _root.mc_santar._y, true)) {
            if (_root.x == 1) {
                _root.mc_santar._y = _root.mc_santar._y+5;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
            if (_root.x == 2) {
                _root.mc_santar._x = _root.mc_santar._x-5;
                _root.mc_sr1._x = _root.mc_santar._x;
            }
            if (_root.x == 3) {
                _root.mc_santar._x = _root.mc_santar._x+5;
                _root.mc_sr1._x = _root.mc_santar._x;
            }
            if (_root.x == 4) {
                _root.mc_santar._y = _root.mc_santar._y-5;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
            if (_root.x == 5) {
                _root.mc_santar._x = _root.mc_santar._x-5;
                _root.mc_santar._y = _root.mc_santar._y-5;
                _root.mc_sr1._x = _root.mc_santar._x;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
            if (_root.x == 6) {
                _root.mc_santar._x = _root.mc_santar._x+5;
                _root.mc_santar._y = _root.mc_santar._y-5;
                _root.mc_sr1._x = _root.mc_santar._x;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
            if (_root.x == 7) {
                _root.mc_santar._x = _root.mc_santar._x-5;
                _root.mc_santar._y = _root.mc_santar._y+5;
                _root.mc_sr1._x = _root.mc_santar._x;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
            if (_root.x == 8) {
                _root.mc_santar._x = _root.mc_santar._x+5;
                _root.mc_santar._y = _root.mc_santar._y+5;
                _root.mc_sr1._x = _root.mc_santar._x;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
        }
        if ((_root.mc_map.hitTest(_root.mc_sr1._x, _root.mc_sr1._y, true)) or (_root.mc_santab.hitTest(_root.mc_sr1._x, _root.mc_sr1._y, true))) {
            if (_root.mc_santab.hitTest(_root.mc_sr1._x, _root.mc_sr1._y, true)) {
                tehitr.text = i;
                i = i+1;
            }
            if (k == 1) {
                k = 2;
                _root.mc_sr1._x = _root.mc_santar._x;
                _root.mc_sr1._y = _root.mc_santar._y;
            }
        } else if ((k == 1) and (l == 1)) {
            _root.mc_sr1._x -= 10;
        } else if ((k == 1) and (r == 1)) {
            _root.mc_sr1._x += 10;
        } else if ((k == 1) and (o == 1)) {
            _root.mc_sr1._y -= 10;
        } else if ((k == 1) and (u == 1)) {
            _root.mc_sr1._y += 10;
        } else if ((k == 1) and (lo == 1)) {
            _root.mc_sr1._y -= 5;
            _root.mc_sr1._x -= 5;
        } else if ((k == 1) and (lu == 1)) {
            _root.mc_sr1._y += 5;
            _root.mc_sr1._x -= 5;
        } else if ((k == 1) and (ro == 1)) {
            _root.mc_sr1._y -= 5;
            _root.mc_sr1._x += 5;
        } else if ((k == 1) and (ru == 1)) {
            _root.mc_sr1._y += 5;
            _root.mc_sr1._x += 5;
        }
    };
    k = 1;
} 


[PinkDragon]

Wow netter Quelltext :P

Zuerstmal wegen der Schusssteuerung:
Lass den Schneeball am Anfang in der bibliothek und gib ihm eine Verknüpfung. (ich nehm jetzt einfach "ball")
Danach würde ich es so in der Art machen:

PHP-Code:

var santa = this.mc_santar;
santa.xspeed = 0;
santa.yspeed = 0;

santa.onEnterFrame = function(){
    if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.RIGHT))) { 
        this.yspeed = 2; 
        this.xspeed = -2; 
        this.gotoAndStop(5); 
    }//Die else sind damit er, wenn keine Taste gedrückt ist, stehen bleibt.
    else if ((Key.isDown(Key.DOWN)) and (Key.isDown(Key.LEFT))) { 
        this.yspeed = 2; 
        this.xspeed = -2; 
        _root.mc_santar.gotoAndStop(6); 
    }
    else if ((Key.isDown(Key.UP)) and (Key.isDown(Key.RIGHT))) { 
        this.yspeed = -2; 
        this.xspeed = 2; 
        this.gotoAndStop(7); 
    } 
    else if ((Key.isDown(Key.UP)) and (Key.isDown(Key.LEFT))) { 
        this.yspeed = -2; 
        this.xspeed = -2; 
        this.gotoAndStop(8); 
    }
    else if (Key.isDown(Key.RIGHT)) { 
        this.xspeed = 5; 
        this.gotoAndStop(2); 
    } 
    else if (Key.isDown(Key.LEFT)) { 
        this.xspeed = -5; 
        this.gotoAndStop(3); 
    } 
    else if (Key.isDown(Key.UP)) { 
        this.yspeed = -5; 
        this.gotoAndStop(1); 
    } 
    else if (Key.isDown(Key.DOWN)) { 
        this.yspeed = 5; 
        this.gotoAndStop(4); 
    }
    else{
       this.xspeed = 0;
       this.yspeed = 0;
       this.gotoAndStop("stand");
    }
  this._x += xspeed;
  this._y += yspeed;
}

function boom(){
  var depth = _root.getNextHighestDepth();
  var ball = _root.attachMovie("ball","ball"+depth,depth);
  ball._x = _root.mc_santar._x;
  ball._y = _root.mc_santar._y;
  ball.xspeed = _root.mc_santar.xspeed*2;
  ball.yspeed = _root.mc_santar.yspeed*2;
  ball.onEnterFrame = function(){
    this._x += this.xspeed;
    this._y += this.yspeed;
    if ((_root.mc_map.hitTest(this._x, this._y, true)) or (_root.mc_santab.hitTest(this._x, this._y, true))){
     [mach was auch immer]
     this.removeMovieClip();
  }
} 


Habs nicht getested, müsste aber so in der Art funktionieren.

mfg
PD

PS: Nein der Script ist auch nicht gerade schön, aber dafür war er schnell geschrieben

[elk]

hm supa!

nur ein problem hab ich mit deiner steuerung.

Wenn ich z.b. links und oben zugleich drücke -- dann links loslasse--- bewegt er sicht nicht nach oben sodnern schräg nach links oben, da er den xwert nicht zurücksätzt ( das else kann ja nicht anspringen, da immerhin noch 1 taste gedrückt wird)

[PinkDragon]

Hab ja gesagt es geht nur so in der Art. Normalwerweise geh ich davon aus, dass man, wenn man in Flash etwas programmiert auch im Stande ist selber zu Denken...
Aber weil ich heute guter Laune bin, kriegste sogar ne halbe Lösung:

PHP-Code:

 else if (Key.isDown(Key.LEFT)) {  
     this.xspeed = -5;
     this.yspeed = 0;
     this.gotoAndStop(3);
} 


Und ausserdem bekommste noch nen kleinen Tipp: Es geht auch nicht stehen zu bleiben und dann zu schießen
Aber ich bin mir sicher, dass du auch das mit ein bischen Denken schaffst zu lösen.

mfg
PD

[elk]

Hier stand erst was anderes -- habs aber geändert, da ich jetzt raus habe. Ein neues Prob--

attach movies kann ich irgendwie nicht ansprechen bzw einen Hittest mit ihnen machen. In deinem Beispiel musste folgendes doch gehen oder... warum gehts net?

PHP-Code:

if (_root.mc_map.hitTest(this._x, this._y, true)){ 
 trace("hallo"); 
this.removeMovieClip(); 
  } 


ich hab für this auch schon mal _root.ball oder so genommen .. klappt aber alles net

[PinkDragon]

Was genau geht nicht? Wird die Funktion nicht aufgerufen? Wird die If-Abfrage nicht wahr? Wird der trace nicht ausgegeben? Wird der MovieClip nicht entfernt?
Lass einfach mal folgendes ausgeben:

PHP-Code:

trace("MC: "+this);
var hit = _root.mc_map.hitTest(this._x, this._y, true);
trace("Hit: "+hit);
if (hit){  
  trace("hallo");
  trace("MC: "+this);
  this.removeMovieClip();
} 


Ausgabe sollte bestenfalls in etwa so aussehen:

PHP-Code:

MC: _leveo0.ballx
Hit: true
hallo
MC: _level0.ballx 


[elk]

Bei meinem hallo hat er nichts ausgegeben.

bei dir gibt er das aus:

MC: _level0
Hit: false

[PinkDragon]

Dann schau mal ob deine Funktion

PHP-Code:

ball.onEnterFrame = function(){} 


stimmt. Gib einfach mal nur

PHP-Code:

ball.onEnterFrame = function(){
  trace(this);
} 


Hmm da fällt mir gerade etwas auf: Hast du mein bsp im kontext probiert oder extra? sollte den onClipEvent einfach ersetzen:

PHP-Code:

 ball.onEnterFrame = function(){
  trace("MC: "+this); 
  var hit = _root.mc_map.hitTest(this._x, this._y, true); 
  trace("Hit: "+hit); 
  if (hit){   
    trace("hallo"); 
    trace("MC: "+this); 
    this.removeMovieClip(); 
  }
} 


Ohne das "ball.onEnterFrame = function()" kann es nicht funktionieren da sonst das "this" nicht der Ball sondern die Bühne ist.

[elk]

hatte ein grundsätzliches Problem in ner Logic--- hat sich jetzt geklährt