Dev Help - trying to deploy first smart contract on Move

Development
1

Here is the code. I get error message "Error: {“message”:“PartialVMError with status INVALID_MAIN_FUNCTION_SIGNATURE”,“error_code”:“invalid_input”,“vm_error_code”:null}
"

module two_sum::addition {
    use std::signer;
    use std::error;

    struct NumbersHolder has key {
        num1: u64,
        num2: u64,
    }

    const ENO_NUMBERS: u64 = 0;

    #[view]
    public fun get_sum(account: signer): u64 acquires NumbersHolder {
        let account_addr = signer::address_of(&account);
        assert!(exists<NumbersHolder>(account_addr), error::not_found(ENO_NUMBERS));
        let holder = borrow_global<NumbersHolder>(account_addr);
        holder.num1 + holder.num2
    }

    public entry fun set_numbers(account: signer, a: u64, b: u64) acquires NumbersHolder {
        let account_addr = signer::address_of(&account);
        if (!exists<NumbersHolder>(account_addr)) {
            move_to(&account, NumbersHolder { num1: a, num2: b })
        } else {
            let holder = borrow_global_mut<NumbersHolder>(account_addr);
            holder.num1 = a;
            holder.num2 = b;
        }
    }


    #[test(account = @0x1)]
    public entry fun test_set_numbers_and_get_sum(account: signer) acquires NumbersHolder {
        let addr = signer::address_of(&account);
        set_numbers(account, 5, 10);
        assert!(get_sum(account) == 15, ENO_NUMBERS);
    }
}
46 Likes
2

I tried your code on Aptos assistant and it provided the following as corrected code:

module two_sum::addition {
use std::signer;
use std::error;

struct NumbersHolder has key {
    num1: u64,
    num2: u64,
}

const ENO_NUMBERS: u64 = 0;

#[view]
public fun get_sum(account: signer): u64 acquires NumbersHolder {
    let account_addr = signer::address_of(&account);
    assert!(exists<NumbersHolder>(account_addr), error::not_found(ENO_NUMBERS));
    let holder = borrow_global<NumbersHolder>(account_addr);
    holder.num1 + holder.num2
}

public entry fun set_numbers(account: signer, a: u64, b: u64) acquires NumbersHolder {
    let account_addr = signer::address_of(&account);
    if (!exists<NumbersHolder>(account_addr)) {
        move_to(&account, NumbersHolder { num1: a, num2: b })
    } else {
        let holder = borrow_global_mut<NumbersHolder>(account_addr);
        holder.num1 = a;
        holder.num2 = b;
    }
}

#[test(account = @0x1)]
public entry fun test_set_numbers_and_get_sum(account: signer) acquires NumbersHolder {
    let addr = signer::address_of(&account);
    set_numbers(account, 5, 10);
    assert!(get_sum(account) == 15, ENO_NUMBERS);
}

}

28 Likes
3

thank you will try this! appreciate the help.

14 Likes
4

hey Chiru did it work?

14 Likes
5

This is a great showcasing of Aptos Assistant - I will have to look further into this when testing my own modules.

12 Likes
6

good one, i hope you get the help you seek friend

9 Likes
7

this lot of effort, I wish you the best my friend

9 Likes
8

all the best fren

10 Likes
9

Go to the node section .

6 Likes
10

For any help with Move, feel free to check out the dev discussion channel of the Aptos official server.

You just need to get a dev role first in the choose role channel to see it. The Aptos Assistant is a very useful tool. Just make sure to double check its answers by asking it if it is sure about the answer. Some of these chatbots can hallucinate at times so also check in with the dev discussion chat to make sure the code answer is a solid one.

Happy learning !

9 Likes
11

best code cc

4 Likes
12

thank you!

4 Likes
13

hey what resources did you use to learn Move initially? (OP or anyone else in this thread) trying to get started but having a tough time tbh! :frowning:

5 Likes
14

commenting to follow this thread. trying to start on Move dev also.

7 Likes
15

Ok, update - with no change to the code, but just fixing that indentation error (based on what was shared in this thread)… still getting the same error message.

5 Likes
16

Wow this so amazing.

5 Likes
17

Very useful thread!

5 Likes
18

thanks! appreciate it :smiley:

4 Likes
19

I tried this and it worked - try this code instead, and let me know if you figure out what changed that made it work :slight_smile:

module two_sum_v2::additionv2 {
  use std::signer;
  use std::error;
  use aptos_framework::account;
  use aptos_framework::event;

  struct NumbersHolder has key {
      num1: u64,
      num2: u64,
      numbers_change_events: event::EventHandle<NumbersChangeEvent>,
  }

  struct NumbersChangeEvent has drop, store {
    from_num1: u64,
    from_num2: u64,
    to_num1: u64,
    to_num2: u64,
  }

  const ENO_NUMBERS: u64 = 1001; // Use a unique non-zero error code

  #[view]
  public fun get_sum(addr: address): u64 acquires NumbersHolder {
    assert!(exists<NumbersHolder>(addr), error::not_found(ENO_NUMBERS));
    *&borrow_global<NumbersHolder>(addr).num1 + *&borrow_global<NumbersHolder>(addr).num2
  }

  public entry fun set_numbers(account: signer, a: u64, b: u64) acquires NumbersHolder {
      let account_addr = signer::address_of(&account);
      if (!exists<NumbersHolder>(account_addr)) {
          move_to(&account, NumbersHolder { num1: a, num2: b, numbers_change_events: account::new_event_handle<NumbersChangeEvent>(&account), 
        })
      } else {
          let old_numbers_holder = borrow_global_mut<NumbersHolder>(account_addr);
          let from_num1 = *&old_numbers_holder.num1;
          let from_num2 = *&old_numbers_holder.num2;
          event::emit_event(&mut old_numbers_holder.numbers_change_events, NumbersChangeEvent {
              from_num1,
              from_num2,
              to_num1: copy a,
              to_num2: copy b,
          });
          old_numbers_holder.num1 = a;
          old_numbers_holder.num2 = b;
      }
  }

  #[test(account = @0x1)]
  public entry fun test_set_numbers_and_get_sum(account: signer) acquires NumbersHolder {
      let addr = signer::address_of(&account);
      aptos_framework::account::create_account_for_test(addr);
      set_numbers(account, 5, 10);
      assert!(get_sum(addr) == 15, ENO_NUMBERS);
  }
}
6 Likes
20

Hey folks! I just got this to deploy correctly via MultiSig using M-safe, Petra and Pontem wallets.

Take a look and see how it works for you! Maybe we can work together on more “advanced Mathematics” stuff if anyone finds this interesting.

6 Likes