Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Welcome To Ask or Share your Answers For Others

Categories

0 votes
746 views
in Technique[技术] by (71.8m points)

string - C++: Getting random negative values when converting char to int

Why do I get negative values when I'm trying to convert chars from the string read from file to int?

fin.getline(text, 512);
fin.close();
someInt = (int)text[0]; // It happens to be the random value and always negative.

WHOLE CODE:

The problem itself is in the Decrypt function. When I read the string from file, and convert chars from that string to int, i get negative values, always random, even if characters are the same.

// WUEncryptor.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <iostream>
#include <string>
#include <fstream>

using namespace std;

void Encrypt(string filePath, string password = "");
void Decrypt(string filePath, string password = "");

int main(int argc, char** argv)
{
    int mode = 0;
    string filePath;

    if (argc > 2) {
        mode = atoi(argv[1]);
        filePath = argv[2];

        /** Checking which mode to use */
        switch (mode) {
        case 0:
            if (argc > 3)
                Encrypt(filePath, argv[3]);
            else
                Encrypt(filePath);
            break;

        case 1:
            if (argc > 3)
                Decrypt(filePath, argv[2]);
            else
                Decrypt(filePath);
            break;
        default:
            return 0;
        }
    }
    else {
        /** If something was entered wrong by user, print the error message and quit */
        cout << "Invalid launch..
You should use arguments
For example: "WUEncryptor 0 EncryptedFile
";
    }
    return 0;
}

    /** Encrypting function */
    void Encrypt(string filePath, string password) {
        string text;
        string pass = password;

        /** Quit if pass word isn't determined */
        if (pass == "") {
            cout << "Use "WUEncryptor 0/1 "filepath" password
";
            return;
        }

        cout << "Enter the message which is going to be encrypted
";
        char tempstr[512];
        gets_s(tempstr);
        text = tempstr;
        /** Loop that replaces characters in the text */
        for (int i = 0, m = 0, temp = 0; i < text.length(); i++) {

            temp = (int)text[i] + (int)pass[m];
            cout << temp;
            if (temp > 255)
                temp -= 256;
            cout << temp;
            text[i] = (char)temp;

            /** Looping the pass code */
            if (m >= pass.length() - 1)
                m = 0;
            else
                m++;
            cout << "Encrypting " << i << " of " << text.length() << "...
";
        }
        cout << "Successfully encrypted
TEXT: " << text << '
';

        ofstream fout;
        try {
            filePath += ".encrypted";
            cout << "Creating the file...
";
            fout.open(filePath, ios_base::trunc);
            cout << "Writing to file...
";
            fout << text;
            fout.close();
            cout << "We're all set!
";
            system("pause");
        }
        catch (exception) {
            cout << "Something went wrong";
        }
    }

    /** Decrypting function */
    void Decrypt(string filePath, string password) {
        string text, pass;
        pass = password;
        ifstream fin;

        /** Quit if pass word isn't determined */
        if (pass == "") {
            cout << "Use "WUEncryptor 0/1 "filepath" password
";
            return;
        }
        string openPath = filePath + ".encrypted";

        char tempstr[512];
        try {
            fin.open(openPath);
            fin.getline(&tempstr[0], 512);
            fin.close();
            text = tempstr;
            //cout << (int)text[1];

            for (int i = 0, m = 0, temp = 0; i < text.length(); i++) {

                temp = (int)text[i] - (int)pass[m];
                cout << temp;
                if (temp < 0) 
                    temp += 256;
                cout << temp;
                text[i] = (char)temp;

                /** Looping the pass code */
                if (m >= pass.length() - 1)
                    m = 0;
                else
                    m++;
                cout << "Decrypting " << i << " of " << text.length() << "...
";
            }
            cout << "Successfully decrypted
TEXT: " << text << '
';

        }
        catch (exception) {}

        ofstream fout;
        try {
            string savePath = filePath + ".txt";
            cout << "Creating the file...
";
            fout.open(savePath, ios_base::trunc);
            cout << "Writing to file...
";
            fout << text;
            fout.close();
            cout << "We're all set!
";
            system("pause");
        }
        catch (exception) {
            cout << "Something went wrong";
        }

    }

Someone, clear my mind :D

See Question&Answers more detail:os

与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
Welcome To Ask or Share your Answers For Others

1 Reply

0 votes
by (71.8m points)

You're probably using char* as datatype for text. Note that char is signed, so if you (think you) have a value of e.g. 200 there, it's really -72, and so if you cast that to int, you get -72 :)


与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…
OGeek|极客中国-欢迎来到极客的世界,一个免费开放的程序员编程交流平台!开放,进步,分享!让技术改变生活,让极客改变未来! Welcome to OGeek Q&A Community for programmer and developer-Open, Learning and Share
Click Here to Ask a Question

...